Attention is currently required from: Martin Roth, Julius Werner, Werner Zeh. Hello build bot (Jenkins), Patrick Georgi, Martin Roth, Frans Hendriks, Paul Menzel, Angel Pons, Werner Zeh,
I'd like you to reexamine a change. Please visit
https://review.coreboot.org/c/coreboot/+/51827
to look at the new patch set (#2).
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 jwerner@chromium.org Change-Id: I1feaba37c469714217fff4d160e595849e0230b9 --- M .checkpatch.conf M util/lint/lint-007-checkpatch 2 files changed, 5 insertions(+), 3 deletions(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/27/51827/2