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 2:
(2 comments)
https://review.coreboot.org/c/coreboot/+/47302/1/src/soc/intel/xeon_sp/nb_ac... File src/soc/intel/xeon_sp/nb_acpi.c:
https://review.coreboot.org/c/coreboot/+/47302/1/src/soc/intel/xeon_sp/nb_ac... PS1, Line 200: }; Also remove the definitions in pci_devs.h
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 see that this matches the results of the ioapicid array, but what about the other skx Pstacks? Jonathan?