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 18:
(3 comments)
https://review.coreboot.org/#/c/33328/18/src/mainboard/asus/p8z77-m_pro/devi... File src/mainboard/asus/p8z77-m_pro/devicetree.cb:
https://review.coreboot.org/#/c/33328/18/src/mainboard/asus/p8z77-m_pro/devi... PS18, Line 125: end trailing whitespace
https://review.coreboot.org/#/c/33328/18/src/mainboard/asus/p8z77-m_pro/roms... File src/mainboard/asus/p8z77-m_pro/romstage.c:
https://review.coreboot.org/#/c/33328/18/src/mainboard/asus/p8z77-m_pro/roms... PS18, Line 74: * See Nuvuton 6779D datasheet, section trailing whitespace
https://review.coreboot.org/#/c/33328/18/src/mainboard/asus/p8z77-m_pro/roms... PS18, Line 75: * "21.10 Configuration Register->Logical Device A (ACPI)", code indent should use tabs where possible