diff options
author | Peter Marheine <pmarheine@chromium.org> | 2021-12-01 09:49:03 +1100 |
---|---|---|
committer | Commit Bot <commit-bot@chromium.org> | 2021-12-07 05:21:51 +0000 |
commit | 5f4a0027da4aabe9ebb347080f2f6bc998bf2cec (patch) | |
tree | f771880fc22b934925f63ae231066354cd2872bd | |
parent | 8cd4c1ba67da3095479eda6e96823029aa28fb01 (diff) | |
download | chrome-ec-5f4a0027da4aabe9ebb347080f2f6bc998bf2cec.tar.gz |
dooly: fix incorrect I2C data pin mappings
The data lines defined for the I2C buses associated with USB-C port 1 seem
to have been copied from port 0 and while the clock line pins were updated,
the data lines were not. This change corrects the data lines to refer to
the correct pins.
BUG=b:208485991
TEST=builds
BRANCH=none
Signed-off-by: Peter Marheine <pmarheine@chromium.org>
Change-Id: Ic093e9a4c305cef5e8b5661e8443adf90d1faa7b
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/3306767
Commit-Queue: Keith Short <keithshort@chromium.org>
Reviewed-by: Keith Short <keithshort@chromium.org>
Reviewed-by: Andrew McRae <amcrae@google.com>
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/3312812
Tested-by: Zick Wei <zick.wei@quanta.corp-partner.google.com>
Commit-Queue: Andrew McRae <amcrae@google.com>
-rw-r--r-- | board/dooly/board.c | 56 |
1 files changed, 49 insertions, 7 deletions
diff --git a/board/dooly/board.c b/board/dooly/board.c index 368dff7e2a..40deb6cb0f 100644 --- a/board/dooly/board.c +++ b/board/dooly/board.c @@ -533,13 +533,55 @@ const struct usb_mux usb_muxes[CONFIG_USB_PD_PORT_MAX_COUNT] = { /******************************************************************************/ /* I2C port map configuration */ const struct i2c_port_t i2c_ports[] = { - {"ina", I2C_PORT_INA, 400, GPIO_I2C0_SCL, GPIO_I2C0_SDA}, - {"ppc0", I2C_PORT_PPC0, 400, GPIO_I2C1_SCL, GPIO_I2C1_SDA}, - {"ppc1", I2C_PORT_PPC1, 400, GPIO_I2C2_SCL, GPIO_I2C1_SDA}, - {"tcpc0", I2C_PORT_TCPC0, 400, GPIO_I2C3_SCL, GPIO_I2C3_SDA}, - {"tcpc1", I2C_PORT_TCPC1, 400, GPIO_I2C4_SCL, GPIO_I2C3_SDA}, - {"power", I2C_PORT_POWER, 400, GPIO_I2C5_SCL, GPIO_I2C5_SDA}, - {"eeprom", I2C_PORT_EEPROM, 400, GPIO_I2C7_SCL, GPIO_I2C7_SDA}, + { + .name = "ina", + .port = I2C_PORT_INA, + .kbps = 400, + .scl = GPIO_I2C0_SCL, + .sda = GPIO_I2C0_SDA + }, + { + .name = "ppc0", + .port = I2C_PORT_PPC0, + .kbps = 400, + .scl = GPIO_I2C1_SCL, + .sda = GPIO_I2C1_SDA + }, + { + .name = "ppc1", + .port = I2C_PORT_PPC1, + .kbps = 400, + .scl = GPIO_I2C2_SCL, + .sda = GPIO_I2C2_SDA + }, + { + .name = "tcpc0", + .port = I2C_PORT_TCPC0, + .kbps = 400, + .scl = GPIO_I2C3_SCL, + .sda = GPIO_I2C3_SDA + }, + { + .name = "tcpc1", + .port = I2C_PORT_TCPC1, + .kbps = 400, + .scl = GPIO_I2C4_SCL, + .sda = GPIO_I2C4_SDA + }, + { + .name = "power", + .port = I2C_PORT_POWER, + .kbps = 400, + .scl = GPIO_I2C5_SCL, + .sda = GPIO_I2C5_SDA + }, + { + .name = "eeprom", + .port = I2C_PORT_EEPROM, + .kbps = 400, + .scl = GPIO_I2C7_SCL, + .sda = GPIO_I2C7_SDA + }, }; const unsigned int i2c_ports_used = ARRAY_SIZE(i2c_ports); |