Change in coreboot[master]: lint: checkpatch: Only exclude specific src/vendorcode/ subdirectories

[Patrick Georgi (Code Review)]

Patrick Georgi has submitted this change. ( https://review.coreboot.org/c/coreboot/+/51827 )

Change subject: lint: checkpatch: Only exclude specific src/vendorcode/ subdirectories
......................................................................

lint: checkpatch: Only exclude specific src/vendorcode/ subdirectories

Some of the src/vendorcode/ directories are used to import a whole
codebase from somewhere else which uses a completely different coding
style. For those directories, excluding them from checkpatch makes
sense. However, other directories are simply implementing
vendor-specific extensions that were written by coreboot developers
specifically for coreboot in coreboot's coding style. Those directories
should be covered by checkpatch.

This patch narrows the existing blanket exception of src/vendorcode/ to
the amd, cavium, intel and mediatek directories (which actually include
large amounts of foreign source). The eltan, google and siemens
directories (which seem to contain code specifically written for
coreboot) will now be covered by checkpatch.

Signed-off-by: Julius Werner 
Change-Id: I1feaba37c469714217fff4d160e595849e0230b9
Reviewed-on: https://review.coreboot.org/c/coreboot/+/51827
Reviewed-by: Angel Pons 
Reviewed-by: Werner Zeh 
Reviewed-by: David Hendricks 
Tested-by: build bot (Jenkins) 
---
M .checkpatch.conf
M util/lint/lint-007-checkpatch
2 files changed, 5 insertions(+), 3 deletions(-)

Approvals:
  build bot (Jenkins): Verified
  David Hendricks: Looks good to me, approved
  Werner Zeh: Looks good to me, approved
  Angel Pons: Looks good to me, but someone else must approve

diff --git a/.checkpatch.conf b/.checkpatch.conf
index 1bf0a32..95019d1 100644
--- a/.checkpatch.conf
+++ b/.checkpatch.conf
@@ -32,5 +32,8 @@
 # some commits unnecessarily.
 --ignore EXECUTE_PERMISSIONS

-# Exclude the vendorcode directory
---exclude src/vendorcode
+# Exclude vendorcode directories that don't follow coreboot's coding style.
+--exclude src/vendorcode/amd
+--exclude src/vendorcode/cavium
+--exclude src/vendorcode/intel
+--exclude src/vendorcode/mediatek
diff --git a/util/lint/lint-007-checkpatch b/util/lint/lint-007-checkpatch
index 113d0ae..7a76878 100755
--- a/util/lint/lint-007-checkpatch
+++ b/util/lint/lint-007-checkpatch
@@ -15,7 +15,6 @@
 ^util/inteltool|\
 ^util/kconfig|\
 ^util/superiotool|\
-^src/vendorcode|\
 ^Documentation"

opts="--max-line-length 96"