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

[Angel Pons (Code Review)]

Angel Pons 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: -Code-Review

(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. […]

The MADT for server platforms is much more complex, and basically does everything differently.

I'm pretty sure I've ranted about it in the past: Xeon-SP is NOT a SoC!