Browse Source

typhoon: get rid of redundant conditional before all to release_firmware()

There's no need to test for a NULL pointer before calling
release_firmware() since the function does that check itself, so
remove the redundant test.

Signed-off-by: Jesper Juhl <jj@chaosbits.net>
Acked-by: David Dillow <dave@thedillows.org>
Signed-off-by: Jiri Kosina <jkosina@suse.cz>
Jesper Juhl 13 years ago
parent
commit
b2cbf2e31b
1 changed files with 1 additions and 2 deletions
  1. 1 2
      drivers/net/ethernet/3com/typhoon.c

+ 1 - 2
drivers/net/ethernet/3com/typhoon.c

@@ -2549,8 +2549,7 @@ typhoon_init(void)
 static void __exit
 typhoon_cleanup(void)
 {
-	if (typhoon_fw)
-		release_firmware(typhoon_fw);
+	release_firmware(typhoon_fw);
 	pci_unregister_driver(&typhoon_driver);
 }