summaryrefslogtreecommitdiff
path: root/board/ezkinil
diff options
context:
space:
mode:
authorEdward Hill <ecgh@chromium.org>2020-06-02 18:28:26 -0600
committerCommit Bot <commit-bot@chromium.org>2020-06-03 05:39:13 +0000
commit5124d453200ce13dfd8badfcd2c56e57334df2df (patch)
treeb397b594cf13c79c77beeebf7bb7ec5b2c03bb86 /board/ezkinil
parentce804f34ccb7c65fc4711c01eac63ceb8c008e26 (diff)
downloadchrome-ec-5124d453200ce13dfd8badfcd2c56e57334df2df.tar.gz
Zork: Switch PS8743 from IOEX_USB_C1_DATA_EN to I2C
Align all Zork devices on using I2C control of PS8743 mux mode, and stop using IOEX_USB_C1_DATA_EN signal/pin. This matches what we were already doing for Dalboz, and gives power savings in both S0 and S3 when nothing is plugged in and mux is set to none. BUG=b:157951317 BRANCH=none TEST=external display and i2ctrace on ezkinil Signed-off-by: Edward Hill <ecgh@chromium.org> Change-Id: Id1c8b172a99bb25bed8b57e90686d933447432f2 Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/2227589 Reviewed-by: Denis Brockus <dbrockus@chromium.org> Commit-Queue: Denis Brockus <dbrockus@chromium.org>
Diffstat (limited to 'board/ezkinil')
-rw-r--r--board/ezkinil/board.c25
1 files changed, 4 insertions, 21 deletions
diff --git a/board/ezkinil/board.c b/board/ezkinil/board.c
index 330322eef7..d3e5e15e5e 100644
--- a/board/ezkinil/board.c
+++ b/board/ezkinil/board.c
@@ -269,31 +269,14 @@ static int board_tusb544_mux_set(const struct usb_mux *me,
static int board_ps8743_mux_set(const struct usb_mux *me,
mux_state_t mux_state)
{
- int rv = EC_SUCCESS;
- int reg = 0;
-
- rv = ps8743_read(me, PS8743_REG_MODE, &reg);
- if (rv)
- return rv;
-
- /* Disable FLIP pin, enable I2C control. */
- reg |= PS8743_MODE_FLIP_REG_CONTROL;
- /* Disable CE_DP pin, enable I2C control. */
- reg |= PS8743_MODE_DP_REG_CONTROL;
-
- /* DP specific config */
- if (mux_state & USB_PD_MUX_DP_ENABLED) {
+ if (mux_state & USB_PD_MUX_DP_ENABLED)
/* Enable IN_HPD on the DB */
ioex_set_level(IOEX_USB_C1_HPD_IN_DB, 1);
- /* Disable USB mode on DB */
- ioex_set_level(IOEX_USB_C1_DATA_EN, 0);
- } else {
+ else
/* Disable IN_HPD on the DB */
ioex_set_level(IOEX_USB_C1_HPD_IN_DB, 0);
- /* Enable USB mode on DB */
- ioex_set_level(IOEX_USB_C1_DATA_EN, 1);
- }
- return ps8743_write(me, PS8743_REG_MODE, reg);
+
+ return EC_SUCCESS;
}
const struct usb_mux usbc1_tusb544 = {