* [PATCH] staging: wfx: Remove redundant NULL check before release_firmware() call
@ 2022-06-06 1:42 cgel.zte
2022-06-06 6:36 ` Jérôme Pouiller
2022-06-10 7:39 ` wifi: " Kalle Valo
0 siblings, 2 replies; 4+ messages in thread
From: cgel.zte @ 2022-06-06 1:42 UTC (permalink / raw)
To: jerome.pouiller
Cc: kvalo, davem, edumazet, kuba, pabeni, linux-wireless, netdev,
linux-kernel, Minghao Chi, Zeal Robot
From: Minghao Chi <chi.minghao@zte.com.cn>
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>
---
drivers/net/wireless/silabs/wfx/fwio.c | 3 +--
1 file changed, 1 insertion(+), 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;
--
2.25.1
^ permalink raw reply related [flat|nested] 4+ messages in thread
* Re: [PATCH] staging: wfx: Remove redundant NULL check before release_firmware() call
2022-06-06 1:42 [PATCH] staging: wfx: Remove redundant NULL check before release_firmware() call cgel.zte
@ 2022-06-06 6:36 ` Jérôme Pouiller
2022-06-08 8:02 ` Kalle Valo
2022-06-10 7:39 ` wifi: " Kalle Valo
1 sibling, 1 reply; 4+ messages in thread
From: Jérôme Pouiller @ 2022-06-06 6:36 UTC (permalink / raw)
To: cgel.zte
Cc: kvalo, davem, edumazet, kuba, pabeni, linux-wireless, netdev,
linux-kernel, Minghao Chi, Zeal Robot
On Monday 6 June 2022 03:42:37 CEST cgel.zte@gmail.com wrote:
> From: Minghao Chi <chi.minghao@zte.com.cn>
>
> 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>
> ---
> drivers/net/wireless/silabs/wfx/fwio.c | 3 +--
> 1 file changed, 1 insertion(+), 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;
> --
> 2.25.1
Signed-off-by: Jérôme Pouiller <jerome.pouiller@silabs.com>
--
Jérôme Pouiller
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH] staging: wfx: Remove redundant NULL check before release_firmware() call
2022-06-06 6:36 ` Jérôme Pouiller
@ 2022-06-08 8:02 ` Kalle Valo
0 siblings, 0 replies; 4+ messages in thread
From: Kalle Valo @ 2022-06-08 8:02 UTC (permalink / raw)
To: Jérôme Pouiller
Cc: cgel.zte, davem, edumazet, kuba, pabeni, linux-wireless, netdev,
linux-kernel, Minghao Chi, Zeal Robot
Jérôme Pouiller <jerome.pouiller@silabs.com> writes:
> On Monday 6 June 2022 03:42:37 CEST cgel.zte@gmail.com wrote:
>> From: Minghao Chi <chi.minghao@zte.com.cn>
>>
>> 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>
[...]
> Signed-off-by: Jérôme Pouiller <jerome.pouiller@silabs.com>
I'll change this to Acked-by, s-o-b should be used only when you are
part of patch distribution:
https://www.kernel.org/doc/html/latest/process/submitting-patches.html#when-to-use-acked-by-cc-and-co-developed-by
And please edit your quotes, otherwise using patchwork will be painful.
--
https://patchwork.kernel.org/project/linux-wireless/list/
https://wireless.wiki.kernel.org/en/developers/documentation/submittingpatches
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: wifi: wfx: Remove redundant NULL check before release_firmware() call
2022-06-06 1:42 [PATCH] staging: wfx: Remove redundant NULL check before release_firmware() call cgel.zte
2022-06-06 6:36 ` Jérôme Pouiller
@ 2022-06-10 7:39 ` Kalle Valo
1 sibling, 0 replies; 4+ messages in thread
From: Kalle Valo @ 2022-06-10 7:39 UTC (permalink / raw)
To: cgel.zte
Cc: jerome.pouiller, davem, edumazet, kuba, pabeni, linux-wireless,
netdev, linux-kernel, Minghao Chi, Zeal Robot
cgel.zte@gmail.com wrote:
> From: Minghao Chi <chi.minghao@zte.com.cn>
>
> 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>
Patch applied to wireless-next.git, thanks.
05a2eebfa650 wifi: wfx: Remove redundant NULL check before release_firmware() call
--
https://patchwork.kernel.org/project/linux-wireless/patch/20220606014237.290466-1-chi.minghao@zte.com.cn/
https://wireless.wiki.kernel.org/en/developers/documentation/submittingpatches
^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2022-06-10 7:39 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2022-06-06 1:42 [PATCH] staging: wfx: Remove redundant NULL check before release_firmware() call cgel.zte
2022-06-06 6:36 ` Jérôme Pouiller
2022-06-08 8:02 ` Kalle Valo
2022-06-10 7:39 ` wifi: " Kalle Valo
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).