diff options
author | Arthur Heymans <arthur@aheymans.xyz> | 2022-10-27 11:37:24 +0200 |
---|---|---|
committer | Martin Roth <martin.roth@amd.corp-partner.google.com> | 2022-10-28 21:30:10 +0000 |
commit | 6e86f77cdac0e98f3bd5f4492eb6240bc5f074e6 (patch) | |
tree | a86901f3368cacd4af1a8f5effe33e1e48b1fc4f | |
parent | 48c825ebd1beadb8ac4fd3ae687a36ff4705ab7a (diff) |
soc/intel/xeon_sp: Remove unused madt setup function
Change-Id: I248974c5a88768ee12f63fa77f3fa67a72ea510e
Signed-off-by: Arthur Heymans <arthur@aheymans.xyz>
Reviewed-on: https://review.coreboot.org/c/coreboot/+/68907
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Reviewed-by: Jonathan Zhang <jonzhang@fb.com>
-rw-r--r-- | src/soc/intel/xeon_sp/cpx/soc_acpi.c | 16 | ||||
-rw-r--r-- | src/soc/intel/xeon_sp/include/soc/acpi.h | 1 | ||||
-rw-r--r-- | src/soc/intel/xeon_sp/skx/soc_acpi.c | 16 |
3 files changed, 0 insertions, 33 deletions
diff --git a/src/soc/intel/xeon_sp/cpx/soc_acpi.c b/src/soc/intel/xeon_sp/cpx/soc_acpi.c index d6e7cf0c87..0da827d8d5 100644 --- a/src/soc/intel/xeon_sp/cpx/soc_acpi.c +++ b/src/soc/intel/xeon_sp/cpx/soc_acpi.c @@ -217,19 +217,3 @@ void soc_power_states_generation(int core, int cores_per_package) /* Fix package length */ 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 (!is_enabled_cpu(cpu)) - continue; - current += acpi_create_madt_lapic((acpi_madt_lapic_t *)current, - num_cpus, cpu->path.apic.apic_id); - num_cpus++; - } - - return current; -} diff --git a/src/soc/intel/xeon_sp/include/soc/acpi.h b/src/soc/intel/xeon_sp/include/soc/acpi.h index c2b815ddb1..cc9f2d1885 100644 --- a/src/soc/intel/xeon_sp/include/soc/acpi.h +++ b/src/soc/intel/xeon_sp/include/soc/acpi.h @@ -21,6 +21,5 @@ typedef struct { unsigned long northbridge_write_acpi_tables(const struct device *device, unsigned long current, struct acpi_rsdp *rsdp); void uncore_inject_dsdt(const struct device *device); -unsigned long xeonsp_acpi_create_madt_lapics(unsigned long current); #endif /* _SOC_ACPI_H_ */ diff --git a/src/soc/intel/xeon_sp/skx/soc_acpi.c b/src/soc/intel/xeon_sp/skx/soc_acpi.c index 73459c714f..88f24d4b2c 100644 --- a/src/soc/intel/xeon_sp/skx/soc_acpi.c +++ b/src/soc/intel/xeon_sp/skx/soc_acpi.c @@ -148,19 +148,3 @@ void uncore_inject_dsdt(const struct device *device) void soc_power_states_generation(int core, int cores_per_package) { } - -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 (!is_enabled_cpu(cpu)) - continue; - current += acpi_create_madt_lapic((acpi_madt_lapic_t *)current, - num_cpus, cpu->path.apic.apic_id); - num_cpus++; - } - - return current; -} |