[coreboot-gerrit] Change in coreboot[master]: mb/asus/p8h61-m_lx3_r2_0: Add new mainboard

[Angel Pons (Code Review)]

Angel Pons has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/39099 )

Change subject: mb/asus/p8h61-m_lx3_r2_0: Add new mainboard
......................................................................

Patch Set 4:

(2 comments)

https://review.coreboot.org/c/coreboot/+/39099/4/src/mainboard/asus/p8h61-m_... 
File src/mainboard/asus/p8h61-m_lx3_r2_0/Kconfig:

https://review.coreboot.org/c/coreboot/+/39099/4/src/mainboard/asus/p8h61-m_... 
PS4, Line 20: select USE_NATIVE_RAMINIT
> do we want to force this as non-deselectable?

It won't build though, non-native raminit needs an additional function to fill the pei_data struct.

https://review.coreboot.org/c/coreboot/+/39099/4/src/mainboard/asus/p8h61-m_... 
File src/mainboard/asus/p8h61-m_lx3_r2_0/dsdt.asl:

https://review.coreboot.org/c/coreboot/+/39099/4/src/mainboard/asus/p8h61-m_... 
PS4, Line 24: #include <drivers/intel/gma/acpi/default_brightness_levels.asl>
> not needed for a desktop board w/only external display

True, not anymore