summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorEugene D Myers <cedarhouse@comcast.net>2020-04-15 19:11:52 -0400
committerPatrick Georgi <pgeorgi@google.com>2020-05-02 17:04:16 +0000
commitf213f1799244ccfe4acab99c22df3dd183a3965b (patch)
treec5608443424b94ae5021a73fa63c8faffb0f1dc6
parent076605bc92730553e9adae543713f0d356a94709 (diff)
intel/stm: Drop now unneeded `num_cpus` param
Suggested by Nico Huber in CB:38766 Change-Id: Ib8a340f17a12951bc6bc67e3093046575e7b0e46 Signed-off-by: Eugene D Myers <cedarhouse@comcast.net> Reviewed-on: https://review.coreboot.org/c/coreboot/+/40438 Tested-by: build bot (Jenkins) <no-reply@coreboot.org> Reviewed-by: Angel Pons <th3fanbus@gmail.com>
-rw-r--r--src/cpu/x86/mp_init.c2
-rw-r--r--src/security/intel/stm/SmmStm.h2
-rw-r--r--src/security/intel/stm/StmPlatformSmm.c4
3 files changed, 4 insertions, 4 deletions
diff --git a/src/cpu/x86/mp_init.c b/src/cpu/x86/mp_init.c
index fa550f137a..44a29151b5 100644
--- a/src/cpu/x86/mp_init.c
+++ b/src/cpu/x86/mp_init.c
@@ -741,7 +741,7 @@ static void asmlinkage smm_do_relocation(void *arg)
mseg = mp_state.perm_smbase +
(mp_state.perm_smsize - CONFIG_MSEG_SIZE);
- stm_setup(mseg, p->cpu, runtime->num_cpus,
+ stm_setup(mseg, p->cpu,
perm_smbase,
mp_state.perm_smbase,
runtime->start32_offset);
diff --git a/src/security/intel/stm/SmmStm.h b/src/security/intel/stm/SmmStm.h
index 4f72816cae..169025553a 100644
--- a/src/security/intel/stm/SmmStm.h
+++ b/src/security/intel/stm/SmmStm.h
@@ -28,7 +28,7 @@
int load_stm_image(uintptr_t mseg);
void stm_setup(
- uintptr_t mseg, int cpu, int num_cpus, uintptr_t smbase,
+ uintptr_t mseg, int cpu, uintptr_t smbase,
uintptr_t smbase_base, uint32_t offset32);
/*
diff --git a/src/security/intel/stm/StmPlatformSmm.c b/src/security/intel/stm/StmPlatformSmm.c
index 248ccc028a..b9d2686f35 100644
--- a/src/security/intel/stm/StmPlatformSmm.c
+++ b/src/security/intel/stm/StmPlatformSmm.c
@@ -154,7 +154,7 @@ extern uint8_t *stm_resource_heap;
static int stm_load_status = 0;
-void stm_setup(uintptr_t mseg, int cpu, int num_cpus, uintptr_t smbase,
+void stm_setup(uintptr_t mseg, int cpu, uintptr_t smbase,
uintptr_t base_smbase, uint32_t offset32)
{
msr_t InitMseg;
@@ -163,7 +163,7 @@ void stm_setup(uintptr_t mseg, int cpu, int num_cpus, uintptr_t smbase,
uintptr_t addr_calc; // used to calculate the stm resource heap area
- printk(BIOS_DEBUG, "STM: set up for cpu %d/%d\n", cpu, num_cpus);
+ printk(BIOS_DEBUG, "STM: set up for cpu %d\n", cpu);
vmx_basic = rdmsr(IA32_VMX_BASIC_MSR);