diff options
| author | Jesper Juhl <jj@chaosbits.net> | 2012-04-09 22:49:49 +0200 |
|---|---|---|
| committer | Gustavo Padovan <gustavo@padovan.org> | 2012-05-09 01:40:31 -0300 |
| commit | 59f34fb3354bb7b5f9d865ccaa2c54d3cf691cb8 (patch) | |
| tree | 5361b8023570521b23d908a8405189b471d0fb93 /drivers/bluetooth | |
| parent | b76bbd6657a2dd7545686ba9ad59625f44192146 (diff) | |
| download | op-kernel-dev-59f34fb3354bb7b5f9d865ccaa2c54d3cf691cb8.zip op-kernel-dev-59f34fb3354bb7b5f9d865ccaa2c54d3cf691cb8.tar.gz | |
Bluetooth: btmrvl_sdio: remove pointless conditional before release_firmware()
release_firmware() deals gracefullt with NULL pointers so there's no
reason to test for one prior to calling the function.
Signed-off-by: Jesper Juhl <jj@chaosbits.net>
Signed-off-by: Gustavo Padovan <gustavo@padovan.org>
Diffstat (limited to 'drivers/bluetooth')
| -rw-r--r-- | drivers/bluetooth/btmrvl_sdio.c | 9 |
1 files changed, 2 insertions, 7 deletions
diff --git a/drivers/bluetooth/btmrvl_sdio.c b/drivers/bluetooth/btmrvl_sdio.c index 27b74b0..ed62c7f 100644 --- a/drivers/bluetooth/btmrvl_sdio.c +++ b/drivers/bluetooth/btmrvl_sdio.c @@ -339,9 +339,7 @@ static int btmrvl_sdio_download_helper(struct btmrvl_sdio_card *card) done: kfree(tmphlprbuf); - if (fw_helper) - release_firmware(fw_helper); - + release_firmware(fw_helper); return ret; } @@ -484,10 +482,7 @@ static int btmrvl_sdio_download_fw_w_helper(struct btmrvl_sdio_card *card) done: kfree(tmpfwbuf); - - if (fw_firmware) - release_firmware(fw_firmware); - + release_firmware(fw_firmware); return ret; } |
