diff options
author | Joe Perches <joe@perches.com> | 2010-10-26 17:23:21 -0400 |
---|---|---|
committer | Linus Torvalds <torvalds@linux-foundation.org> | 2010-10-26 19:52:21 -0400 |
commit | 93ed0e2d07b25aff4db1d61bfbcd1e82074c0ad5 (patch) | |
tree | 7b0c7f31bbf7ca0f083c1372e2cf6ebc3eaac20a /scripts/checkpatch.pl | |
parent | cb710eca6820493add0ddd3d7e8e3ee53f2b6e57 (diff) |
scripts/checkpatch.pl: add check for declaration of pci_device_id
Signed-off-by: Joe Perches <joe@perches.com>
Cc: Andy Whitcroft <apw@canonical.com>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
Diffstat (limited to 'scripts/checkpatch.pl')
-rwxr-xr-x | scripts/checkpatch.pl | 5 |
1 files changed, 5 insertions, 0 deletions
diff --git a/scripts/checkpatch.pl b/scripts/checkpatch.pl index 2ec5fc6a4046..90b54d4697fd 100755 --- a/scripts/checkpatch.pl +++ b/scripts/checkpatch.pl | |||
@@ -1881,6 +1881,11 @@ sub process { | |||
1881 | $herecurr); | 1881 | $herecurr); |
1882 | } | 1882 | } |
1883 | 1883 | ||
1884 | # check for declarations of struct pci_device_id | ||
1885 | if ($line =~ /\bstruct\s+pci_device_id\s+\w+\s*\[\s*\]\s*\=\s*\{/) { | ||
1886 | WARN("Use DEFINE_PCI_DEVICE_TABLE for struct pci_device_id\n" . $herecurr); | ||
1887 | } | ||
1888 | |||
1884 | # check for new typedefs, only function parameters and sparse annotations | 1889 | # check for new typedefs, only function parameters and sparse annotations |
1885 | # make sense. | 1890 | # make sense. |
1886 | if ($line =~ /\btypedef\s/ && | 1891 | if ($line =~ /\btypedef\s/ && |