aboutsummaryrefslogtreecommitdiffstats
path: root/scripts
diff options
context:
space:
mode:
authorJoe Perches <joe@perches.com>2011-07-25 20:13:24 -0400
committerLinus Torvalds <torvalds@linux-foundation.org>2011-07-25 23:57:16 -0400
commit39b7e2878e783af027ddd3530f7a0abec330905d (patch)
treef8b5adf97fc9806310c560e12463d1cb56c20b2f /scripts
parent2011247550c1b903a9ecd68f6eb3e9e7b7b07f52 (diff)
checkpatch: add a "prefer __aligned" check
Prefer the use of __aligned(size) over __attribute__((__aligned___(size))) Link: http://lkml.kernel.org/r/20110609094526.1571774c.akpm@linux-foundation.org Suggested-by: Andrew Morton <akpm@linux-foundation.org> Signed-off-by: Joe Perches <joe@perches.com> Signed-off-by: Andrew Morton <akpm@linux-foundation.org> Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
Diffstat (limited to 'scripts')
-rwxr-xr-xscripts/checkpatch.pl5
1 files changed, 5 insertions, 0 deletions
diff --git a/scripts/checkpatch.pl b/scripts/checkpatch.pl
index b14f830cb0fa..8f35f0e03518 100755
--- a/scripts/checkpatch.pl
+++ b/scripts/checkpatch.pl
@@ -2891,6 +2891,11 @@ sub process {
2891 WARN("__packed is preferred over __attribute__((packed))\n" . $herecurr); 2891 WARN("__packed is preferred over __attribute__((packed))\n" . $herecurr);
2892 } 2892 }
2893 2893
2894# Check for __attribute__ aligned, prefer __aligned
2895 if ($line =~ /\b__attribute__\s*\(\s*\(.*aligned/) {
2896 WARN("__aligned(size) is preferred over __attribute__((aligned(size)))\n" . $herecurr);
2897 }
2898
2894# check for sizeof(&) 2899# check for sizeof(&)
2895 if ($line =~ /\bsizeof\s*\(\s*\&/) { 2900 if ($line =~ /\bsizeof\s*\(\s*\&/) {
2896 WARN("sizeof(& should be avoided\n" . $herecurr); 2901 WARN("sizeof(& should be avoided\n" . $herecurr);