Karthik Ramasubramanian has uploaded this change for review.

View Change

util/lint: Provide some exemption to common parts of a vendor's MB

Adding common definitions for a vendor's mainboard(eg. CB:37705) requires
creating dummy board_info.txt and Kconfig.name files due to lint checks.
Update the lint scripts to provide exemption to common definitions for a
vendor's mainboard.

BUG=None
TEST=Ensure that no lint errors are observed when common definitions are
added for a vendor's mainboard.

Signed-off-by: Karthikeyan Ramasubramanian <kramasub@google.com>
Change-Id: I55c3474c17eacd008fee5f4089da33d4f7d6002a
---
M util/lint/lint-stable-005-board-status
M util/lint/lint-stable-006-board-name
2 files changed, 7 insertions(+), 0 deletions(-)

git pull ssh://review.coreboot.org:29418/coreboot refs/changes/07/37807/1
diff --git a/util/lint/lint-stable-005-board-status b/util/lint/lint-stable-005-board-status
index 952a9b5..e8313f2 100755
--- a/util/lint/lint-stable-005-board-status
+++ b/util/lint/lint-stable-005-board-status
@@ -17,6 +17,10 @@

LC_ALL=C export LC_ALL
for mobodir in $(git ls-files src/mainboard | sed -n 's,^\(src/mainboard/[^/]*/[^/]*\)/.*$,\1,p'|sort|uniq); do
+ if [[ ${mobodir} =~ ^src\/mainboard\/.*\/common$ ]]; then
+ continue
+ fi
+
board_info="$mobodir/board_info.txt"
if ! [ -f "$board_info" ]; then
echo "No $board_info found"
diff --git a/util/lint/lint-stable-006-board-name b/util/lint/lint-stable-006-board-name
index b2418a1..cf3b0e2 100755
--- a/util/lint/lint-stable-006-board-name
+++ b/util/lint/lint-stable-006-board-name
@@ -29,6 +29,9 @@
done

for i in src/mainboard/*/*/; do
+ if [[ ${i} =~ ^src\/mainboard\/.*\/common\/$ ]]; then
+ continue
+ fi
if [ -r $i/Kconfig ]; then
if [ ! -r $i/Kconfig.name ]; then
BOARD="$(grep -A2 MAINBOARD_PART_NUMBER $i/Kconfig | tail -1 | cut -f2 -d\")"

To view, visit change 37807. To unsubscribe, or for help writing mail filters, visit settings.

Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: I55c3474c17eacd008fee5f4089da33d4f7d6002a
Gerrit-Change-Number: 37807
Gerrit-PatchSet: 1
Gerrit-Owner: Karthik Ramasubramanian <kramasub@google.com>
Gerrit-MessageType: newchange