Angel Pons has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/38988 )
Change subject: mainboard: Add new Ivy Bridge board ASUS P8Z77-M ......................................................................
Patch Set 11:
(2 comments)
https://review.coreboot.org/c/coreboot/+/38988/11/src/mainboard/asus/p8z77-m... File src/mainboard/asus/p8z77-m/early_init.c:
https://review.coreboot.org/c/coreboot/+/38988/11/src/mainboard/asus/p8z77-m... PS11, Line 164: pei->max_ddr3_freq = 1600; /* 1333=Sandy; 1600=Ivy */
has a devicetree option "max_mem_clock_mhz" but it looks like some other boards also set this in C c […]
The devicetree option also affects native raminit.
https://review.coreboot.org/c/coreboot/+/38988/11/src/mainboard/asus/p8z77-m... PS11, Line 165: pei->usb_port_config = { : /* {enabled, oc_pin, cable len 0x0080=<8inches/20cm} */ : { 1, 0, 0x0080 }, : { 1, 0, 0x0080 }, : { 1, 1, 0x0080 }, : { 1, 1, 0x0080 }, : { 1, 2, 0x0080 }, : { 1, 2, 0x0080 }, : { 1, 3, 0x0080 }, : { 1, 3, 0x0080 }, : { 1, 4, 0x0080 }, : { 1, 4, 0x0080 }, : { 1, 6, 0x0080 }, : { 1, 5, 0x0080 }, : { 1, 5, 0x0080 }, : { 1, 6, 0x0080 } : }; I don't think this compiles.