summaryrefslogtreecommitdiff
path: root/util/lint
diff options
context:
space:
mode:
authorMartin Roth <martinroth@google.com>2016-08-29 15:32:10 -0600
committerMartin Roth <martinroth@google.com>2016-09-20 21:44:11 +0200
commit57bfbb0508e81c338696539f051ce04f4ba5393d (patch)
tree79ec93dd26cbf8cbb866fc290d9d19b8e2a33710 /util/lint
parent1ce2ba6832ad13ea95b5e1184dc696d0a13ce036 (diff)
downloadcoreboot-57bfbb0508e81c338696539f051ce04f4ba5393d.tar.xz
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. Change-Id: I12292d5b42bf6da9130bb969ebe00fca8efcf049 Signed-off-by: Martin Roth <martinroth@google.com> Reviewed-on: https://review.coreboot.org/16358 Tested-by: build bot (Jenkins) Reviewed-by: Patrick Georgi <pgeorgi@google.com>
Diffstat (limited to 'util/lint')
-rwxr-xr-xutil/lint/checkpatch.pl3
1 files changed, 2 insertions, 1 deletions
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)
}