build bot (Jenkins) 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 12:
(2 comments)
https://review.coreboot.org/#/c/33328/12/src/mainboard/asus/p8z77-m_pro/cmos... File src/mainboard/asus/p8z77-m_pro/cmos.layout:
https://review.coreboot.org/#/c/33328/12/src/mainboard/asus/p8z77-m_pro/cmos... PS12, Line 167: # in an effective way. See this: trailing whitespace
https://review.coreboot.org/#/c/33328/12/src/mainboard/asus/p8z77-m_pro/roms... File src/mainboard/asus/p8z77-m_pro/romstage.c:
https://review.coreboot.org/#/c/33328/12/src/mainboard/asus/p8z77-m_pro/roms... PS12, Line 171: read_spd(&spd[0], 0x50, id_only); /* got the 0x50-0x53 inspecting SMBus */ line over 80 characters