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>

diff --git a/drivers/net/ethernet/brocade/bna/bnad.c b/drivers/net/ethernet/brocade/bna/bnad.c
index ff78f770dec91e12e9a816c413b768d31481acd4..d86390cc033ea8d4baa4aad56a3925808a7e213e 100644 (file)
--- 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);