From b7d79cddf0287a2e925d2003ef8d0cde6bbbae2a Mon Sep 17 00:00:00 2001 From: Aaron Durbin Date: Mon, 22 Jan 2018 21:31:48 -0700 Subject: 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 Reviewed-on: https://review.coreboot.org/23372 Tested-by: build bot (Jenkins) Reviewed-by: Martin Roth Reviewed-by: Furquan Shaikh Reviewed-by: Subrata Banik --- src/soc/intel/common/block/i2c/i2c.c | 130 ++--------------------------------- 1 file changed, 4 insertions(+), 126 deletions(-) (limited to 'src/soc') 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 -#include #include #include #include @@ -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[] = { -- cgit v1.2.3