[coreboot-gerrit] Change in coreboot[master]: [WIP] mb/asus/sklkbl_h: Add base for Asus Skylake Boards

[Angel Pons (Code Review)]

Attention is currently required from:  Felix Singer, Marvin Drees.
Angel Pons has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/48648 )

Change subject: [WIP] mb/asus/sklkbl_h: Add base for Asus Skylake Boards
......................................................................

Patch Set 16:

(1 comment)

Commit Message:

https://review.coreboot.org/c/coreboot/+/48648/comment/2fdb3564_d0635d68 
PS16, Line 24: - Figure out why system_reset() gets called during
             : FSP MemoryInit on cold boot once
If by "cold boot" you mean after the power supply was unplugged, it's completely normal.

I'm pretty sure it's the same as on Haswell, where the ME needs a `system_reset()` to perform "dynamic fusing". I don't know what exactly this "dynamic fusing" does, but it updates the CAPID0 register in the host bridge's PCI config space, most likely according to the capabilities of the processor and PCH. I think this is how Intel restricts memory overclocking to mainboards with a Z-series PCH, among other things.