Paul Menzel has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/40581 )
Change subject: mainboard/clevo/n141cu: Add new mainboard ......................................................................
Patch Set 27:
(5 comments)
Nice.
https://review.coreboot.org/c/coreboot/+/40581/27//COMMIT_MSG Commit Message:
https://review.coreboot.org/c/coreboot/+/40581/27//COMMIT_MSG@7 PS27, Line 7: mainboard/clevo/n141cu: Add new mainboard Add new Comet Lake mainboard
https://review.coreboot.org/c/coreboot/+/40581/27//COMMIT_MSG@7 PS27, Line 7: mainboard mb
https://review.coreboot.org/c/coreboot/+/40581/27//COMMIT_MSG@24 PS27, Line 24: Mention that you base this on a System76 board?
https://review.coreboot.org/c/coreboot/+/40581/27//COMMIT_MSG@26 PS27, Line 26: - All of the hardware above What model/brand?
https://review.coreboot.org/c/coreboot/+/40581/27/src/mainboard/clevo/cml-u/... File src/mainboard/clevo/cml-u/acpi/ac.asl:
https://review.coreboot.org/c/coreboot/+/40581/27/src/mainboard/clevo/cml-u/... PS27, Line 15: * GNU General Public License for more details. Use SPDX headers?