[coreboot-gerrit] Change in coreboot[master]: mb/prodrive/hermes: Fix PCIe ClkSrc configuration

[Michael Niewöhner (Code Review)]

Attention is currently required from: Patrick Rudolph, Christian Walter, Angel Pons.
Michael Niewöhner has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/58065 )

Change subject: mb/prodrive/hermes: Fix PCIe ClkSrc configuration
......................................................................

Patch Set 2: Code-Review+2

(2 comments)

Commit Message:

https://review.coreboot.org/c/coreboot/+/58065/comment/8a62a0fb_bb6ffff2 
PS2, Line 10: Apparently, FSP does not turn off unused PCIe clock sources when using
> Yes, the ClkSrc configuration in FSP UPDs is wrong. […]

Ah! Now I got it 😄 Yes, I guess it's not necessary since wrong values is reason enough for fixing it :-)

https://review.coreboot.org/c/coreboot/+/58065/comment/7fa36c2d_935885fa 
PS2, Line 12: firmware.
> I changed the CSME IFD to match the SPS IFD (and devicetree) and now all PCIe devices show up. […]

erm, that change is required, bc the UPDs are plain wrong