[coreboot-gerrit] Patch set updated for coreboot: amd/pi/hudson/acpi: Only declare S3 if it is supported

Marshall Dawson (marshalldawson3rd@gmail.com) gerrit at coreboot.org
Fri Mar 3 00:05:34 CET 2017


Marshall Dawson (marshalldawson3rd at gmail.com) just uploaded a new patch set to gerrit, which you can find at https://review.coreboot.org/18493

-gerrit

commit 588a0d391379584357c52f068e83b82531c298ec
Author: Marc Jones <marcj303 at gmail.com>
Date:   Wed Feb 22 16:21:49 2017 -0700

    amd/pi/hudson/acpi: Only declare S3 if it is supported
    
    Only declare S3 support in ACPI if CONFIG_HAVE_ACPI_RESUME
    is set.
    
    Change-Id: I6f8f62a92478f3db5de6feaa9822baad3f8e147e
    Signed-off-by: Marc Jones <marcj303 at gmail.com>
---
 src/southbridge/amd/pi/hudson/acpi/sleepstates.asl | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/src/southbridge/amd/pi/hudson/acpi/sleepstates.asl b/src/southbridge/amd/pi/hudson/acpi/sleepstates.asl
index 912c0df..d93f068 100644
--- a/src/southbridge/amd/pi/hudson/acpi/sleepstates.asl
+++ b/src/southbridge/amd/pi/hudson/acpi/sleepstates.asl
@@ -23,9 +23,11 @@ If (LAnd(SSFG, 0x01)) {
 If (LAnd(SSFG, 0x02)) {
 	Name(\_S2, Package () {0x02, 0x02, 0x00, 0x00} )	/* (S2) - "light" Suspend to RAM */
 }
+#if CONFIG_HAVE_ACPI_RESUME
 If (LAnd(SSFG, 0x04)) {
 	Name(\_S3, Package () {0x03, 0x03, 0x00, 0x00} )	/* (S3) - Suspend to RAM */
 }
+#endif
 If (LAnd(SSFG, 0x08)) {
 	Name(\_S4, Package () {0x04, 0x04, 0x00, 0x00} )	/* (S4) - Suspend to Disk */
 }



More information about the coreboot-gerrit mailing list