Commit dc3f4dee authored by Stanislaw Gruszka's avatar Stanislaw Gruszka Committed by Andrew Morton
Browse files

scripts: checkpatch: allow "case" macros 

Do not report errors like below:

./scripts/checkpatch.pl -f drivers/net/wireless/ath/ath10k/wmi.h

ERROR: Macros with complex values should be enclosed in parentheses
+#define C2S(x) case x: return #x

since many "case ..." macros are already used by some in-kernel drivers.

Link: https://lkml.kernel.org/r/20221027134334.164301-1-stf_xl@wp.pl


Signed-off-by: default avatarStanislaw Gruszka <stf_xl@wp.pl>
Acked-by: default avatarJoe Perches <joe@perches.com>
Signed-off-by: default avatarAndrew Morton <akpm@linux-foundation.org>
parent 5cc81d5c

scripts/checkpatch.pl

+1 −0
Original line number Diff line number Diff line
@@ -5918,6 +5918,7 @@ sub process { 
			    $dstat !~ /$exceptions/ &&

			    $dstat !~ /^\.$Ident\s*=/ &&				# .foo =

			    $dstat !~ /^(?:\#\s*$Ident|\#\s*$Constant)\s*$/ &&		# stringification #foo

			    $dstat !~ /^case\b/ &&					# case ...

			    $dstat !~ /^do\s*$Constant\s*while\s*$Constant;?$/ &&	# do {...} while (...); // do {...} while (...)

			    $dstat !~ /^while\s*$Constant\s*$Constant\s*$/ &&		# while (...) {...}

			    $dstat !~ /^for\s*$Constant$/ &&				# for (...)