diff options
author | Aaron Durbin <adurbin@chromium.org> | 2018-01-22 21:31:48 -0700 |
---|---|---|
committer | Aaron Durbin <adurbin@chromium.org> | 2018-01-24 05:03:10 +0000 |
commit | b7d79cddf0287a2e925d2003ef8d0cde6bbbae2a (patch) | |
tree | d5c79a089bd5ea0d0d5fdc2830e93f8bc3d07108 /src/soc/intel/common/block | |
parent | 9aee8194c46f31eeda6ee7419e40cad2de47187e (diff) | |
download | coreboot-b7d79cddf0287a2e925d2003ef8d0cde6bbbae2a.tar.xz |
drives/i2c/designware: incorporate device_operations support
In ramstage the device_operations are needed for the i2c designware
host controller. Move the intel/common/block/i2c implementation
into the generic driver so other platforms can take advantage of it.
BUG=b:72121803
Change-Id: Id249933fadcc016bfba00e7a6d65f56dfc220724
Signed-off-by: Aaron Durbin <adurbin@chromium.org>
Reviewed-on: https://review.coreboot.org/23372
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Reviewed-by: Martin Roth <martinroth@google.com>
Reviewed-by: Furquan Shaikh <furquan@google.com>
Reviewed-by: Subrata Banik <subrata.banik@intel.com>
Diffstat (limited to 'src/soc/intel/common/block')
-rw-r--r-- | src/soc/intel/common/block/i2c/i2c.c | 130 |
1 files changed, 4 insertions, 126 deletions
diff --git a/src/soc/intel/common/block/i2c/i2c.c b/src/soc/intel/common/block/i2c/i2c.c index 4b2b4eb5a8..319597a5bc 100644 --- a/src/soc/intel/common/block/i2c/i2c.c +++ b/src/soc/intel/common/block/i2c/i2c.c @@ -13,8 +13,6 @@ * GNU General Public License for more details. */ -#include <arch/acpigen.h> -#include <device/i2c_bus.h> #include <device/pci.h> #include <device/pci_def.h> #include <device/pci_ids.h> @@ -45,141 +43,21 @@ uintptr_t dw_i2c_base_address(unsigned int bus) return (uintptr_t)NULL; } -static int lpss_i2c_dev_to_bus(struct device *dev) +int dw_i2c_soc_dev_to_bus(struct device *dev) { pci_devfn_t devfn = dev->path.pci.devfn; return dw_i2c_soc_devfn_to_bus(devfn); } -/* - * Write ACPI object to describe speed configuration. - * - * ACPI Object: Name ("xxxx", Package () { scl_lcnt, scl_hcnt, sda_hold } - * - * SSCN: I2C_SPEED_STANDARD - * FMCN: I2C_SPEED_FAST - * FPCN: I2C_SPEED_FAST_PLUS - * HSCN: I2C_SPEED_HIGH - */ -static void lpss_i2c_acpi_write_speed_config( - const struct dw_i2c_speed_config *config) -{ - if (!config) - return; - if (!config->scl_lcnt && !config->scl_hcnt && !config->sda_hold) - return; - - if (config->speed >= I2C_SPEED_HIGH) - acpigen_write_name("HSCN"); - else if (config->speed >= I2C_SPEED_FAST_PLUS) - acpigen_write_name("FPCN"); - else if (config->speed >= I2C_SPEED_FAST) - acpigen_write_name("FMCN"); - else - acpigen_write_name("SSCN"); - - /* Package () { scl_lcnt, scl_hcnt, sda_hold } */ - acpigen_write_package(3); - acpigen_write_word(config->scl_hcnt); - acpigen_write_word(config->scl_lcnt); - acpigen_write_dword(config->sda_hold); - acpigen_pop_len(); -} - -/* - * The device should already be enabled and out of reset, - * either from early init in coreboot or SiliconInit in FSP. - */ -static void lpss_i2c_dev_init(struct device *dev) -{ - const struct dw_i2c_bus_config *config; - int bus = lpss_i2c_dev_to_bus(dev); - - if (bus < 0) - return; - - config = dw_i2c_get_soc_cfg(bus, dev); - - if (!config) - return; - - dw_i2c_init(bus, config); -} - -/* - * Generate I2C timing information into the SSDT for the OS driver to consume, - * optionally applying override values provided by the caller. - */ -static void lpss_i2c_acpi_fill_ssdt(struct device *dev) -{ - const struct dw_i2c_bus_config *bcfg; - uintptr_t dw_i2c_addr; - struct dw_i2c_speed_config sgen; - enum i2c_speed speeds[DW_I2C_SPEED_CONFIG_COUNT] = { - I2C_SPEED_STANDARD, - I2C_SPEED_FAST, - I2C_SPEED_FAST_PLUS, - I2C_SPEED_HIGH, - }; - int i, bus; - const char *path; - - if (!dev->enabled) - return; - - bus = lpss_i2c_dev_to_bus(dev); - - if (bus < 0) - return; - - bcfg = dw_i2c_get_soc_cfg(bus, dev); - - if (!bcfg) - return; - - dw_i2c_addr = dw_i2c_base_address(bus); - if (!dw_i2c_addr) - return; - - path = acpi_device_path(dev); - if (!path) - return; - - acpigen_write_scope(path); - - /* Report timing values for the OS driver */ - for (i = 0; i < DW_I2C_SPEED_CONFIG_COUNT; i++) { - /* Generate speed config. */ - if (dw_i2c_gen_speed_config(dw_i2c_addr, speeds[i], bcfg, - &sgen) < 0) - continue; - - /* Generate ACPI based on selected speed config */ - lpss_i2c_acpi_write_speed_config(&sgen); - } - - acpigen_pop_len(); -} - -static int lpss_i2c_dev_transfer(struct device *dev, - const struct i2c_msg *msg, size_t count) -{ - return dw_i2c_transfer(lpss_i2c_dev_to_bus(dev), msg, count); -} - -static const struct i2c_bus_operations i2c_bus_ops = { - .transfer = lpss_i2c_dev_transfer, -}; - static struct device_operations i2c_dev_ops = { .read_resources = &pci_dev_read_resources, .set_resources = &pci_dev_set_resources, .enable_resources = &pci_dev_enable_resources, .scan_bus = &scan_smbus, - .ops_i2c_bus = &i2c_bus_ops, + .ops_i2c_bus = &dw_i2c_bus_ops, .ops_pci = &pci_dev_ops_pci, - .init = &lpss_i2c_dev_init, - .acpi_fill_ssdt_generator = &lpss_i2c_acpi_fill_ssdt, + .init = &dw_i2c_dev_init, + .acpi_fill_ssdt_generator = &dw_i2c_acpi_fill_ssdt, }; static const unsigned short pci_device_ids[] = { |