summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorInsu Yun <wuninsu@gmail.com>2016-02-16 03:23:47 +0100
committerDavid S. Miller <davem@davemloft.net>2016-02-18 05:32:05 +0100
commit562a9f91a0300ecaaf28722f57bcdd9cd5f38f0e (patch)
treefa67d5b783611231fef0f49e4c4a614c5ac5ce81
parentMerge branch 'thunderx-fixes' (diff)
downloadlinux-562a9f91a0300ecaaf28722f57bcdd9cd5f38f0e.tar.xz
linux-562a9f91a0300ecaaf28722f57bcdd9cd5f38f0e.zip
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>
-rw-r--r--drivers/net/ethernet/agere/et131x.c2
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/net/ethernet/agere/et131x.c b/drivers/net/ethernet/agere/et131x.c
index 3f3bcbea15bd..0907ab6ff309 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;