diff options
| author | Anil Kumar <anil.kumar.k@intel.com> | 2024-01-19 15:08:47 -0800 |
|---|---|---|
| committer | Subrata Banik <subratabanik@google.com> | 2024-01-22 03:19:25 +0000 |
| commit | d7062425d35aa76e9921fcdb9d4bb874174a4509 (patch) | |
| tree | f63e872c2b9b6a07c6dae3c56d29f682f538ee26 /src/soc/intel | |
| parent | d06fa34103da31682a8ad3a431f52a9ae5619977 (diff) | |
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 <anil.kumar.k@intel.com>
Change-Id: I9bcdacc4d01a8d827a6abdf9af2b9e5d686ed847
Reviewed-on: https://review.coreboot.org/c/coreboot/+/80144
Reviewed-by: Jamie Ryu <jamie.m.ryu@intel.com>
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Reviewed-by: Subrata Banik <subratabanik@google.com>
Diffstat (limited to 'src/soc/intel')
| -rw-r--r-- | src/soc/intel/common/block/pmc/pmclib.c | 10 |
1 files 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); |