From 89a3dc4f7d29ae461e714050226e37de1b4cc0e3 Mon Sep 17 00:00:00 2001 From: Martin Roth Date: Sat, 4 Mar 2017 18:51:16 -0700 Subject: [PATCH] UPSTREAM: 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. BUG=none BRANCH=none TEST=none Change-Id: If6648109dac8e91066f03e8d3c733db2f239b978 Signed-off-by: Patrick Georgi Original-Commit-Id: 2e17eaed1bfb2e6362790b4a7590c48c6602a8bf Original-Change-Id: Ic07677b19520b5d22363834c77f5dee7bba9e429 Original-Signed-off-by: Martin Roth Original-Reviewed-on: https://review.coreboot.org/18569 Original-Tested-by: build bot (Jenkins) Original-Reviewed-by: Lee Leahy Original-Reviewed-by: Patrick Georgi Original-Reviewed-by: Philippe Mathieu-Daud Reviewed-on: https://chromium-review.googlesource.com/452365 --- .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 +