wireless, atmel: remove pointless test for NULL before release_firmware() call
authorJesper Juhl <jj@chaosbits.net>
Mon, 9 Apr 2012 20:50:53 +0000 (22:50 +0200)
committerJohn W. Linville <linville@tuxdriver.com>
Thu, 12 Apr 2012 19:06:07 +0000 (15:06 -0400)
release_firmware() does its own test. Explicitly checking before the
call is redundant.

Signed-off-by: Jesper Juhl <jj@chaosbits.net>
Signed-off-by: John W. Linville <linville@tuxdriver.com>
drivers/net/wireless/atmel.c

index 6c87a823f5a9d4c6ec228651687b14c7fb545c8e..d07c0301da6abeca20f1aa8fbbaf1cf0b8ae4d32 100644 (file)
@@ -3989,8 +3989,7 @@ static int reset_atmel_card(struct net_device *dev)
                        atmel_copy_to_card(priv->dev, 0x8000, &fw[0x6000], len - 0x6000);
                }
 
-               if (fw_entry)
-                       release_firmware(fw_entry);
+               release_firmware(fw_entry);
        }
 
        err = atmel_wakeup_firmware(priv);