diff options
author | Syam Sidhardhan <s.syam@samsung.com> | 2013-03-06 20:14:54 (GMT) |
---|---|---|
committer | Greg Kroah-Hartman <gregkh@linuxfoundation.org> | 2013-03-11 16:43:44 (GMT) |
commit | d39625c4eb3349569f0959a4488e71faa927ec4a (patch) | |
tree | 34edbfbb50e5d4843cc82986a32392e5f48718c4 | |
parent | f54ab7d916ee4504e91b552c38cfa2f82df3718d (diff) | |
download | linux-fsl-qoriq-d39625c4eb3349569f0959a4488e71faa927ec4a.tar.xz |
staging: silicom: Remove redundant NULL check before kfree
kfree on NULL pointer is a no-op.
Signed-off-by: Syam Sidhardhan <s.syam@samsung.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
-rw-r--r-- | drivers/staging/silicom/bp_mod.c | 6 |
1 files changed, 2 insertions, 4 deletions
diff --git a/drivers/staging/silicom/bp_mod.c b/drivers/staging/silicom/bp_mod.c index 61a912d..45a2227 100644 --- a/drivers/staging/silicom/bp_mod.c +++ b/drivers/staging/silicom/bp_mod.c @@ -6995,15 +6995,13 @@ static void __exit bypass_cleanup_module(void) /* unmap all devices */ for (i = 0; i < device_num; i++) { #ifdef BP_SELF_TEST - if (bpctl_dev_arr[i].bp_tx_data) - kfree(bpctl_dev_arr[i].bp_tx_data); + kfree(bpctl_dev_arr[i].bp_tx_data); #endif iounmap((void *)(bpctl_dev_arr[i].mem_map)); } /* free all devices space */ - if (bpctl_dev_arr) - kfree(bpctl_dev_arr); + kfree(bpctl_dev_arr); /* * Unregister the device |