summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorSubrata Banik <subrata.banik@intel.com>2018-09-27 19:27:39 +0530
committerSubrata Banik <subrata.banik@intel.com>2018-10-03 04:50:37 +0000
commit7d8c0c2c588fea989691d95bbbd6bcf9137cc675 (patch)
treeafbcaeaa79679db09f2809daf98ce0f94d612c30
parentcac2217c938dd615080b52405fc4b348399d4cb5 (diff)
downloadcoreboot-7d8c0c2c588fea989691d95bbbd6bcf9137cc675.tar.xz
soc/intel/commom/block/i2c: Make I2C controller out of reset
This patch ensures I2C controllers are out of reset without any assumptions. BUG=b:116191230 BRANCH=none TEST=Dump MMIO offset 0x204 to check if I2C host controller is NOT at reset (by reading Bit 0-1 as 3) Change-Id: I4b335a834333e01cfa2d802e4aad0735d0212dcc Signed-off-by: Subrata Banik <subrata.banik@intel.com> Reviewed-on: https://review.coreboot.org/28762 Tested-by: build bot (Jenkins) <no-reply@coreboot.org> Reviewed-by: Furquan Shaikh <furquan@google.com>
-rw-r--r--src/soc/intel/common/block/i2c/i2c.c27
1 files changed, 25 insertions, 2 deletions
diff --git a/src/soc/intel/common/block/i2c/i2c.c b/src/soc/intel/common/block/i2c/i2c.c
index 11bd018c8b..c6c511575d 100644
--- a/src/soc/intel/common/block/i2c/i2c.c
+++ b/src/soc/intel/common/block/i2c/i2c.c
@@ -1,7 +1,7 @@
/*
* This file is part of the coreboot project.
*
- * Copyright 2017 Intel Corporation.
+ * Copyright 2017-2018 Intel Corporation.
*
* This program is free software; you can redistribute it and/or modify
* it under the terms of the GNU General Public License as published by
@@ -146,6 +146,29 @@ uintptr_t dw_i2c_base_address(unsigned int bus)
return (uintptr_t)NULL;
}
+/*
+ * This function ensures that the device is actually out of reset and
+ * its ready for initialization sequence.
+ */
+static void dw_i2c_device_init(struct device *dev)
+{
+ uintptr_t base_address;
+ int bus = dw_i2c_soc_dev_to_bus(dev);
+
+ if (bus < 0)
+ return;
+
+ base_address = dw_i2c_base_address(bus);
+ if (!base_address)
+ return;
+
+ /* Take device out of reset if its not done before */
+ if (lpss_is_controller_in_reset(base_address))
+ lpss_reset_release(base_address);
+
+ dw_i2c_dev_init(dev);
+}
+
static struct device_operations i2c_dev_ops = {
.read_resources = &pci_dev_read_resources,
.set_resources = &pci_dev_set_resources,
@@ -153,7 +176,7 @@ static struct device_operations i2c_dev_ops = {
.scan_bus = &scan_smbus,
.ops_i2c_bus = &dw_i2c_bus_ops,
.ops_pci = &pci_dev_ops_pci,
- .init = &dw_i2c_dev_init,
+ .init = &dw_i2c_device_init,
.acpi_fill_ssdt_generator = &dw_i2c_acpi_fill_ssdt,
};