[coreboot-gerrit] New patch to review for coreboot: amd/pi/hudson/acpi: Only declare S3 if it is supported

Marc Jones (marc@marcjonesconsulting.com) gerrit at coreboot.org
Sat Feb 25 00:49:05 CET 2017


Marc Jones (marc at marcjonesconsulting.com) just uploaded a new patch set to gerrit, which you can find at https://review.coreboot.org/18493

-gerrit

commit 1c1eded5c30db113d35e5caa15cf187485501072
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