ixgbe: drop null test before destroy functions

Remove unneeded NULL test.

The semantic patch that makes this change is as follows:
(http://coccinelle.lip6.fr/)

// <smpl>
@@ expression x; @@
-if (x != NULL)
  \(kmem_cache_destroy\|mempool_destroy\|dma_pool_destroy\)(x);
// </smpl>

Signed-off-by: Julia Lawall <[email protected]>
Tested-by: Darin Miller <[email protected]>
Signed-off-by: Jeff Kirsher <[email protected]>

diff --git a/drivers/net/ethernet/intel/ixgbe/ixgbe_fcoe.c b/drivers/net/ethernet/intel/ixgbe/ixgbe_fcoe.c
index 631c603fc96649c95995b286577ca7b489d3f58d..5f988703e1b7e464a75b936bf7fb1fbb7f997b2c 100644 (file)
--- a/drivers/net/ethernet/intel/ixgbe/ixgbe_fcoe.c
+++ b/drivers/net/ethernet/intel/ixgbe/ixgbe_fcoe.c
@@ -620,8 +620,7 @@ static void ixgbe_fcoe_dma_pool_free(struct ixgbe_fcoe *fcoe, unsigned int cpu)
        struct ixgbe_fcoe_ddp_pool *ddp_pool;
 
        ddp_pool = per_cpu_ptr(fcoe->ddp_pool, cpu);
-       if (ddp_pool->pool)
-               dma_pool_destroy(ddp_pool->pool);
+       dma_pool_destroy(ddp_pool->pool);
        ddp_pool->pool = NULL;
 }