summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorJesper Juhl <jj@chaosbits.net>2012-04-10 00:50:24 +0400
committerJiri Kosina <jkosina@suse.cz>2012-04-30 15:18:51 +0400
commit294ca868ed598519cab1cd699873378d46733198 (patch)
treeb6893d1ebdcf51abd9df1bfc434a320f401a1273
parente3c5530b96b9c5994cfb2544a037161790c4f7c3 (diff)
downloadlinux-294ca868ed598519cab1cd699873378d46733198.tar.xz
bna: remove redundant NULL test before release_firmware()
release_firmware() does its own NULL test so explicit test before call is unneeded. Signed-off-by: Jesper Juhl <jj@chaosbits.net> Acked-by: Rasesh Mody <rmody@brocade.com> Signed-off-by: Jiri Kosina <jkosina@suse.cz>
-rw-r--r--drivers/net/ethernet/brocade/bna/bnad.c4
1 files changed, 1 insertions, 3 deletions
diff --git a/drivers/net/ethernet/brocade/bna/bnad.c b/drivers/net/ethernet/brocade/bna/bnad.c
index ff78f770dec9..d86390cc033e 100644
--- a/drivers/net/ethernet/brocade/bna/bnad.c
+++ b/drivers/net/ethernet/brocade/bna/bnad.c
@@ -3546,9 +3546,7 @@ static void __exit
bnad_module_exit(void)
{
pci_unregister_driver(&bnad_pci_driver);
-
- if (bfi_fw)
- release_firmware(bfi_fw);
+ release_firmware(bfi_fw);
}
module_init(bnad_module_init);