[coreboot-gerrit] Change in coreboot[master]: soc/amd/picasso: Give the mainboard the ability to modify the MADT table

[Aaron Durbin (Code Review)]

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

Change subject: soc/amd/picasso: Give the mainboard the ability to modify the MADT table
......................................................................

Patch Set 1:

(1 comment)

https://review.coreboot.org/c/coreboot/+/41577/1/src/soc/amd/picasso/acpi.c 
File src/soc/amd/picasso/acpi.c:

https://review.coreboot.org/c/coreboot/+/41577/1/src/soc/amd/picasso/acpi.c@... 
PS1, Line 40: unsigned long __weak acpi_mb_madt_irqoverride(unsigned long current)
Why is the ACPI table creation directly exposed? Can't we drive this through configuration tables instead of making the mainboards implement the proper sequence?