Vlado Cibic 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 53:
(1 comment)
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.
A friend lent me a 4x4GB 1333 kit and works perfectly! Added it to the doc. I'm trying to fix the COM port on board and trying to figure our some internal headers I'm not sure what they are used for. Marking patch as WIP again.
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
Ack. the case of the 'b' letter distinguishes between bits and bytes. […]
Done