From 22521ab2e6fca66319b14f293500101e3bfc2f33 Mon Sep 17 00:00:00 2001 From: Kyösti Mälkki Date: Fri, 4 Jan 2019 05:16:38 +0200 Subject: amdfam10 boards: Drop extern on apicid_sp5100 MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit The value get_bus_conf() initialises this value to is discarded. Change-Id: I8382861574e6f8ab52839169502a5af7c3742daa Signed-off-by: Kyösti Mälkki Reviewed-on: https://review.coreboot.org/c/30780 Tested-by: build bot (Jenkins) Reviewed-by: Angel Pons --- src/mainboard/supermicro/h8scm_fam10/get_bus_conf.c | 9 --------- src/mainboard/supermicro/h8scm_fam10/mptable.c | 5 +---- 2 files changed, 1 insertion(+), 13 deletions(-) (limited to 'src/mainboard/supermicro/h8scm_fam10') diff --git a/src/mainboard/supermicro/h8scm_fam10/get_bus_conf.c b/src/mainboard/supermicro/h8scm_fam10/get_bus_conf.c index 7153f79df2..f6ea9334fc 100644 --- a/src/mainboard/supermicro/h8scm_fam10/get_bus_conf.c +++ b/src/mainboard/supermicro/h8scm_fam10/get_bus_conf.c @@ -25,11 +25,9 @@ * and acpi_tables busnum is default. */ u8 bus_sr5650[14]; -u32 apicid_sp5100; void get_bus_conf(void) { - u32 apicid_base; struct device *dev; int i; @@ -54,11 +52,4 @@ void get_bus_conf(void) } } } - - /* I/O APICs: APIC ID Version State Address */ - if (IS_ENABLED(CONFIG_LOGICAL_CPUS)) - apicid_base = get_apicid_base(1); - else - apicid_base = CONFIG_MAX_PHYSICAL_CPUS; - apicid_sp5100 = apicid_base + 0; } diff --git a/src/mainboard/supermicro/h8scm_fam10/mptable.c b/src/mainboard/supermicro/h8scm_fam10/mptable.c index 22f064a4ed..ca9136879a 100644 --- a/src/mainboard/supermicro/h8scm_fam10/mptable.c +++ b/src/mainboard/supermicro/h8scm_fam10/mptable.c @@ -22,14 +22,11 @@ extern u8 bus_sr5650[14]; -extern u32 apicid_sp5100; - - - static void *smp_write_config_table(void *v) { struct mp_config_table *mc; int bus_isa; + u32 apicid_sp5100; u32 apicid_sr5650; mc = (void *)(((char *)v) + SMP_FLOATING_TABLE_LEN); -- cgit v1.2.3