Change in coreboot[master]: [WIP] soc/amd: Use madt_ioapic_from_hw()

[Kyösti Mälkki (Code Review)]

Attention is currently required from: Jason Glenesk, Raul Rangel, Marshall Dawson, Felix Held.
Kyösti Mälkki has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/55571 )

Change subject: [WIP] soc/amd: Use madt_ioapic_from_hw()
......................................................................

Patch Set 1:

(2 comments)

Patchset:

PS1:
> gave this one a try and linux kernel panicked during boot and complained that the "timer doesn't wor […]

Thanks for testing.

Seems like static FCH_IOAPIC_ID and GNB_IOAPIC_ID are passed to FSP-S or FSP-M. So it's better not to use the auto-incrementing scheme for I/O APIC IDs here.

I think it depends of PCI BDF order, not linking order.

File src/soc/amd/picasso/acpi.c:

https://review.coreboot.org/c/coreboot/+/55571/comment/b9ae8687_29d61518 
PS1, Line 40: 						   GNB_IO_APIC_ADDR);
Reviewed elsewhere; it should be allowed to swap the order of the two calls above, but currently that changes GSI assignments too.