[S] Change in coreboot[master]: sb/intel/sleepstates.asl: Use variable to enable sleepstates

[Arthur Heymans (Code Review)]

Arthur Heymans has uploaded this change for review. ( https://review.coreboot.org/c/coreboot/+/74761 )

Change subject: sb/intel/sleepstates.asl: Use variable to enable sleepstates
......................................................................

sb/intel/sleepstates.asl: Use variable to enable sleepstates

In order to make supported sleep states a runtime configuration option use a
variable. A follow-up patch will implement updating this variable based on an
SSDT generated IntObj.

Signed-off-by: Arthur Heymans 
Change-Id: I6910c2c75e668e6f75a6f431813edeb59d52dd93
---
M src/southbridge/intel/common/acpi/sleepstates.asl
1 file changed, 41 insertions(+), 8 deletions(-)

git pull ssh://review.coreboot.org:29418/coreboot refs/changes/61/74761/1

diff --git a/src/southbridge/intel/common/acpi/sleepstates.asl b/src/southbridge/intel/common/acpi/sleepstates.asl
index 7028c63..baee300 100644
--- a/src/southbridge/intel/common/acpi/sleepstates.asl
+++ b/src/southbridge/intel/common/acpi/sleepstates.asl
@@ -1,12 +1,31 @@
 /* SPDX-License-Identifier: GPL-2.0-only */

-Name(_S0, Package(){0x0,0x0,0x0,0x0})
-#if !CONFIG(HAVE_ACPI_RESUME) && !CONFIG(PLATFORM_LACKS_ACPI_S1)
-Name(_S1, Package(){0x1,0x0,0x0,0x0})
+/* S1 support: bit 0, S2 Support: bit 1, etc. S0 & S5 assumed */
+#if CONFIG(HAVE_ACPI_RESUME)
+Name (SSFG, 0x0D)
 #else
-Name(_S3, Package(){0x5,0x0,0x0,0x0})
+Name (SSFG, 0x09)
 #endif
-#if !CONFIG(DISABLE_ACPI_HIBERNATE)
-Name(_S4, Package(){0x6,0x0,0x0,0x0})
-#endif
-Name(_S5, Package(){0x7,0x0,0x0,0x0})
+
+If (CONFIG(PLATFORM_LACKS_ACPI_S1)) {
+	SSFG &= 0xfe
+}
+
+If (CONFIG(DISABLE_ACPI_HIBERNATE)) {
+	SSFG &= 0xf7
+}
+
+/* Supported sleep states: */
+Name(_S0, Package () {0x00, 0x00, 0x00, 0x00} )	/* (S0) - working state */
+
+If (SSFG & 0x01) {
+	Name(_S1, Package () {0x01, 0x00, 0x00, 0x00} )	/* (S1) - sleeping w/CPU context */
+}
+If (SSFG & 0x04) {
+	Name(_S3, Package () {0x05, 0x00, 0x00, 0x00} )	/* (S3) - Suspend to RAM */
+}
+If (SSFG & 0x08) {
+	Name(_S4, Package () {0x06, 0x04, 0x00, 0x00} )	/* (S4) - Suspend to Disk */
+}
+
+Name(_S5, Package () {0x07, 0x00, 0x00, 0x00} )	/* (S5) - Soft Off */