514a4bcb23
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 Reviewed-on: https://review.coreboot.org/c/coreboot/+/51827 Reviewed-by: Angel Pons <th3fanbus@gmail.com> Reviewed-by: Werner Zeh <werner.zeh@siemens.com> Reviewed-by: David Hendricks <david.hendricks@gmail.com> Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
39 lines
1.1 KiB
Text
39 lines
1.1 KiB
Text
# Not Linux, so don't expect a Linux tree.
|
|
--no-tree
|
|
|
|
# Ignore aspects we don't follow here.
|
|
--ignore C99_COMMENTS
|
|
--ignore GLOBAL_INITIALISERS
|
|
--ignore INITIALISED_STATIC
|
|
--ignore LINE_SPACING
|
|
--ignore NEW_TYPEDEFS
|
|
--ignore PREFER_ALIGNED
|
|
--ignore PREFER_PACKED
|
|
--ignore PREFER_PRINTF
|
|
--ignore SPLIT_STRING
|
|
--ignore BLOCK_COMMENT_STYLE
|
|
--ignore AVOID_EXTERNS
|
|
--ignore VOLATILE
|
|
--ignore CONFIG_DESCRIPTION
|
|
--ignore MISSING_SPACE
|
|
--ignore CORRUPTED_PATCH
|
|
--ignore SPDX_LICENSE_TAG
|
|
--ignore UNDOCUMENTED_DT_STRING
|
|
--ignore PRINTK_WITHOUT_KERN_LEVEL
|
|
--ignore ASSIGN_IN_IF
|
|
--ignore UNNECESSARY_ELSE
|
|
|
|
# FILE_PATH_CHANGES seems to not be working correctly. It will
|
|
# choke on added / deleted files even if the MAINTAINERS file
|
|
# is touched.
|
|
--ignore FILE_PATH_CHANGES
|
|
|
|
# This one has a linux path hard coded, so it would choke on
|
|
# some commits unnecessarily.
|
|
--ignore EXECUTE_PERMISSIONS
|
|
|
|
# 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
|