[coreboot-gerrit] New patch to review for coreboot: checkpatch.conf: Update rules
Martin Roth (martinroth@google.com)
gerrit at coreboot.org
Sun Mar 5 03:34:33 CET 2017
Martin Roth (martinroth at google.com) just uploaded a new patch set to gerrit, which you can find at https://review.coreboot.org/18569
-gerrit
commit cd51cb1e5f9300a8da3e1955f8af39327c0eb8ef
Author: Martin Roth <martinroth at google.com>
Date: Sat Mar 4 18:51:16 2017 -0700
checkpatch.conf: Update rules
- Remove the "MISSING_SPACE" check which checks for breaks at a space in
a string concatenation. Most of the time this makes sense, but we
occasionally need to break where there isn't a space, so having a hard
rule doesn't always work.
- Don't check the vendorcode directory for compliance to coreboot's
code format rules.
Change-Id: Ic07677b19520b5d22363834c77f5dee7bba9e429
Signed-off-by: Martin Roth <martinroth at google.com>
---
.checkpatch.conf | 5 +++++
1 file changed, 5 insertions(+)
diff --git a/.checkpatch.conf b/.checkpatch.conf
index f194ced..ad04d39 100644
--- a/.checkpatch.conf
+++ b/.checkpatch.conf
@@ -15,6 +15,7 @@
--ignore AVOID_EXTERNS
--ignore VOLATILE
--ignore CONFIG_DESCRIPTION
+--ignore MISSING_SPACE
# FILE_PATH_CHANGES seems to not be working correctly. It will
# choke on added / deleted files even if the MAINTAINERS file
@@ -24,3 +25,7 @@
# This one has a linux path hard coded, so it would choke on
# some commits unnecessarily.
--ignore EXECUTE_PERMISSIONS
+
+# Exclude the vendorcode directory
+--exclude src/vendorcode
+
More information about the coreboot-gerrit
mailing list