[coreboot-gerrit] New patch to review for coreboot: 8e28487 google/panther: Disable DEVSLP for SATA

Matt DeVillier (matt.devillier@gmail.com) gerrit at coreboot.org
Fri Jun 13 19:31:22 CEST 2014


Matt DeVillier (matt.devillier at gmail.com) just uploaded a new patch set to gerrit, which you can find at http://review.coreboot.org/5999

-gerrit

commit 8e28487db8ad5ec0980fe1a30b6a404295fd26eb
Author: Stefan Reinauer <reinauer at chromium.org>
Date:   Wed Jan 8 15:20:59 2014 -0800

    google/panther: Disable DEVSLP for SATA
    
    Some SSD modules don't support DEVSLP correctly due to their
    firmware. Since the power savings are minimal, don't use
    DEVSLP to prevent potential problems. Some of the symptoms
    are that sometimes this causes USB devices to not work properly.
    
    BUG=chrome-os-partner:23186,
    BRANCH=panther
    TEST=Boot tested on Panther
    
    Change-Id: Iba3f721c73e0e760b6a9861ca23480ddb923df40
    Signed-off-by: Matt DeVillier <matt.devillier at gmail.com>
    Signed-off-by: Stefan Reinauer <reinauer at google.com>
    Reviewed-on: https://chromium-review.googlesource.com/181957
    Reviewed-by: Duncan Laurie <dlaurie at chromium.org>
---
 src/mainboard/google/panther/devicetree.cb | 1 +
 1 file changed, 1 insertion(+)

diff --git a/src/mainboard/google/panther/devicetree.cb b/src/mainboard/google/panther/devicetree.cb
index f461c58..9fbe8e6 100644
--- a/src/mainboard/google/panther/devicetree.cb
+++ b/src/mainboard/google/panther/devicetree.cb
@@ -55,6 +55,7 @@ chip northbridge/intel/haswell
 			register "ide_legacy_combined" = "0x0"
 			register "sata_ahci" = "0x1"
 			register "sata_port_map" = "0x1"
+			register "sata_devslp_disable" = "0x1"
 
 			register "sio_acpi_mode" = "0"
 			register "sio_i2c0_voltage" = "0" # 3.3V



More information about the coreboot-gerrit mailing list