[coreboot-gerrit] Change in coreboot[master]: soc/intel/xeon_sp: Fix generating DMAR IOAPIC

[Arthur Heymans (Code Review)]

Arthur Heymans has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/47253 )

Change subject: soc/intel/xeon_sp: Fix generating DMAR IOAPIC
......................................................................

Patch Set 1:

(1 comment)

https://review.coreboot.org/c/coreboot/+/47253/1/src/soc/intel/xeon_sp/nb_ac... 
File src/soc/intel/xeon_sp/nb_acpi.c:

https://review.coreboot.org/c/coreboot/+/47253/1/src/soc/intel/xeon_sp/nb_ac... 
PS1, Line 183: static int stack_to_ioapic_idx(const IIO_STACKS stack)
> I think that this needs to go in soc_util.c to support each cpu. […]

Ok. Maybe an stack to ioapic_id callback then? That way the synchronisation between the APIC and DMAR acpi table is not hacky like here?