ipw2200: remove a redundant NULL check before calling release_firmware()
authorJesper Juhl <jj@chaosbits.net>
Mon, 9 Apr 2012 20:51:01 +0000 (22:51 +0200)
committerJohn W. Linville <linville@tuxdriver.com>
Thu, 12 Apr 2012 19:06:07 +0000 (15:06 -0400)
The release_firmware() function does its own NULL test, so testing
before calling it is rather redundant.

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

index 2b022571a8595a5dd41c89bdd515d0f1fd0dc70b..77c5d2f5115a7373d76eb6a190faeaa4c0465749 100644 (file)
@@ -3657,8 +3657,7 @@ static int ipw_load(struct ipw_priv *priv)
                priv->rxq = NULL;
        }
        ipw_tx_queue_free(priv);
-       if (raw)
-               release_firmware(raw);
+       release_firmware(raw);
 #ifdef CONFIG_PM
        fw_loaded = 0;
        raw = NULL;