The patch titled scripts/checkpatch.pl: add check for declaration of pci_device_id has been added to the -mm tree. Its filename is scripts-checkpatchpl-add-check-for-declaration-of-pci_device_id.patch Before you just go and hit "reply", please: a) Consider who else should be cc'ed b) Prefer to cc a suitable mailing list as well c) Ideally: find the original patch on the mailing list and do a reply-to-all to that, adding suitable additional cc's *** Remember to use Documentation/SubmitChecklist when testing your code *** See http://userweb.kernel.org/~akpm/stuff/added-to-mm.txt to find out what to do about this The current -mm tree may be found at http://userweb.kernel.org/~akpm/mmotm/ ------------------------------------------------------ Subject: scripts/checkpatch.pl: add check for declaration of pci_device_id From: Joe Perches <joe@xxxxxxxxxxx> Return-Path: <joe@xxxxxxxxxxx> X-Spam-Checker-Version: SpamAssassin 3.3.1 (2010-03-16) on z X-Spam-Level: X-Spam-Status: No, score=-101.5 required=7.0 tests=BAYES_00,USER_IN_WHITELIST autolearn=ham version=3.3.1 Received: from localhost (localhost [127.0.0.1]) by localhost.localdomain (8.14.3/8.14.3) with ESMTP id o9M6W29c025321 for <akpm@localhost>; Thu, 21 Oct 2010 23:32:02 -0700 Received: from imap1.linux-foundation.org [140.211.169.55] by localhost with IMAP (fetchmail-6.3.11) for <akpm@localhost> (single-drop); Thu, 21 Oct 2010 23:32:02 -0700 (PDT) Received: from smtp1.linux-foundation.org (smtp1.linux-foundation.org [140.211.169.13]) by imap1.linux-foundation.org (8.13.5.20060308/8.13.5/Debian-3ubuntu1.1) with ESMTP id o9M6Uncx031082 for <akpm@xxxxxxxxxxxxxxxxxxxxxxxxxx>; Thu, 21 Oct 2010 23:30:49 -0700 Received: from mail.perches.com (mail.perches.com [173.55.12.10]) by smtp1.linux-foundation.org (8.14.2/8.13.5/Debian-3ubuntu1.1) with ESMTP id o9M6UFSK007872 for <akpm@xxxxxxxxxxxxxxxxxxxx>; Thu, 21 Oct 2010 23:30:16 -0700 Received: from [192.168.1.162] (unknown [192.168.1.162]) by mail.perches.com (Postfix) with ESMTP id 900DB24368; Thu, 21 Oct 2010 23:29:58 -0700 (PDT) Subject: [PATCH] scripts/checkpatch.pl: Add check for declaration of pci_device_id To: Andy Whitcroft <apw@xxxxxxxxxxxxx> Cc: Andrew Morton <akpm@xxxxxxxxxxxxxxxxxxxx>, LKML <linux-kernel@xxxxxxxxxxxxxxx> Content-Type: text/plain; charset="UTF-8" Date: Thu, 21 Oct 2010 23:30:09 -0700 Message-ID: <1287729009.28393.4.camel@Joe-Laptop> Mime-Version: 1.0 X-Mailer: Evolution 2.30.3 Content-Transfer-Encoding: 7bit Received-SPF: none (domain of joe@xxxxxxxxxxx does not designate permitted sender hosts) X-MIMEDefang-Filter: lf$Revision: 1.188 $ X-Scanned-By: MIMEDefang 2.63 on 140.211.169.13 Signed-off-by: Joe Perches <joe@xxxxxxxxxxx> --- scripts/checkpatch.pl | 5 +++++ 1 files changed, 5 insertions(+), 0 deletions(-) diff --git a/scripts/checkpatch.pl b/scripts/checkpatch.pl index 2039acd..9929cec 100755 --- a/scripts/checkpatch.pl +++ b/scripts/checkpatch.pl @@ -1806,6 +1806,11 @@ sub process { $herecurr); } +# check for declarations of struct pci_device_id + if ($line =~ /\bstruct\s+pci_device_id\s+\w+\s*\[\s*\]\s*\=\s*\{/) { + WARN("Use DEFINE_PCI_DEVICE_TABLE for struct pci_device_id\n" . $herecurr); + } + # check for new typedefs, only function parameters and sparse annotations # make sense. if ($line =~ /\btypedef\s/ && Patches currently in -mm which might be from joe@xxxxxxxxxxx are linux-next.patch kernelh-add-minmax3-macros.patch kernelh-add-minmax3-macros-fix.patch replace-nested-max-min-macros-with-maxmin3-macro.patch vsprintfc-use-default-pointer-field-size-for-null-strings.patch vsprintfc-use-default-pointer-field-size-for-null-strings-fix.patch lib-fix-scnprintf-if-size-is-==-0.patch scripts-get_maintainerpl-add-git-blame-rolestats-authored-lines-information.patch scripts-get_maintainerpl-use-correct-indentation.patch scripts-get_maintainerpl-dont-search-maintainers-for-keywords-or-emails.patch scripts-get_maintainerpl-add-default-git-fallback-remove-default-git.patch scripts-get_maintainerpl-use-get_maintainerconf-from-then-home-then-scripts.patch scripts-get_maintainerpl-add-interactive-mode.patch scripts-get_maintainerpl-improve-interactive-ui.patch scripts-get_maintainerpl-update-interactive-ui-improve-hg-runtime.patch scripts-get_maintainerpl-use-case-insensitive-name-de-duplication.patch scripts-get_maintainerpl-fix-mailmap-handling.patch scripts-get_maintainerpl-correct-indentation-in-a-few-places.patch scripts-get_maintainerpl-use-mailmap-in-name-deduplication-and-other-updates.patch scripts-get_maintainerpl-dont-deduplicate-unnamed-addresses-ie-mailing-lists.patch maintainers-fix-colibri-pxa270-file-pattern.patch maintainers-merge-imote2-and-stargate.patch maintainers-merge-s3c-244x-sections.patch maintainers-merge-s3c6400-and-6410-to-64xx.patch maintainers-remove-usb-ov511-driver.patch maintainers-remove-usb-zc0301-driver.patch maintainers-use-t-git-and-whitespace-trivia.patch checkpatch-add-check-for-space-after-struct-union-and-enum.patch checkpatch-add-additional-attribute-defines.patch scripts-checkpatchpl-add-warnings-for-static-char-that-could-be-static-const-char.patch scripts-checkpatchpl-add-check-for-declaration-of-pci_device_id.patch gpio-add-topcliff-pch-gpio-driver.patch gpio-add-topcliff-pch-gpio-driver-fix.patch -- To unsubscribe from this list: send the line "unsubscribe mm-commits" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html