diff options
| author | Kyösti Mälkki <kyosti.malkki@gmail.com> | 2020-01-03 10:55:59 +0200 |
|---|---|---|
| committer | Kyösti Mälkki <kyosti.malkki@gmail.com> | 2020-01-04 22:08:38 +0000 |
| commit | 895fb4b36156c18ddec22157020c6da232927f68 (patch) | |
| tree | 3bdde86c18e2c1e5731bc726339e71db12858f0a /src/device/smbus_ops.c | |
| parent | 6118cda8589272b89a1db12feb001a77591067f9 (diff) | |
| download | coreboot-895fb4b36156c18ddec22157020c6da232927f68.tar.gz coreboot-895fb4b36156c18ddec22157020c6da232927f68.tar.bz2 coreboot-895fb4b36156c18ddec22157020c6da232927f68.zip | |
device/smbus: Drop unused smbus_set_link()
I expect it to be easier to just remodel the support for i2c
multiplexers instead. Besides, there was no proper bounds for
pbus_num when accessing pbus_a[].
Change-Id: I17f33b308c01e48bc03b142550535c32862442ac
Signed-off-by: Kyösti Mälkki <kyosti.malkki@gmail.com>
Reviewed-on: https://review.coreboot.org/c/coreboot/+/38161
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Reviewed-by: Paul Menzel <paulepanter@users.sourceforge.net>
Reviewed-by: Nico Huber <nico.h@gmx.de>
Diffstat (limited to 'src/device/smbus_ops.c')
| -rw-r--r-- | src/device/smbus_ops.c | 37 |
1 files changed, 0 insertions, 37 deletions
diff --git a/src/device/smbus_ops.c b/src/device/smbus_ops.c index 13e6cadd55e9..3b7a69eabe33 100644 --- a/src/device/smbus_ops.c +++ b/src/device/smbus_ops.c @@ -27,43 +27,6 @@ struct bus *get_pbus_smbus(struct device *dev) return pbus; } -/* - * Multi-level I2C MUX? May need to find the first I2C device and then set link - * down to current dev. - * - * 1 store get_pbus_smbus list link - * 2 reverse the link and call set link. - * - * @param dev TODO. - */ -int smbus_set_link(struct device *dev) -{ - struct bus *pbus_a[4]; // 4 level mux only. Enough? - struct bus *pbus = dev->bus; - int pbus_num = 0; - int i; - - while (pbus && pbus->dev && (pbus->dev->path.type == DEVICE_PATH_I2C)) { - pbus_a[pbus_num++] = pbus; - pbus = pbus->dev->bus; - } - - // printk(BIOS_INFO, "smbus_set_link: "); - for (i = pbus_num - 1; i >= 0; i--) { - // printk(BIOS_INFO, " %s[%d] -> ", dev_path(pbus_a[i]->dev), - // pbus_a[i]->link); - if (ops_smbus_bus(get_pbus_smbus(pbus_a[i]->dev))) { - if (pbus_a[i]->dev->ops - && pbus_a[i]->dev->ops->set_link) - pbus_a[i]->dev->ops->set_link(pbus_a[i]->dev, - pbus_a[i]->link_num); - } - } - // printk(BIOS_INFO, " %s\n", dev_path(dev)); - - return pbus_num; -} - #define CHECK_PRESENCE(x) \ if (!ops_smbus_bus(get_pbus_smbus(dev))->x) { \ printk(BIOS_ERR, "%s missing " #x "\n", \ |