diff options
| author | Wolfram Sang <w.sang@pengutronix.de> | 2010-03-05 16:43:51 -0500 |
|---|---|---|
| committer | Linus Torvalds <torvalds@linux-foundation.org> | 2010-03-06 14:26:44 -0500 |
| commit | 52131292c069b74f4b5f3c786ff66ff6e82b0e69 (patch) | |
| tree | b84324464503adc35bda0573c1c35f7497d9ecce /scripts | |
| parent | 691e669ba8c64d31ac08d87b1751e6acfa3ff65e (diff) | |
checkpatch: fix false positive on __initconst
checkpatch falsely complained about '__initconst' because it thought the
'const' needed a space before. Fix this by changing the list of
attributes:
- add '__initconst'
- force plain 'init' to contain a word-boundary at the end
Signed-off-by: Wolfram Sang <w.sang@pengutronix.de>
Cc: Andy Whitcroft <apw@shadowen.org>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
Diffstat (limited to 'scripts')
| -rwxr-xr-x | scripts/checkpatch.pl | 5 |
1 files changed, 4 insertions, 1 deletions
diff --git a/scripts/checkpatch.pl b/scripts/checkpatch.pl index 9a3894dbf752..1530dbf1dab5 100755 --- a/scripts/checkpatch.pl +++ b/scripts/checkpatch.pl | |||
| @@ -145,11 +145,14 @@ our $Sparse = qr{ | |||
| 145 | __kprobes| | 145 | __kprobes| |
| 146 | __ref | 146 | __ref |
| 147 | }x; | 147 | }x; |
| 148 | |||
| 149 | # Notes to $Attribute: | ||
| 150 | # We need \b after 'init' otherwise 'initconst' will cause a false positive in a check | ||
| 148 | our $Attribute = qr{ | 151 | our $Attribute = qr{ |
| 149 | const| | 152 | const| |
| 150 | __read_mostly| | 153 | __read_mostly| |
| 151 | __kprobes| | 154 | __kprobes| |
| 152 | __(?:mem|cpu|dev|)(?:initdata|init)| | 155 | __(?:mem|cpu|dev|)(?:initdata|initconst|init\b)| |
| 153 | ____cacheline_aligned| | 156 | ____cacheline_aligned| |
| 154 | ____cacheline_aligned_in_smp| | 157 | ____cacheline_aligned_in_smp| |
| 155 | ____cacheline_internodealigned_in_smp| | 158 | ____cacheline_internodealigned_in_smp| |
