Martin Roth (martinroth@google.com) just uploaded a new patch set to gerrit, which you can find at https://review.coreboot.org/16010
-gerrit
commit c9aa1efe8ca1f389ed307f66adad889773867304 Author: Stefan Reinauer reinauer@chromium.org Date: Mon Aug 1 12:21:30 2016 -0700
Update .checkpatch.conf
- Don't require a description of every config statement in the Kconfig changes - Don't complain about externs in .c files - Don't complain about the use of the volatile keyword. The kernel may not want it, but we definitely need it. - Disable checks that seem to be broken.
Change-Id: Ic419b81cd36852a91e887e610d4a04984ab5fbd7 Signed-off-by: Stefan Reinauer stefan.reinauer@coreboot.org Signed-off-by: Martin Roth martinroth@google.com --- .checkpatch.conf | 15 ++++++++++++--- 1 file changed, 12 insertions(+), 3 deletions(-)
diff --git a/.checkpatch.conf b/.checkpatch.conf index f7b9b32..f194ced 100644 --- a/.checkpatch.conf +++ b/.checkpatch.conf @@ -1,9 +1,6 @@ # Not Linux, so don't expect a Linux tree. --no-tree
-# Require every Kconfig symbol to have at least two lines in their description. ---min-conf-desc-length=2 - # Ignore aspects we don't follow here. --ignore C99_COMMENTS --ignore GLOBAL_INITIALISERS @@ -15,3 +12,15 @@ --ignore PREFER_PRINTF --ignore SPLIT_STRING --ignore BLOCK_COMMENT_STYLE +--ignore AVOID_EXTERNS +--ignore VOLATILE +--ignore CONFIG_DESCRIPTION + +# FILE_PATH_CHANGES seems to not be working correctly. It will +# choke on added / deleted files even if the MAINTAINERS file +# is touched. +--ignore FILE_PATH_CHANGES + +# This one has a linux path hard coded, so it would choke on +# some commits unnecessarily. +--ignore EXECUTE_PERMISSIONS