diff options
| author | Felix Held <felix-coreboot@felixheld.de> | 2023-02-21 21:03:39 +0100 |
|---|---|---|
| committer | Felix Held <felix-coreboot@felixheld.de> | 2023-02-22 22:10:46 +0000 |
| commit | 6b2b8355b321301f39335246afc0db71496961c3 (patch) | |
| tree | c443fa5cd7b90d4028ad83a1600c101093f93e45 | |
| parent | 9298ba3889bde4e7e4812ceeff4867518528aaa7 (diff) | |
nb/amd/pi/00730F01/acpi_tables: use existing IO_APIC2_ADDR definition
Use the existing IO_APIC2_ADDR definition instead of a magic value.
TEST=Timeless build results in identical image for pcengines/apu2
Signed-off-by: Felix Held <felix-coreboot@felixheld.de>
Change-Id: I7ee039e23309fdae0d614bb1fb0610d82564bf3b
Reviewed-on: https://review.coreboot.org/c/coreboot/+/73186
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Reviewed-by: Elyes Haouas <ehaouas@noos.fr>
Reviewed-by: Fred Reitberger <reitbergerfred@gmail.com>
| -rw-r--r-- | src/northbridge/amd/pi/00730F01/acpi_tables.c | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/src/northbridge/amd/pi/00730F01/acpi_tables.c b/src/northbridge/amd/pi/00730F01/acpi_tables.c index 4febfafd37..d7862158e9 100644 --- a/src/northbridge/amd/pi/00730F01/acpi_tables.c +++ b/src/northbridge/amd/pi/00730F01/acpi_tables.c @@ -2,6 +2,7 @@ #include <acpi/acpi.h> #include <arch/ioapic.h> +#include <northbridge/amd/nb_common.h> unsigned long acpi_fill_madt(unsigned long current) { @@ -14,7 +15,7 @@ unsigned long acpi_fill_madt(unsigned long current) /* TODO: Remove the hardcode */ current += acpi_create_madt_ioapic((acpi_madt_ioapic_t *)current, CONFIG_MAX_CPUS + 1, - 0xFEC20000, 24); + IO_APIC2_ADDR, 24); current += acpi_create_madt_irqoverride((acpi_madt_irqoverride_t *) current, 0, 0, 2, 0); |