Angel Pons has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/33328 )
Change subject: mainboard: Add support for ASUS P8Z77-M PRO desktop mainboard ......................................................................
Patch Set 51: Code-Review+1
(1 comment)
Looks very good!
If you haven't done so, I would suggest testing with four memory modules. Even if you don't have four matched sticks, you could put them in pairs. If the mainboard slots are in order, that would mean installing a matched pair on the blue slots, and another matched pair on the black slots.
https://review.coreboot.org/#/c/33328/38/Documentation/mainboard/asus/p8z77-... File Documentation/mainboard/asus/p8z77-m_pro.md:
https://review.coreboot.org/#/c/33328/38/Documentation/mainboard/asus/p8z77-... PS38, Line 110: Gb
Byte
Ack. the case of the 'b' letter distinguishes between bits and bytes. 'Gb' stands for Gigabit, where 'GB' stands for Gigabyte.