linux-kernel - Re: [PATCH] libertas: debugfs: Added missing spaces

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Date:   Mon, 24 Jul 2023 17:26:14 +0300
From:   Kalle Valo <kvalo@...nel.org>
To:     hanyu001@...suo.com
Cc:     libertas-dev@...ts.infradead.org, linux-wireless@...r.kernel.org,
        linux-kernel@...r.kernel.org
Subject: Re: [PATCH] libertas: debugfs: Added missing spaces

hanyu001@...suo.com writes:

> Fixes the following checkpatch errors:
>
> ./drivers/net/wireless/marvell/libertas/debugfs.c:712: ERROR: spaces
> required around that '=' (ctx:VxV)
> ./drivers/net/wireless/marvell/libertas/debugfs.c:712: ERROR: spaces
> required around that '<' (ctx:VxV)
> ./drivers/net/wireless/marvell/libertas/debugfs.c:723: ERROR: spaces
> required around that '=' (ctx:VxV)
> ./drivers/net/wireless/marvell/libertas/debugfs.c:723: ERROR: spaces
> required around that '<' (ctx:VxV)
> ./drivers/net/wireless/marvell/libertas/debugfs.c:734: ERROR: spaces
> required around that '=' (ctx:VxV)
> ./drivers/net/wireless/marvell/libertas/debugfs.c:734: ERROR: spaces
> required around that '<' (ctx:VxV)
> ./drivers/net/wireless/marvell/libertas/debugfs.c:754: ERROR: spaces
> required around that '=' (ctx:VxV)
> ./drivers/net/wireless/marvell/libertas/debugfs.c:754: ERROR: spaces
> required around that '<' (ctx:VxV)
> ./drivers/net/wireless/marvell/libertas/debugfs.c:754: ERROR: space
> required before the open parenthesis '('
> ./drivers/net/wireless/marvell/libertas/debugfs.c:759: ERROR: spaces
> required around that '=' (ctx:VxV)
> ./drivers/net/wireless/marvell/libertas/debugfs.c:759: ERROR: spaces
> required around that '<' (ctx:VxV)
> ./drivers/net/wireless/marvell/libertas/debugfs.c:759: ERROR: space
> required before the open parenthesis '('
> ./drivers/net/wireless/marvell/libertas/debugfs.c:766: ERROR: spaces
> required around that '=' (ctx:VxV)
> ./drivers/net/wireless/marvell/libertas/debugfs.c:766: ERROR: spaces
> required around that '<' (ctx:VxV)
> ./drivers/net/wireless/marvell/libertas/debugfs.c:766: ERROR: space
> required before the open parenthesis '('
>
> Signed-off-by: Yu Han <hanyu001@...suo.com>

Please don't submit checkpatch fixes to linux-wireless, they are too
much work with very little gain. And libertas is an orphaned driver, we
want to make the absolute minimal changes to it.

I'm now going to silently reject all checkpatch patches in patchwork.

-- 
https://patchwork.kernel.org/project/linux-wireless/list/

https://wireless.wiki.kernel.org/en/developers/documentation/submittingpatches