From: Insu Yun <wuninsu@gmail.com>
Date: Tue, 16 Feb 2016 02:23:47 +0000 (-0500)
Subject: et131x: check return value of dma_alloc_coherent
X-Git-Tag: v4.9.8~2654^2~36
X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=562a9f91a0300ecaaf28722f57bcdd9cd5f38f0e;p=platform%2Fkernel%2Flinux-rpi3.git

et131x: check return value of dma_alloc_coherent

For error handling, dma_alloc_coherent's return value
needs to be checked, not argument.

Signed-off-by: Insu Yun <wuninsu@gmail.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
---

diff --git a/drivers/net/ethernet/agere/et131x.c b/drivers/net/ethernet/agere/et131x.c
index 3f3bcbe..0907ab6 100644
--- a/drivers/net/ethernet/agere/et131x.c
+++ b/drivers/net/ethernet/agere/et131x.c
@@ -2380,7 +2380,7 @@ static int et131x_tx_dma_memory_alloc(struct et131x_adapter *adapter)
 						    sizeof(u32),
 						    &tx_ring->tx_status_pa,
 						    GFP_KERNEL);
-	if (!tx_ring->tx_status_pa) {
+	if (!tx_ring->tx_status) {
 		dev_err(&adapter->pdev->dev,
 			"Cannot alloc memory for Tx status block\n");
 		return -ENOMEM;