Nico Huber has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/45513 )
Change subject: lint: check for misuse of Kconfig SUBSYSTEM_*_ID ......................................................................
Patch Set 4:
(4 comments)
https://review.coreboot.org/c/coreboot/+/45513/4/util/lint/check_lint_tests File util/lint/check_lint_tests:
https://review.coreboot.org/c/coreboot/+/45513/4/util/lint/check_lint_tests@... PS4, Line 50: #lint-stable-024-kconfig-no-subsystem Why mention it here, is this file parsed somewhere? (beside the shell)
https://review.coreboot.org/c/coreboot/+/45513/4/util/lint/lint-stable-024-k... File util/lint/lint-stable-024-kconfig-no-subsystem:
https://review.coreboot.org/c/coreboot/+/45513/4/util/lint/lint-stable-024-k... PS4, Line 11: D End with `>` to make the match exact?
https://review.coreboot.org/c/coreboot/+/45513/4/util/lint/lint-stable-024-k... PS4, Line 11: ^ I'd make it `^\s*` (could be indented)
https://review.coreboot.org/c/coreboot/+/45513/4/util/lint/lint-stable-024-k... PS4, Line 12: %%/ Why was the / added in the first place?