From 2e17eaed1bfb2e6362790b4a7590c48c6602a8bf Mon Sep 17 00:00:00 2001 From: Martin Roth Date: Sat, 4 Mar 2017 18:51:16 -0700 Subject: [PATCH] checkpatch.conf: Update rules MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit - 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 Reviewed-on: https://review.coreboot.org/18569 Tested-by: build bot (Jenkins) Reviewed-by: Lee Leahy Reviewed-by: Patrick Georgi Reviewed-by: Philippe Mathieu-Daudé --- .checkpatch.conf | 5 +++++ 1 file changed, 5 insertions(+) diff --git a/.checkpatch.conf b/.checkpatch.conf index f194cedc3d..ad04d39c3b 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 +