diff options
| author | Minghao Chi <chi.minghao@zte.com.cn> | 2022-06-06 01:42:37 +0000 |
|---|---|---|
| committer | Kalle Valo <kvalo@kernel.org> | 2022-06-10 10:38:55 +0300 |
| commit | 05a2eebfa650258b461d1a8aea768f2cb9a6b6ff (patch) | |
| tree | 67649d8ae222e000b0211d65819bf0f4b5eb5d45 | |
| parent | 85393ca610ded5875611d2d81c7cfd90397afe93 (diff) | |
wifi: wfx: Remove redundant NULL check before release_firmware() call
release_firmware() checks for NULL pointers internally so checking
before calling it is redundant.
Reported-by: Zeal Robot <zealci@zte.com.cn>
Signed-off-by: Minghao Chi <chi.minghao@zte.com.cn>
Acked-by: Jérôme Pouiller <jerome.pouiller@silabs.com>
Signed-off-by: Kalle Valo <kvalo@kernel.org>
Link: https://lore.kernel.org/r/20220606014237.290466-1-chi.minghao@zte.com.cn
| -rw-r--r-- | drivers/net/wireless/silabs/wfx/fwio.c | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/drivers/net/wireless/silabs/wfx/fwio.c b/drivers/net/wireless/silabs/wfx/fwio.c index 3d1b8a135dc0..52c7f560b062 100644 --- a/drivers/net/wireless/silabs/wfx/fwio.c +++ b/drivers/net/wireless/silabs/wfx/fwio.c @@ -286,8 +286,7 @@ static int load_firmware_secure(struct wfx_dev *wdev) error: kfree(buf); - if (fw) - release_firmware(fw); + release_firmware(fw); if (ret) print_boot_status(wdev); return ret; |