From: Ashish Kumar <Ashish.Kumar@nxp.com>
Date: Thu, 26 Oct 2017 10:33:22 +0000 (+0530)
Subject: drivers: net: ldpaa_eth: Correct error handler for qbman_swp_acquire()
X-Git-Tag: v2018.01-rc1~127^2~12
X-Git-Url: https://git.sur5r.net/?a=commitdiff_plain;h=5e9445da288c5121546235e8201768abd087b281;p=u-boot

drivers: net: ldpaa_eth: Correct error handler for qbman_swp_acquire()

The zero value returned from qbman_swp_acquire() is an error
condition meaning no free buffer for allocation.

Signed-off-by: Ashish Kumar <Ashish.Kumar@nxp.com>
Signed-off-by: Kushwaha Prabhakar <prabhakar@freescale.com>
[YS: revised commit message]
Reviewed-by: York Sun <york.sun@nxp.com>
---

diff --git a/drivers/net/ldpaa_eth/ldpaa_eth.c b/drivers/net/ldpaa_eth/ldpaa_eth.c
index f235b622b6..21be79a58a 100644
--- a/drivers/net/ldpaa_eth/ldpaa_eth.c
+++ b/drivers/net/ldpaa_eth/ldpaa_eth.c
@@ -334,7 +334,7 @@ static int ldpaa_eth_tx(struct eth_device *net_dev, void *buf, int len)
 					&buffer_start, 1);
 	} while (err == -EBUSY);
 
-	if (err < 0) {
+	if (err <= 0) {
 		printf("qbman_swp_acquire() failed\n");
 		return -ENOMEM;
 	}