From 243149ee82aea8127566969aa7a0d33ff849b405 Mon Sep 17 00:00:00 2001 From: Michael Ellerman Date: Wed, 23 Jul 2008 21:28:57 -0700 Subject: [PATCH] --- yaml --- r: 105382 b: refs/heads/master c: f3db6639fee577f6ed92c0a1fc881e748c47ec48 h: refs/heads/master v: v3 --- [refs] | 2 +- trunk/scripts/checkpatch.pl | 4 ++++ 2 files changed, 5 insertions(+), 1 deletion(-) diff --git a/[refs] b/[refs] index 7e664919809d..a6f7da18d0f8 100644 --- a/[refs] +++ b/[refs] @@ -1,2 +1,2 @@ --- -refs/heads/master: c8cb2ca37ed51aa1f3b20e3eff1e72df1c400f70 +refs/heads/master: f3db6639fee577f6ed92c0a1fc881e748c47ec48 diff --git a/trunk/scripts/checkpatch.pl b/trunk/scripts/checkpatch.pl index 5420db6502fc..cf70f123f574 100755 --- a/trunk/scripts/checkpatch.pl +++ b/trunk/scripts/checkpatch.pl @@ -2108,6 +2108,10 @@ sub process { if ($line =~ /\bsimple_(strto.*?)\s*\(/) { WARN("consider using strict_$1 in preference to simple_$1\n" . $herecurr); } +# check for __initcall(), use device_initcall() explicitly please + if ($line =~ /^.\s*__initcall\s*\(/) { + WARN("please use device_initcall() instead of __initcall()\n" . $herecurr); + } # use of NR_CPUS is usually wrong # ignore definitions of NR_CPUS and usage to define arrays as likely right