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

[Angel Pons (Code Review)]

Attention is currently required from: Máté Kukri, Nicholas Chin.

Angel Pons has posted comments on this change by Nicholas Chin. ( https://review.coreboot.org/c/coreboot/+/81022?usp=email )

Change subject: mb/asrock: Add Z87E-ITX (Haswell)
......................................................................

Patch Set 2:

(1 comment)

File src/mainboard/asrock/z87e-itx/hda_verb.c:

https://review.coreboot.org/c/coreboot/+/81022/comment/e328788e_3c841b85?usp... :
PS2, Line 20:
> I thought Haswell already incorporates HDA verbs for the Mini-HD in chipset code

It does, so no need to add the Mini-HD verbs here. I don't think any of the Haswell boards have them (and if they do, it's probably an oversight while porting).

https://github.com/coreboot/coreboot/blob/f38c9407549b70bf813248c1958d389236...