[coreboot-gerrit] [M] Change in coreboot[main]: soc/amd: commonize PCI root IOAPIC initialization

[Felix Held (Code Review)]

Attention is currently required from: Fred Reitberger, Jason Glenesk, Matt DeVillier.

Felix Held has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/80286?usp=email )

Change subject: soc/amd: commonize PCI root IOAPIC initialization
......................................................................

Patch Set 1:

(2 comments)

Commit Message:

https://review.coreboot.org/c/coreboot/+/80286/comment/0ed0a5e8_dbc41989 :
PS1, Line 13: Gena
> Genoa

Done

https://review.coreboot.org/c/coreboot/+/80286/comment/4156ef5f_13369ce6 :
PS1, Line 17: Both the FCH IOAPIC and the PCIe root IOAPIC are still initialized
            : on Mandolin
> does this change affect the timing or order of the IOAPIC init?

haven't checked the timing before this patch, but the ioapic ids stay the same, since we have a special case for the fch ioapic that needs to be the first one in the system. since the function that gets called only sets the ioapic id and clears the vectors i don't see any possible problems there