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

[Paul Menzel (Code Review)]

Paul Menzel 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 10:

(2 comments)

https://review.coreboot.org/c/coreboot/+/38988/10/src/mainboard/asus/p8z77-m... 
File src/mainboard/asus/p8z77-m/early_init.c:

https://review.coreboot.org/c/coreboot/+/38988/10/src/mainboard/asus/p8z77-m... 
PS10, Line 100: int
unsigned int

https://review.coreboot.org/c/coreboot/+/38988/10/src/mainboard/asus/p8z77-m... 
PS10, Line 108: 	for (i = 0; i < max; i += 2)
> Bootblock code space efficiency. […]

Difficult question how to optimize between code size and readability. Maybe add this as a separate commit, and keep it unmerged, until it’s clear that you need the 239 bytes?