[coreboot-gerrit] Change in coreboot[master]: mb/asus/p8z77-v_lx2: Add new mainboard

[Angel Pons (Code Review)]

Angel Pons has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/39299 )

Change subject: mb/asus/p8z77-v_lx2: Add new mainboard
......................................................................

Patch Set 3:

(3 comments)

https://review.coreboot.org/c/coreboot/+/39299/3//COMMIT_MSG 
Commit Message:

https://review.coreboot.org/c/coreboot/+/39299/3//COMMIT_MSG@7 
PS3, Line 7: mb/asus/p8z77-v_lx2: Add new mainboard
> Then Ivy Bridge.

*replaces CPU with an i7-2600* Um, now it's not Ivy Bridge!

You know what? I'll just use the socket name: LGA1155 😄

https://review.coreboot.org/c/coreboot/+/39299/3//COMMIT_MSG@27 
PS3, Line 27:
> Add tested payload and OS?

Oh, I thought I had that somewhere. In any case, it's SeaBIOS and Arch Linux

https://review.coreboot.org/c/coreboot/+/39299/3/src/mainboard/asus/p8z77-v_... 
File src/mainboard/asus/p8z77-v_lx2/early_init.c:

https://review.coreboot.org/c/coreboot/+/39299/3/src/mainboard/asus/p8z77-v_... 
PS3, Line 48: 	/* Select SIO pin states. */
> It’s not needed in a comment, and for consistency (see below). […]

Ah, I see the inconsistency