diff options
author | Marshall Dawson <marshalldawson3rd@gmail.com> | 2018-10-01 14:21:47 -0600 |
---|---|---|
committer | Martin Roth <martinroth@google.com> | 2018-10-05 01:36:44 +0000 |
commit | 0425b0a4bbd964bda9368aa9adc40a71d0b83362 (patch) | |
tree | 5315db17ec24845b2a82a3c6207fbe85c0581b61 | |
parent | 0b9896f328d3c07f9f9db697d1a8fcde25200e00 (diff) | |
download | coreboot-0425b0a4bbd964bda9368aa9adc40a71d0b83362.tar.xz |
amd/stoneyridge: Remove SATA D0 on suspend
Remove the step of setting the SATA controller to S0 as the system is
entering S3. This had been duplicated from AMD's FchCarrizo.asl file,
but upon closer inspection, the conditions for this step to run cannot
be met. This does not affect Grunt's behavior, as the SATA controller
is disabled.
TEST=Suspend and resume Grunt
BUG=b:77602074
Change-Id: Ib269a5363d03c7048abd0c8a9a28df92a773790c
Signed-off-by: Marshall Dawson <marshalldawson3rd@gmail.com>
Reviewed-on: https://review.coreboot.org/28863
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Reviewed-by: Paul Menzel <paulepanter@users.sourceforge.net>
Reviewed-by: Martin Roth <martinroth@google.com>
-rw-r--r-- | src/soc/amd/stoneyridge/acpi/sb_pci0_fch.asl | 4 |
1 files changed, 0 insertions, 4 deletions
diff --git a/src/soc/amd/stoneyridge/acpi/sb_pci0_fch.asl b/src/soc/amd/stoneyridge/acpi/sb_pci0_fch.asl index 77c288b870..7ac91395c4 100644 --- a/src/soc/amd/stoneyridge/acpi/sb_pci0_fch.asl +++ b/src/soc/amd/stoneyridge/acpi/sb_pci0_fch.asl @@ -592,10 +592,6 @@ Method(FPTS,0, Serialized) /* FCH _PTS */ if(LNotEqual(U2TD, 0x03)) { FDDC(18, 3) } - - if(LNotEqual(SATD, 0x00)) { - FDDC(15, 0) /* wake SATA before sleep */ - } } Method(FWAK,0, Serialized) /* FCH _WAK */ |