[coreboot-gerrit] Change in coreboot[master]: mainboard: Add new board ASUS P8Z77-M

[Arthur Heymans (Code Review)]

Arthur Heymans has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/38988 )

Change subject: mainboard: Add new board ASUS P8Z77-M
......................................................................

Patch Set 13:

(1 comment)

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 89: /* This sequence enables early serial */
             : 	SWITCH_TO, NCT6779D_SP1,
             : 	PNP_IDX_EN, 0,
             : 	PNP_IDX_IO0, CONFIG_TTYS0_BASE >> 8,
             : 	PNP_IDX_IO0 + 1, CONFIG_TTYS0_BASE & 0xff,
             : 	PNP_IDX_EN, 1,
> See above.

No I meant that nuvoton_enable_serial is an existing function that should be called instead reimplementing it here.