[coreboot-gerrit] [L] Change in coreboot[main]: mb/asrock: Add Z87 Extreme4 (Haswell)

[Angel Pons (Code Review)]

Attention is currently required from: Jan Philipp Groß, Nicholas Chin.

Angel Pons has posted comments on this change by Jan Philipp Groß. ( https://review.coreboot.org/c/coreboot/+/84672?usp=email )

Change subject: mb/asrock: Add Z87 Extreme4 (Haswell)
......................................................................

Patch Set 2:

(2 comments)

Commit Message:

https://review.coreboot.org/c/coreboot/+/84672/comment/7f487855_a9ed5df4?usp... :
PS1, Line 50: resulting in the wrong
            : fan being controlled.
> Isn't that set in the devicetree. […]

It is, then not sure what it might be.

File src/mainboard/asrock/z87_extreme4/Kconfig:

https://review.coreboot.org/c/coreboot/+/84672/comment/4efc0f17_a6059614?usp... :
PS1, Line 28: config USBDEBUG_HCD_INDEX # FIXME: check this
            : 	int
            : 	default 2
> I got no match in any of the tested USB ports. I've tried with a USB mouse and a USB-2.0-Stick. […]

Let's leave it as-is.