diff options
| author | Arthur Heymans <arthur@aheymans.xyz> | 2023-04-25 15:48:46 +0200 |
|---|---|---|
| committer | Lean Sheng Tan <sheng.tan@9elements.com> | 2023-04-26 15:41:03 +0000 |
| commit | cbc5d3f34b87db779829eabc90c32780a3865a56 (patch) | |
| tree | c91d7e0d243d3989cbc13825bc833fee6a3b4905 /src/southbridge/intel/common/acpi | |
| parent | 1dc55aa35ecd814a0080653d3fc3199584724057 (diff) | |
soc/intel: Don't report _S1 state when unsupported
Since skylake Intel hardware does not support this sleep state. Trying
to enter S1 by having the OS enter sleep results in a system hang on at
least Alder lake (prodrive/atlas).
CONFIG_SOC_INTEL_COMMON_BLOCK_PMC is a good proxy whether devices
support 'skylake style' PMC PCI device for ACPI registers.
Signed-off-by: Arthur Heymans <arthur@aheymans.xyz>
Change-Id: Ic9e19410696240755e8714db53a0525284f3a2da
Reviewed-on: https://review.coreboot.org/c/coreboot/+/74760
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Reviewed-by: Lean Sheng Tan <sheng.tan@9elements.com>
Reviewed-by: Werner Zeh <werner.zeh@siemens.com>
Diffstat (limited to 'src/southbridge/intel/common/acpi')
| -rw-r--r-- | src/southbridge/intel/common/acpi/sleepstates.asl | 2 |
1 files changed, 2 insertions, 0 deletions
diff --git a/src/southbridge/intel/common/acpi/sleepstates.asl b/src/southbridge/intel/common/acpi/sleepstates.asl index eae7642d2b..30e12a03ed 100644 --- a/src/southbridge/intel/common/acpi/sleepstates.asl +++ b/src/southbridge/intel/common/acpi/sleepstates.asl @@ -2,7 +2,9 @@ Name(\_S0, Package(){0x0,0x0,0x0,0x0}) #if !CONFIG(HAVE_ACPI_RESUME) +#if !CONFIG(ACPI_S1_NOT_SUPPORTED) Name(\_S1, Package(){0x1,0x0,0x0,0x0}) +#endif #else Name(\_S3, Package(){0x5,0x0,0x0,0x0}) #endif |