diff options
author | Jesper Juhl <jj@chaosbits.net> | 2012-04-09 16:52:34 -0400 |
---|---|---|
committer | John W. Linville <linville@tuxdriver.com> | 2012-04-12 15:06:08 -0400 |
commit | e3e07e0b10b3e6ca5cc7f6f4bf28d3419a9a12e4 (patch) | |
tree | 673ddb591b434e30a330689ed973035de99f6eb6 /drivers | |
parent | f5123117091995b94e17c44a43cbf4b5dd736319 (diff) |
ipw2100: remove a redundant NULL check before calling release_firmware()
The release_firmware() function does its own NULL test so a test
before calling it is rather redundant.
Signed-off-by: Jesper Juhl <jj@chaosbits.net>
Signed-off-by: John W. Linville <linville@tuxdriver.com>
Diffstat (limited to 'drivers')
-rw-r--r-- | drivers/net/wireless/ipw2x00/ipw2100.c | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/drivers/net/wireless/ipw2x00/ipw2100.c b/drivers/net/wireless/ipw2x00/ipw2100.c index f0551f807f6..d8d804e3a4b 100644 --- a/drivers/net/wireless/ipw2x00/ipw2100.c +++ b/drivers/net/wireless/ipw2x00/ipw2100.c | |||
@@ -8508,8 +8508,7 @@ static void ipw2100_release_firmware(struct ipw2100_priv *priv, | |||
8508 | struct ipw2100_fw *fw) | 8508 | struct ipw2100_fw *fw) |
8509 | { | 8509 | { |
8510 | fw->version = 0; | 8510 | fw->version = 0; |
8511 | if (fw->fw_entry) | 8511 | release_firmware(fw->fw_entry); |
8512 | release_firmware(fw->fw_entry); | ||
8513 | fw->fw_entry = NULL; | 8512 | fw->fw_entry = NULL; |
8514 | } | 8513 | } |
8515 | 8514 | ||