[coreboot-gerrit] [XS] Change in coreboot[master]: acpi/acpi.c: Don't set MADT lapic addr on !ENV_X86

[Arthur Heymans (Code Review)]

Attention is currently required from: Kyösti Mälkki, Lance Zhao, Tim Wawrzynczak.

Arthur Heymans has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/76070?usp=email )

Change subject: acpi/acpi.c: Don't set MADT lapic addr on !ENV_X86
......................................................................

Patch Set 1:

(1 comment)

File src/acpi/acpi.c:

https://review.coreboot.org/c/coreboot/+/76070/comment/c665a451_e7890c85 :
PS1, Line 161: 	if (ENV_X86)
> I'm fine with this, but one might consider moving this and the COMMON_MADT conditionals below over to acpi_apic.c. Current expectation is to not see LAPIC or IOAPIC entries for ARM at least?

I plan on adding ACPI_HAVE_GIC... here to implement the ARM equivalents. I guess it's safe to say that those will be arch bound. Maybe that can be made explicit in Kconfig?