From 6b2b8355b321301f39335246afc0db71496961c3 Mon Sep 17 00:00:00 2001
From: Felix Held <felix-coreboot@felixheld.de>
Date: Tue, 21 Feb 2023 21:03:39 +0100
Subject: 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>
---
 src/northbridge/amd/pi/00730F01/acpi_tables.c | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

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);
-- 
cgit v1.2.3