[PATCH] checkpatch: drop ENOSYS warning

[PATCH] checkpatch: drop ENOSYS warning

[Ahmad Fatoum]

We don't have system calls in barebox and we make ample use of ENOSYS,
so let's drop the checkpatch warning to avoid confusion.

Reported-by: Christian Thießen <christian.thiessen@airbus.com>
Signed-off-by: Ahmad Fatoum <a.fatoum@barebox.org>
---
 scripts/checkpatch.pl | 8 --------
 1 file changed, 8 deletions(-)

diff --git a/scripts/checkpatch.pl b/scripts/checkpatch.pl
index 9093841d8179..1592497fb38a 100755
--- a/scripts/checkpatch.pl
+++ b/scripts/checkpatch.pl
@@ -4102,14 +4102,6 @@ sub process {
 			     "Prefer dev_$level(... to dev_printk(KERN_$orig, ...\n" . $herecurr);
 		}
 
-# ENOSYS means "bad syscall nr" and nothing else.  This will have a small
-# number of false positives, but assembly files are not checked, so at
-# least the arch entry code will not trigger this warning.
-		if ($line =~ /\bENOSYS\b/) {
-			WARN("ENOSYS",
-			     "ENOSYS means 'invalid syscall nr' and nothing else\n" . $herecurr);
-		}
-
 # function brace can't be on same line, except for #defines of do while,
 # or if closed on same line
 		if ($perl_version_ok &&
-- 
2.47.3

Re: [PATCH] checkpatch: drop ENOSYS warning

[Sascha Hauer]

On Tue, 16 Sep 2025 09:19:35 +0200, Ahmad Fatoum wrote:
> We don't have system calls in barebox and we make ample use of ENOSYS,
> so let's drop the checkpatch warning to avoid confusion.
> 
> 

Applied, thanks!

[1/1] checkpatch: drop ENOSYS warning
      https://git.pengutronix.de/cgit/barebox/commit/?id=6fe17982f930 (link may not be stable)

Best regards,
-- 
Sascha Hauer <s.hauer@pengutronix.de>