[coreboot-gerrit] Patch set updated for coreboot: checkpatch.conf: Update rules

Martin Roth (martinroth@google.com) gerrit at coreboot.org
Tue Mar 7 17:49:20 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 e076481748e48d8b526079d24221709e60106c9e
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