Michael Niewöhner 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 6:
(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
Hm, I did this because #lint-stable-008-kconfig is there, too, but now realized nr. […]
yup, works without. see CB:45513
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?
Done
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)
Done
https://review.coreboot.org/c/coreboot/+/45513/4/util/lint/lint-stable-024-k... PS4, Line 12: %%/
uhm oh, does "it was late" count as as answer? ;)
Done