summaryrefslogtreecommitdiff
path: root/.checkpatch.conf
diff options
context:
space:
mode:
authorStefan Reinauer <reinauer@chromium.org>2016-08-01 12:21:30 -0700
committerMartin Roth <martinroth@google.com>2016-09-02 18:22:04 +0200
commit36d405268f040208cd26902f3c0b5346f7d4d25b (patch)
tree4093797a6716c4ce180b6e367d4ab0591aee7212 /.checkpatch.conf
parent9391aac2e091345f0b9d45e0f4cddd1f2ecb0c30 (diff)
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> Reviewed-on: https://review.coreboot.org/16010 Tested-by: build bot (Jenkins) Reviewed-by: Alexander Couzens <lynxis@fe80.eu>
Diffstat (limited to '.checkpatch.conf')
-rw-r--r--.checkpatch.conf15
1 files changed, 12 insertions, 3 deletions
diff --git a/.checkpatch.conf b/.checkpatch.conf
index f7b9b32519..f194cedc3d 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