Messages in this thread | | | From | Kalle Valo <> | Subject | Re: [PATCH] staging: wfx: Remove redundant NULL check before release_firmware() call | Date | Wed, 08 Jun 2022 11:02:27 +0300 |
| |
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
| |