Evan Green has uploaded this change for review. ( https://review.coreboot.org/c/coreboot/+/64002 )
Change subject: soc/intel: Return ACPI_S4 as previous sleep state ......................................................................
soc/intel: Return ACPI_S4 as previous sleep state
pmc_prev_sleep_state() isn't handling the case where acpi_sleep_from_pm1() returns ACPI_S4. Pass that value along so it can get set as a prev_sleep_state.
BUG=b:230031158 TEST=Resume from S4 on Primus4es
Signed-off-by: Evan Green evgreen@chromium.org Change-Id: I3fb3dc428a749db80293e51a04a2096514a7b689 --- M src/soc/intel/common/block/pmc/pmclib.c 1 file changed, 3 insertions(+), 0 deletions(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/02/64002/1
diff --git a/src/soc/intel/common/block/pmc/pmclib.c b/src/soc/intel/common/block/pmc/pmclib.c index a2e3e9e..a733920 100644 --- a/src/soc/intel/common/block/pmc/pmclib.c +++ b/src/soc/intel/common/block/pmc/pmclib.c @@ -399,6 +399,9 @@ if (CONFIG(HAVE_ACPI_RESUME)) prev_sleep_state = ACPI_S3; break; + case ACPI_S4: + prev_sleep_state = ACPI_S4; + break; case ACPI_S5: prev_sleep_state = ACPI_S5; break;