From d7062425d35aa76e9921fcdb9d4bb874174a4509 Mon Sep 17 00:00:00 2001 From: Anil Kumar Date: Fri, 19 Jan 2024 15:08:47 -0800 Subject: [PATCH] soc/intel/cmn/block/pmc: Fix prev_sleep_state string name mapping commit d078ef2152052b5ce8686249dcd05ebd50010889 ("soc/intel/cmn/block/pmc: Add previous sleep state strings in log") used SLP_TYP numbers to map ACPI sleep state value. This incorrectly printed wrong string for prev_sleep_state during S5. ex: after a cold reset the previous sleep state printed was [DEBUG] prev_sleep_state 5 (S3) This patch corrects this by using ACPI sleep state numbers for mapping the prev_sleep_state values. TEST=test the logs on google/rex board after cold reset [DEBUG] prev_sleep_state 5 (S5) Signed-off-by: Anil Kumar Change-Id: I9bcdacc4d01a8d827a6abdf9af2b9e5d686ed847 Reviewed-on: https://review.coreboot.org/c/coreboot/+/80144 Reviewed-by: Jamie Ryu Tested-by: build bot (Jenkins) Reviewed-by: Subrata Banik --- src/soc/intel/common/block/pmc/pmclib.c | 10 +++++----- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/src/soc/intel/common/block/pmc/pmclib.c b/src/soc/intel/common/block/pmc/pmclib.c index f11926c2fd..c58b3a444b 100644 --- a/src/soc/intel/common/block/pmc/pmclib.c +++ b/src/soc/intel/common/block/pmc/pmclib.c @@ -463,11 +463,11 @@ int pmc_fill_power_state(struct chipset_power_state *ps) { /* Define the sleep state string */ static const char * const acpi_sleep_states[] = { - [SLP_TYP_S0] = "S0", - [SLP_TYP_S1] = "S1", - [SLP_TYP_S3] = "S3", - [SLP_TYP_S4] = "S4", - [SLP_TYP_S5] = "S5", + [ACPI_S0] = "S0", + [ACPI_S1] = "S1", + [ACPI_S3] = "S3", + [ACPI_S4] = "S4", + [ACPI_S5] = "S5", }; pmc_fill_pm_reg_info(ps);