diff options
| author | Felix Held <felix-coreboot@felixheld.de> | 2022-07-12 01:02:48 +0200 |
|---|---|---|
| committer | Felix Held <felix-coreboot@felixheld.de> | 2022-07-13 10:44:08 +0000 |
| commit | 207225c68619d4f4c24138454fd2b622ea76dd47 (patch) | |
| tree | 093b54edefd016775aefa82e9461111257955836 /src | |
| parent | 29ae26704c4c4a2ee555348b4bd290d20004d461 (diff) | |
cpu/x86/mp_init: retype do_smm element in mp_state struct to bool
The do_smm struct element in the mp_state struct was an int even though
it only had two possible states, so change it to bool to make this more
obvious. Also change the return type of is_smm_enabled from int to bool.
Signed-off-by: Felix Held <felix-coreboot@felixheld.de>
Change-Id: I8d2d95f0497649d67565243d14a5ab9c9cdda412
Reviewed-on: https://review.coreboot.org/c/coreboot/+/65776
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Reviewed-by: Arthur Heymans <arthur@aheymans.xyz>
Diffstat (limited to 'src')
| -rw-r--r-- | src/cpu/x86/mp_init.c | 8 | ||||
| -rw-r--r-- | src/security/intel/stm/SmmStm.c | 2 |
2 files changed, 5 insertions, 5 deletions
diff --git a/src/cpu/x86/mp_init.c b/src/cpu/x86/mp_init.c index 7cc7fbcd18..3f80694533 100644 --- a/src/cpu/x86/mp_init.c +++ b/src/cpu/x86/mp_init.c @@ -675,23 +675,23 @@ struct mp_state { size_t perm_smsize; size_t smm_save_state_size; uintptr_t reloc_start32_offset; - int do_smm; + bool do_smm; } mp_state; -static int is_smm_enabled(void) +static bool is_smm_enabled(void) { return CONFIG(HAVE_SMI_HANDLER) && mp_state.do_smm; } static void smm_disable(void) { - mp_state.do_smm = 0; + mp_state.do_smm = false; } static void smm_enable(void) { if (CONFIG(HAVE_SMI_HANDLER)) - mp_state.do_smm = 1; + mp_state.do_smm = true; } /* diff --git a/src/security/intel/stm/SmmStm.c b/src/security/intel/stm/SmmStm.c index 1ebe77d2f1..4cb2d0126c 100644 --- a/src/security/intel/stm/SmmStm.c +++ b/src/security/intel/stm/SmmStm.c @@ -72,7 +72,7 @@ extern struct mp_state { uintptr_t perm_smbase; size_t perm_smsize; size_t smm_save_state_size; - int do_smm; + bool do_smm; } mp_state; typedef union { |