GNUBoot
/
coreboot-kgpe-d16
2
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

soc/intel/cmn/block/pmc: Fix prev_sleep_state string name mapping 

commit d078ef2152
("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>
This commit is contained in:
Anil Kumar 2024-01-19 15:08:47 -08:00 committed by Subrata Banik
parent d06fa34103
commit d7062425d3
1 changed files with 5 additions and 5 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

10
src/soc/intel/common/block/pmc/pmclib.c
View File

@ -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);