diff options
| author | Christoph Grenz <christophg+cb@grenz-bonn.de> | 2013-07-04 03:41:39 +0200 |
|---|---|---|
| committer | Patrick Georgi <patrick@georgi-clan.de> | 2013-08-15 20:32:35 +0200 |
| commit | dc92d682ff8d5d2439f4c5cdc4e449bba6d651bb (patch) | |
| tree | c061420aaef19549936f6e02a9799572236ab295 /src/superio/winbond | |
| parent | ec3a462d03527b2f4e620f43a5fa8fe518c5dee7 (diff) | |
| download | coreboot-dc92d682ff8d5d2439f4c5cdc4e449bba6d651bb.tar.xz | |
w83627hf/acpi: Fix logical device power down in ACPI
As Nico noticed for the W83627DHG, the power management bits to power down
individual logical devices on Winbond superios are named counterintuitively
and need to be set when the logical device should be powered.
This corrects the power management methods for the W83627HF.
Change-Id: I98bccd550a0513c62bfa9480275f88c566691bc8
Signed-off-by: Christoph Grenz <christophg+cb@grenz-bonn.de>
Reviewed-on: http://review.coreboot.org/3605
Tested-by: build bot (Jenkins)
Reviewed-by: Ronald G. Minnich <rminnich@gmail.com>
Reviewed-by: Paul Menzel <paulepanter@users.sourceforge.net>
Diffstat (limited to 'src/superio/winbond')
| -rw-r--r-- | src/superio/winbond/w83627hf/acpi/superio.asl | 24 |
1 files changed, 12 insertions, 12 deletions
diff --git a/src/superio/winbond/w83627hf/acpi/superio.asl b/src/superio/winbond/w83627hf/acpi/superio.asl index 05b570b3a9..8f79b2594c 100644 --- a/src/superio/winbond/w83627hf/acpi/superio.asl +++ b/src/superio/winbond/w83627hf/acpi/superio.asl @@ -260,13 +260,13 @@ Device(SIO) { /* Disable power saving mode */ Method (_PS0) { ENCM (0xFF) - Store (Zero, FDPW) + Store (One, FDPW) EXCM () } /* Enable power saving mode */ Method (_PS1) { ENCM (0xFF) - Store (One, FDPW) + Store (Zero, FDPW) EXCM () } @@ -471,12 +471,12 @@ Device(SIO) { } Method (_PS0) { ENCM (0xFF) - Store (Zero, PRPW) + Store (One, PRPW) EXCM () } Method (_PS1) { ENCM (0xFF) - Store (One, PRPW) + Store (Zero, PRPW) EXCM () } @@ -648,12 +648,12 @@ Device(SIO) { } Method (_PS0) { ENCM (0xFF) - Store (Zero, UAPW) + Store (One, UAPW) EXCM () } Method (_PS1) { ENCM (0xFF) - Store (One, UAPW) + Store (Zero, UAPW) EXCM () } @@ -773,12 +773,12 @@ Device(SIO) { } Method (_PS0) { ENCM (0xFF) - Store (Zero, UBPW) + Store (One, UBPW) EXCM () } Method (_PS1) { ENCM (0xFF) - Store (One, UBPW) + Store (Zero, UBPW) EXCM () } @@ -898,12 +898,12 @@ Device(SIO) { } Method (_PS0) { ENCM (0xFF) - Store (Zero, UBPW) + Store (One, UBPW) EXCM () } Method (_PS1) { ENCM (0xFF) - Store (One, UBPW) + Store (Zero, UBPW) EXCM () } @@ -1423,14 +1423,14 @@ Device(SIO) { Method (_PS0) { ENCM (0xFF) - Store (Zero, HWPW) + Store (One, HWPW) EXCM () } Method (_PS1) { ENCM (0xFF) - Store (One, HWPW) + Store (Zero, HWPW) EXCM () } |