From 81943646e3a280f04c224982911f66748aeff625 Mon Sep 17 00:00:00 2001 From: Felix Held Date: Tue, 21 Mar 2023 16:21:43 +0100 Subject: soc/amd/*/acpi: assign proper boolean values in get_pstate_core_freq Assign true/false instead of 1/0 to the valid_freq_divisor bool variable in get_pstate_core_freq. Signed-off-by: Felix Held Change-Id: I92d0eb029c55f80a2027ff6d404c63ed84282750 Reviewed-on: https://review.coreboot.org/c/coreboot/+/73880 Tested-by: build bot (Jenkins) Reviewed-by: Matt DeVillier Reviewed-by: Eric Lai --- src/soc/amd/picasso/acpi.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) (limited to 'src/soc/amd/picasso') diff --git a/src/soc/amd/picasso/acpi.c b/src/soc/amd/picasso/acpi.c index ce8caa4e51..faff625fd4 100644 --- a/src/soc/amd/picasso/acpi.c +++ b/src/soc/amd/picasso/acpi.c @@ -118,13 +118,13 @@ uint32_t get_pstate_core_freq(msr_t pstate_def) } else if ((core_freq_div >= PSTATE_DEF_LO_FREQ_DIV_MIN) && (core_freq_div <= PSTATE_DEF_LO_EIGHTH_STEP_MAX)) { /* Allow 1/8 integer steps for this range */ - valid_freq_divisor = 1; + valid_freq_divisor = true; } else if ((core_freq_div > PSTATE_DEF_LO_EIGHTH_STEP_MAX) && (core_freq_div <= PSTATE_DEF_LO_FREQ_DIV_MAX) && !(core_freq_div & 0x1)) { /* Only allow 1/4 integer steps for this range */ - valid_freq_divisor = 1; + valid_freq_divisor = true; } else { - valid_freq_divisor = 0; + valid_freq_divisor = false; } if (valid_freq_divisor) { -- cgit v1.2.3