[coreboot-gerrit] Patch set updated for coreboot: ac5d919 baytrail: Put devices in ACPI mode after setup

Aaron Durbin (adurbin@google.com) gerrit at coreboot.org
Mon May 12 16:41:04 CEST 2014


Aaron Durbin (adurbin at google.com) just uploaded a new patch set to gerrit, which you can find at http://review.coreboot.org/5017

-gerrit

commit ac5d9199850bf299e2a9e0ada339430b660761df
Author: Duncan Laurie <dlaurie at chromium.org>
Date:   Thu Dec 12 16:55:36 2013 -0800

    baytrail: Put devices in ACPI mode after setup
    
    Make sure reg_script is executed before the device is put into
    ACPI mode.
    
    BUG=chrome-os-partner:24380
    BRANCH=none
    TEST=build and boot rambi from eMMC in ACPI mode
    
    Change-Id: I4090babbfc7fb0f3be4da869386e998d87a513ba
    Signed-off-by: Duncan Laurie <dlaurie at chromium.org>
    Reviewed-on: https://chromium-review.googlesource.com/179896
    Reviewed-by: Aaron Durbin <adurbin at chromium.org>
    Signed-off-by: Aaron Durbin <adurbin at chromium.org>
---
 src/soc/intel/baytrail/emmc.c | 5 +++--
 src/soc/intel/baytrail/lpss.c | 3 +--
 2 files changed, 4 insertions(+), 4 deletions(-)

diff --git a/src/soc/intel/baytrail/emmc.c b/src/soc/intel/baytrail/emmc.c
index a724c4a..f88614b 100644
--- a/src/soc/intel/baytrail/emmc.c
+++ b/src/soc/intel/baytrail/emmc.c
@@ -53,10 +53,11 @@ static void emmc_init(device_t dev)
 {
 	struct soc_intel_baytrail_config *config = dev->chip_info;
 
-	if (config->scc_acpi_mode)
-		scc_enable_acpi_mode(dev, SCC_MMC_CTL, SCC_NVS_MMC);
 	printk(BIOS_DEBUG, "eMMC init\n");
 	reg_script_run_on_dev(dev, emmc_ops);
+
+	if (config->scc_acpi_mode)
+		scc_enable_acpi_mode(dev, SCC_MMC_CTL, SCC_NVS_MMC);
 }
 
 static struct device_operations device_ops = {
diff --git a/src/soc/intel/baytrail/lpss.c b/src/soc/intel/baytrail/lpss.c
index a543fd8..3ee648a 100644
--- a/src/soc/intel/baytrail/lpss.c
+++ b/src/soc/intel/baytrail/lpss.c
@@ -165,11 +165,10 @@ static void lpss_init(device_t dev)
 		return;
 	}
 	dev_enable_snoop_and_pm(dev, iosf_reg);
+	i2c_disable_resets(dev);
 
 	if (config->lpss_acpi_mode)
 		dev_enable_acpi_mode(dev, iosf_reg, nvs_index);
-
-	i2c_disable_resets(dev);
 }
 
 static struct device_operations device_ops = {



More information about the coreboot-gerrit mailing list