Thread (1 message) 1 message, 1 author, 2022-06-08

Re: [PATCH] staging: wfx: Remove redundant NULL check before release_firmware() call

From: Kalle Valo <kvalo@kernel.org>
Date: 2022-06-08 08:52:03
Also in: linux-wireless, lkml

Jérôme Pouiller [off-list ref] writes:
On Monday 6 June 2022 03:42:37 CEST cgel.zte@gmail.com wrote:
quoted
From: Minghao Chi <redacted>

release_firmware() checks for NULL pointers internally so checking
before calling it is redundant.

Reported-by: Zeal Robot <redacted>
Signed-off-by: Minghao Chi <redacted>
[...]
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
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help