[coreboot-gerrit] Change in coreboot[master]: soc/intel/common/block/acpi: Allow additional soc overrides

[Nico Huber (Code Review)]

Nico Huber has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/48246 )

Change subject: soc/intel/common/block/acpi: Allow additional soc overrides
......................................................................

Patch Set 4:

(1 comment)

https://review.coreboot.org/c/coreboot/+/48246/4/src/soc/intel/common/block/... 
File src/soc/intel/common/block/acpi/acpi.c:

https://review.coreboot.org/c/coreboot/+/48246/4/src/soc/intel/common/block/... 
PS4, Line 85: __weak unsigned long acpi_fill_madt(unsigned long current)
As usual, I would prefer not to have a weak function with actual content.
In the past this has led to confusion and bugs.

What is it that the new implementation would make differently? Could it
be handled by a Kconfig switch maybe?