summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorMartin Roth <martinroth@google.com>2016-08-29 15:40:57 -0600
committerMartin Roth <martinroth@google.com>2016-09-20 21:43:11 +0200
commit1ce2ba6832ad13ea95b5e1184dc696d0a13ce036 (patch)
treed4f18130e5d41ead983aee807236ddcbcf30986b
parent374c39e3cfbb51927860756d32a77d0afd3752a6 (diff)
downloadcoreboot-1ce2ba6832ad13ea95b5e1184dc696d0a13ce036.tar.xz
lint/lint-007-checkpatch: Update lint script
- Check Kconfig files as well. - Accept a list of directories to check as a command line argument. - Only look at src & util directories by default. - Skip src/vendorcode. - Remove bypass of payloads/coreinfo/util/kconfig directory, it no longer exists. Change-Id: Ia522d3ddc29914220bdaae36ea23ded7338c48fd Signed-off-by: Martin Roth <martinroth@google.com> Reviewed-on: https://review.coreboot.org/16359 Tested-by: build bot (Jenkins) Reviewed-by: Paul Menzel <paulepanter@users.sourceforge.net> Reviewed-by: Patrick Georgi <pgeorgi@google.com>
-rwxr-xr-xutil/lint/lint-007-checkpatch33
1 files changed, 26 insertions, 7 deletions
diff --git a/util/lint/lint-007-checkpatch b/util/lint/lint-007-checkpatch
index 9d9d96ebc6..b4033e09c5 100755
--- a/util/lint/lint-007-checkpatch
+++ b/util/lint/lint-007-checkpatch
@@ -12,12 +12,31 @@
# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
# GNU General Public License for more details.
#
-# DESCR: Checkpatch on all .c and .h files in the tree
+# DESCR: Checkpatch on .c, .h, & Kconfig files in the tree
LC_ALL=C export LC_ALL
-util/lint/checkpatch.pl --show-types --file $( git ls-files \*.[ch] | \
- grep -v ^payloads/libpayload/util/kconfig | \
- grep -v ^payloads/libpayload/curses/PDCurses-3.4 | \
- grep -v ^payloads/coreinfo/util/kconfig | \
- grep -v ^util/kconfig \
- )
+
+# GNU BRE syntax list of files to examine
+INCLUDED_FILES='.*\.[ch]\|Kconfig.*$'
+
+EXCLUDED_DIRS="^payloads/libpayload/util/kconfig\|\
+^payloads/libpayload/curses/PDCurses\|\
+^util/kconfig\|\
+^src/vendorcode"
+
+# Space separated list of directories to test
+if [ "$1" = "" ]; then
+ INCLUDED_DIRS="src util"
+else
+ INCLUDED_DIRS="$1"
+fi
+
+# We want word splitting here, so disable the shellcheck warnings
+# shellcheck disable=SC2046,SC2086
+FILELIST=$( git ls-files $INCLUDED_DIRS | \
+ grep $INCLUDED_FILES | \
+ grep -v $EXCLUDED_DIRS )
+
+for FILE in $FILELIST; do
+ util/lint/checkpatch.pl --show-types --file --quiet "$FILE"
+done