[coreboot-gerrit] [S] Change in coreboot[main]: soc/intel/xeon_sp: Remove xeonsp_acpi_create_madt_lapics

[Shuo Liu (Code Review)]

Attention is currently required from: Arthur Heymans, Christian Walter, Johnny Lin, Jonathan Zhang, Lean Sheng Tan, Patrick Rudolph, Tim Chu.

Shuo Liu has uploaded this change for review. ( https://review.coreboot.org/c/coreboot/+/82110?usp=email )

Change subject: soc/intel/xeon_sp: Remove xeonsp_acpi_create_madt_lapics
......................................................................

soc/intel/xeon_sp: Remove xeonsp_acpi_create_madt_lapics

Remove xeonsp_acpi_create_madt_lapics due to ACPI common
code can handle the same logic well with ACPI_COMMON_MADT_LAPIC
set.

TEST=Build and boot on intel/archercity CRB

Change-Id: I06e5ff635c37253b1c8f151b62f696ff7e5e22ef
Signed-off-by: Shuo Liu 
---
M src/soc/intel/xeon_sp/include/soc/acpi.h
M src/soc/intel/xeon_sp/spr/soc_acpi.c
2 files changed, 0 insertions(+), 20 deletions(-)

git pull ssh://review.coreboot.org:29418/coreboot refs/changes/10/82110/1

diff --git a/src/soc/intel/xeon_sp/include/soc/acpi.h b/src/soc/intel/xeon_sp/include/soc/acpi.h
index e374544..e11b19e 100644
--- a/src/soc/intel/xeon_sp/include/soc/acpi.h
+++ b/src/soc/intel/xeon_sp/include/soc/acpi.h
@@ -18,7 +18,6 @@

unsigned long northbridge_write_acpi_tables(const struct device *device,
    unsigned long current, struct acpi_rsdp *rsdp);
-unsigned long xeonsp_acpi_create_madt_lapics(unsigned long current);
 unsigned long acpi_fill_cedt(unsigned long current);
 unsigned long acpi_fill_hmat(unsigned long current);
 unsigned long cxl_fill_srat(unsigned long current);
diff --git a/src/soc/intel/xeon_sp/spr/soc_acpi.c b/src/soc/intel/xeon_sp/spr/soc_acpi.c
index 1249b8f..c939dee 100644
--- a/src/soc/intel/xeon_sp/spr/soc_acpi.c
+++ b/src/soc/intel/xeon_sp/spr/soc_acpi.c
@@ -155,25 +155,6 @@
    acpigen_pop_len();
 }

-unsigned long xeonsp_acpi_create_madt_lapics(unsigned long current)
-{
-	struct device *cpu;
-	uint8_t num_cpus = 0;
-
-	for (cpu = all_devices; cpu; cpu = cpu->next) {
-		if ((cpu->path.type != DEVICE_PATH_APIC)
-		    || (cpu->upstream->dev->path.type != DEVICE_PATH_CPU_CLUSTER)) {
-			continue;
-		}
-		if (!cpu->enabled)
-			continue;
-		current = acpi_create_madt_one_lapic(current, num_cpus, cpu->path.apic.apic_id);
-		num_cpus++;
-	}
-
-	return current;
-}
-
 unsigned long acpi_fill_cedt(unsigned long current)
 {
    const IIO_UDS *hob = get_iio_uds();