[coreboot-gerrit] Change in coreboot[master]: soc/intel/xeon_sp: Synchronize DMAR and MADT IOAPIC id's

[Marc Jones (Code Review)]

Marc Jones has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/47302 )

Change subject: soc/intel/xeon_sp: Synchronize DMAR and MADT IOAPIC id's
......................................................................

Patch Set 3: Code-Review+1

(1 comment)

https://review.coreboot.org/c/coreboot/+/47302/2/src/soc/intel/xeon_sp/skx/s... 
File src/soc/intel/xeon_sp/skx/soc_util.c:

https://review.coreboot.org/c/coreboot/+/47302/2/src/soc/intel/xeon_sp/skx/s... 
PS2, Line 306: }
> I don't know if PSTACK3/4 for MCP have IOAPICs. […]

Looking at the hob display, I'm convinced that this is ok. I think it would be better to use the hob to check personalities instead of hard coding, but this is correct for these cpus.