diff options
| author |   Insu Yun <wuninsu@gmail.com> | 2016-02-15 21:23:47 -0500 |
|---|---|---|
| committer |   David S. Miller <davem@davemloft.net> | 2016-02-17 23:32:05 -0500 |
| commit | 562a9f91a0300ecaaf28722f57bcdd9cd5f38f0e (patch) | |
| tree | fa67d5b783611231fef0f49e4c4a614c5ac5ce81 /drivers/net/ethernet/agere/et131x.c | |
| parent | Merge branch 'thunderx-fixes' (diff) | |
| download | linux-dev-562a9f91a0300ecaaf28722f57bcdd9cd5f38f0e.tar.xz linux-dev-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>
Diffstat (limited to 'drivers/net/ethernet/agere/et131x.c')
| -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; |