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

From: Kalle Valo <[email-masked]>
Date: Wed, 08 Jun 2022 11:02:27 +0300

Jérôme Pouiller <[email-masked]> writes:

> On Monday 6 June 2022 03:42:37 CEST [email-masked] wrote:
>> From: Minghao Chi <[email-masked]>
>> 
>> release_firmware() checks for NULL pointers internally so checking
>> before calling it is redundant.
>> 
>> Reported-by: Zeal Robot <[email-masked]>
>> Signed-off-by: Minghao Chi <[email-masked]>

[...]

> Signed-off-by: Jérôme Pouiller <[email-masked]>

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