From 58dde171d5c0591569b79cbe267371391a8afc1b Mon Sep 17 00:00:00 2001 From: Martin Roth Date: Mon, 29 Aug 2016 15:32:10 -0600 Subject: [PATCH] UPSTREAM: checkpatch.pl: ignore '#define asmlinkage' checkpatch warns that the asmlinkage storage class should be at the beginning of the declaration when we define it to be an empty value. BUG=None BRANCH=None TEST=None Signed-off-by: Martin Roth Reviewed-on: https://review.coreboot.org/16358 Tested-by: build bot (Jenkins) Reviewed-by: Patrick Georgi Change-Id: I12292d5b42bf6da9130bb969ebe00fca8efcf049 Reviewed-on: https://chromium-review.googlesource.com/388295 Commit-Ready: Furquan Shaikh Tested-by: Furquan Shaikh Reviewed-by: Aaron Durbin --- util/lint/checkpatch.pl | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/util/lint/checkpatch.pl b/util/lint/checkpatch.pl index 646cb0f328..84574f1f90 100755 --- a/util/lint/checkpatch.pl +++ b/util/lint/checkpatch.pl @@ -5340,7 +5340,8 @@ sub process { } # Check that the storage class is at the beginning of a declaration - if ($line =~ /\b$Storage\b/ && $line !~ /^.\s*$Storage\b/) { +# coreboot: skip complaint about our '#define asmlinkage' lines + if ($line =~ /\b$Storage\b/ && $line !~ /^.\s*$Storage\b/ && $line !~ /^.\s*#\s*define\s+$Storage\b/) { WARN("STORAGE_CLASS", "storage class should be at the beginning of the declaration\n" . $herecurr) }