[S] Change in coreboot[master]: soc/intel/baytrail,braswell: Use COMMON_MADT_IOAPIC

[Kyösti Mälkki (Code Review)]

Kyösti Mälkki has uploaded this change for review. ( https://review.coreboot.org/c/coreboot/+/74351 )

Change subject: soc/intel/baytrail,braswell: Use COMMON_MADT_IOAPIC
......................................................................

soc/intel/baytrail,braswell: Use COMMON_MADT_IOAPIC

Change IRQ #0 to GSI #2 override to positive edge trigger from
the bus ISA default (positive edge).

Change-Id: I3a61a0ceb0e6e4a09570beef6d0170354eb498ac
Signed-off-by: Kyösti Mälkki 
---
M src/soc/intel/baytrail/Kconfig
M src/soc/intel/baytrail/acpi.c
M src/soc/intel/braswell/Kconfig
M src/soc/intel/braswell/acpi.c
4 files changed, 15 insertions(+), 24 deletions(-)

git pull ssh://review.coreboot.org:29418/coreboot refs/changes/51/74351/1

diff --git a/src/soc/intel/baytrail/Kconfig b/src/soc/intel/baytrail/Kconfig
index 0f908b6..958af34 100644
--- a/src/soc/intel/baytrail/Kconfig
+++ b/src/soc/intel/baytrail/Kconfig
@@ -7,6 +7,7 @@

config CPU_SPECIFIC_OPTIONS
    def_bool y
+	select ACPI_COMMON_MADT_IOAPIC
    select ACPI_COMMON_MADT_LAPIC
    select ACPI_INTEL_HARDWARE_SLEEP_VALUES
    select ARCH_X86
diff --git a/src/soc/intel/baytrail/acpi.c b/src/soc/intel/baytrail/acpi.c
index 6433e2c..e2cb12b 100644
--- a/src/soc/intel/baytrail/acpi.c
+++ b/src/soc/intel/baytrail/acpi.c
@@ -283,15 +283,3 @@
    /* Add a method to notify processor nodes */
    acpigen_write_processor_cnot(pattrs->num_cpus);
 }
-
-unsigned long acpi_fill_madt(unsigned long current)
-{
-	/* IOAPIC */
-	current += acpi_create_madt_ioapic_from_hw((acpi_madt_ioapic_t *)current, IO_APIC_ADDR);
-
-	/* INT_SRC_OVR */
-	current += acpi_create_madt_irqoverride((void *)current, 0, 0, 2, 0);
-	current += acpi_create_madt_sci_override((void *)current);
-
-	return current;
-}
diff --git a/src/soc/intel/braswell/Kconfig b/src/soc/intel/braswell/Kconfig
index 8d540f3..f618118 100644
--- a/src/soc/intel/braswell/Kconfig
+++ b/src/soc/intel/braswell/Kconfig
@@ -7,6 +7,7 @@

config CPU_SPECIFIC_OPTIONS
    def_bool y
+	select ACPI_COMMON_MADT_IOAPIC
    select ACPI_COMMON_MADT_LAPIC
    select ACPI_INTEL_HARDWARE_SLEEP_VALUES
    select ARCH_X86
diff --git a/src/soc/intel/braswell/acpi.c b/src/soc/intel/braswell/acpi.c
index 59aaca3..afd7e9d 100644
--- a/src/soc/intel/braswell/acpi.c
+++ b/src/soc/intel/braswell/acpi.c
@@ -314,18 +314,6 @@
    acpigen_write_processor_cnot(pattrs->num_cpus);
 }

-unsigned long acpi_fill_madt(unsigned long current)
-{
-	/* IOAPIC */
-	current += acpi_create_madt_ioapic_from_hw((acpi_madt_ioapic_t *)current, IO_APIC_ADDR);
-
-	/* INT_SRC_OVR */
-	current += acpi_create_madt_irqoverride((void *)current, 0, 0, 2, 0);
-	current += acpi_create_madt_sci_override((void *)current);
-
-	return current;
-}
-
 unsigned long southcluster_write_acpi_tables(const struct device *device, unsigned long current,
    				     struct acpi_rsdp *rsdp)
 {