Stefan Ott has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/40620 )
Change subject: mb/lenovo/*: Add support for ThinkLight ......................................................................
Patch Set 3:
(3 comments)
https://review.coreboot.org/c/coreboot/+/40620/1//COMMIT_MSG Commit Message:
https://review.coreboot.org/c/coreboot/+/40620/1//COMMIT_MSG@11 PS1, Line 11: https://review.coreboot.org/c/coreboot/+/19644.
Please use the commit hash and summary.
Done
https://review.coreboot.org/c/coreboot/+/40620/1//COMMIT_MSG@16 PS1, Line 16: echo off >/proc/acpi/ibm/light
Please indent with four spaces (as in Markdown).
Done
https://review.coreboot.org/c/coreboot/+/40620/1//COMMIT_MSG@18 PS1, Line 18: I have tested it on an X200 and it seems to work fine.
With what Linux kernel?
Done