Martin Roth (martinroth@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@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@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 +