From 0425b0a4bbd964bda9368aa9adc40a71d0b83362 Mon Sep 17 00:00:00 2001 From: Marshall Dawson Date: Mon, 1 Oct 2018 14:21:47 -0600 Subject: 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 Reviewed-on: https://review.coreboot.org/28863 Tested-by: build bot (Jenkins) Reviewed-by: Paul Menzel Reviewed-by: Martin Roth --- src/soc/amd/stoneyridge/acpi/sb_pci0_fch.asl | 4 ---- 1 file changed, 4 deletions(-) (limited to 'src/soc') 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 */ -- cgit v1.2.3