[coreboot-gerrit] Change in coreboot[master]: soc/amd/picasso/acpi: Add MADT entry for NB ioapic, if initialized.

[Aaron Durbin (Code Review)]

Aaron Durbin has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/45056 )

Change subject: soc/amd/picasso/acpi: Add MADT entry for NB ioapic, if initialized.
......................................................................

Patch Set 5:

(1 comment)

https://review.coreboot.org/c/coreboot/+/45056/5/src/soc/amd/picasso/acpi.c 
File src/soc/amd/picasso/acpi.c:

https://review.coreboot.org/c/coreboot/+/45056/5/src/soc/amd/picasso/acpi.c@... 
PS5, Line 59: CONFIG_MAX_CPUS
I think we should do one of two things:

1. FSP has definies for io apic ids use here through fsp headers.
2. Add Kconfigs for the ids.

We shouldn't make these ids dependent on CONFIG_MAX_CPUS because we know things are being hard coded in FSP at the moment.