[L] Change in coreboot[main]: mb/hp: Add Pro 3500 series (Sandy/Ivy Bridge)

[Joel Linn (Code Review)]

Attention is currently required from: Matt DeVillier, Nicholas Chin.

Joel Linn has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/81368?usp=email )

Change subject: mb/hp: Add Pro 3500 series (Sandy/Ivy Bridge)
......................................................................

Patch Set 2:

(3 comments)

File src/mainboard/hp/pro_3500_series/Kconfig:

https://review.coreboot.org/c/coreboot/+/81368/comment/efc2ac99_e47ac625 :
PS2, Line 37: # FIXME: check this
> if checked, remove fixme

I unfortunately have no EHCI debugger so I can't test this

File src/mainboard/hp/pro_3500_series/early_init.c:

https://review.coreboot.org/c/coreboot/+/81368/comment/9b854217_2163701b :
PS2, Line 18: FIXME: Unknown current: RCBA(0x350c)=0x350c
> address fixme comments

Can you point me in some direction/to some documentation about the usb configuration?

File src/superio/ite/common/early_serial.c:

PS2:
> 2nd

Yes will move them to the previous changeset in the chain as it was already suggested there.