diff options
| author | Joe Perches <joe@perches.com> | 2010-10-26 17:23:16 -0400 |
|---|---|---|
| committer | Linus Torvalds <torvalds@linux-foundation.org> | 2010-10-26 19:52:20 -0400 |
| commit | 03f1df7da5696ddfa6e167b37e0c0ce5aad3de79 (patch) | |
| tree | 46e1ac76d5a6a31f96f1ca040fb2620eb93f4a36 | |
| parent | 3bf9a009fccea422bc355414a3bdf5f35fff9f36 (diff) | |
checkpatch: Add additional attribute #defines
On Wed, 2010-08-11 at 12:35 -0400, Dave Jones wrote:
> I just got this from a patch I merged..
>
> ERROR: need consistent spacing around '*' (ctx:WxV)
> #121: FILE: arch/x86/kernel/cpu/cpufreq/pcc-cpufreq.c:113:
> +static struct pcc_cpu __percpu *pcc_cpu_info;
> ^
> which doesn't seem right.
Perhaps these need to be added to checkpatch.
[apw@canonical.com: added tests]
Signed-off-by: Joe Perches <joe@perches.com>
Signed-off-by: Andy Whitcroft <apw@canonical.com>
Cc: Dave Jones <davej@codemonkey.org.uk>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
| -rwxr-xr-x | scripts/checkpatch.pl | 14 |
1 files changed, 14 insertions, 0 deletions
diff --git a/scripts/checkpatch.pl b/scripts/checkpatch.pl index 93fa145671a0..c151c9e142ad 100755 --- a/scripts/checkpatch.pl +++ b/scripts/checkpatch.pl | |||
| @@ -152,6 +152,20 @@ our $Sparse = qr{ | |||
| 152 | # We need \b after 'init' otherwise 'initconst' will cause a false positive in a check | 152 | # We need \b after 'init' otherwise 'initconst' will cause a false positive in a check |
| 153 | our $Attribute = qr{ | 153 | our $Attribute = qr{ |
| 154 | const| | 154 | const| |
| 155 | __percpu| | ||
| 156 | __nocast| | ||
| 157 | __safe| | ||
| 158 | __bitwise__| | ||
| 159 | __packed__| | ||
| 160 | __packed2__| | ||
| 161 | __naked| | ||
| 162 | __maybe_unused| | ||
| 163 | __always_unused| | ||
| 164 | __noreturn| | ||
| 165 | __used| | ||
| 166 | __cold| | ||
| 167 | __noclone| | ||
| 168 | __deprecated| | ||
| 155 | __read_mostly| | 169 | __read_mostly| |
| 156 | __kprobes| | 170 | __kprobes| |
| 157 | __(?:mem|cpu|dev|)(?:initdata|initconst|init\b)| | 171 | __(?:mem|cpu|dev|)(?:initdata|initconst|init\b)| |
