diff options
author | Insu Yun <wuninsu@gmail.com> | 2016-02-16 03:23:47 +0100 |
---|---|---|
committer | David S. Miller <davem@davemloft.net> | 2016-02-18 05:32:05 +0100 |
commit | 562a9f91a0300ecaaf28722f57bcdd9cd5f38f0e (patch) | |
tree | fa67d5b783611231fef0f49e4c4a614c5ac5ce81 | |
parent | Merge branch 'thunderx-fixes' (diff) | |
download | linux-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.c | 2 |
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; |