Commit 7af348be authored by Markus Elfring's avatar Markus Elfring Committed by David S. Miller

net: ep93xx_eth: Delete unnecessary checks before the function call "kfree"

The kfree() function tests whether its argument is NULL and then
returns immediately. Thus the test around the call is not needed.

This issue was detected by using the Coccinelle software.
Signed-off-by: default avatarMarkus Elfring <elfring@users.sourceforge.net>
Signed-off-by: default avatarH Hartley Sweeten <hsweeten@visionengravers.com>
Signed-off-by: default avatarDavid S. Miller <davem@davemloft.net>
parent 27b917e5
...@@ -475,8 +475,7 @@ static void ep93xx_free_buffers(struct ep93xx_priv *ep) ...@@ -475,8 +475,7 @@ static void ep93xx_free_buffers(struct ep93xx_priv *ep)
if (d) if (d)
dma_unmap_single(dev, d, PKT_BUF_SIZE, DMA_FROM_DEVICE); dma_unmap_single(dev, d, PKT_BUF_SIZE, DMA_FROM_DEVICE);
if (ep->rx_buf[i] != NULL) kfree(ep->rx_buf[i]);
kfree(ep->rx_buf[i]);
} }
for (i = 0; i < TX_QUEUE_ENTRIES; i++) { for (i = 0; i < TX_QUEUE_ENTRIES; i++) {
...@@ -486,8 +485,7 @@ static void ep93xx_free_buffers(struct ep93xx_priv *ep) ...@@ -486,8 +485,7 @@ static void ep93xx_free_buffers(struct ep93xx_priv *ep)
if (d) if (d)
dma_unmap_single(dev, d, PKT_BUF_SIZE, DMA_TO_DEVICE); dma_unmap_single(dev, d, PKT_BUF_SIZE, DMA_TO_DEVICE);
if (ep->tx_buf[i] != NULL) kfree(ep->tx_buf[i]);
kfree(ep->tx_buf[i]);
} }
dma_free_coherent(dev, sizeof(struct ep93xx_descs), ep->descs, dma_free_coherent(dev, sizeof(struct ep93xx_descs), ep->descs,
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment