[coreboot-gerrit] Change in coreboot[master]: mainboard/system76: Add System76 Lemur Pro (lemp9)

[Matt DeVillier (Code Review)]

Matt DeVillier has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/38463 )

Change subject: mainboard/system76: Add System76 Lemur Pro (lemp9)
......................................................................

Patch Set 9:

(2 comments)

https://review.coreboot.org/c/coreboot/+/38463/6/src/mainboard/system76/lemp... 
File src/mainboard/system76/lemp9/acpi/ec.asl:

https://review.coreboot.org/c/coreboot/+/38463/6/src/mainboard/system76/lemp... 
PS6, Line 1: /*
> Seeing as this is the only mainboard so far in upstream coreboot, I would prefer to do the work of g […]

Ack

https://review.coreboot.org/c/coreboot/+/38463/6/src/mainboard/system76/lemp... 
File src/mainboard/system76/lemp9/devicetree.cb:

https://review.coreboot.org/c/coreboot/+/38463/6/src/mainboard/system76/lemp... 
PS6, Line 20: # CPU (soc/intel/cannonlake/cpu.c)
> These comments identify where the setting is used, which I have found helpful when things break and  […]

indeed, it was a pleasant surprise when reviewing