diff options
author | Philip Chen <philipchen@google.com> | 2019-06-12 18:14:28 -0700 |
---|---|---|
committer | Commit Bot <commit-bot@chromium.org> | 2019-06-14 01:26:48 +0000 |
commit | b30ae71143782737f739690ea200ddd1516a14c4 (patch) | |
tree | 1b70889230b9448e9ca82f8d8aaddaf577cefaca /board | |
parent | 6231bcc61127a4bc75d2fa2ec2e4ea58fb164207 (diff) | |
download | chrome-ec-b30ae71143782737f739690ea200ddd1516a14c4.tar.gz |
hatch: Make MST support optional for subboards
This CL cleans up the dependence on the MST support
in hatch baseboard.
Subboards can optionally define USB_PD_PORT_TCPC_MST when
MST support is needed.
BUG=b:133788856
BRANCH=none
TEST=make buildall
Change-Id: Ia83e31e68881ce419056716df4776e4be6786c45
Signed-off-by: Philip Chen <philipchen@google.com>
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/1657320
Reviewed-by: Paul Fagerburg <pfagerburg@chromium.org>
Reviewed-by: Tim Wawrzynczak <twawrzynczak@chromium.org>
Tested-by: Tim Wawrzynczak <twawrzynczak@chromium.org>
Tested-by: Philip Chen <philipchen@chromium.org>
Commit-Queue: Philip Chen <philipchen@chromium.org>
Diffstat (limited to 'board')
-rw-r--r-- | board/hatch/board.h | 2 | ||||
-rw-r--r-- | board/helios/board.h | 8 | ||||
-rw-r--r-- | board/helios/gpio.inc | 3 | ||||
-rw-r--r-- | board/kindred/board.h | 2 | ||||
-rw-r--r-- | board/kohaku/board.h | 8 | ||||
-rw-r--r-- | board/kohaku/gpio.inc | 1 |
6 files changed, 2 insertions, 22 deletions
diff --git a/board/hatch/board.h b/board/hatch/board.h index 06a7e7945c..356d2883d8 100644 --- a/board/hatch/board.h +++ b/board/hatch/board.h @@ -100,7 +100,7 @@ * one Type C port. This will need to be chagned to support 2 Type C ports * connected to the same MST chip. */ -#define USB_PD_PORT_TCPC 1 +#define USB_PD_PORT_TCPC_MST USB_PD_PORT_TCPC_1 /* * Macros for GPIO signals used in common code that don't match the diff --git a/board/helios/board.h b/board/helios/board.h index a7b98b9bb5..ff0b803a09 100644 --- a/board/helios/board.h +++ b/board/helios/board.h @@ -82,14 +82,6 @@ #define CONFIG_THROTTLE_AP #define CONFIG_STEINHART_HART_3V3_51K1_47K_4050B -/* MST */ -/* - * TDOD (b/124068003): This inherently assumes the MST chip is connected to only - * one Type C port. This will need to be chagned to support 2 Type C ports - * connected to the same MST chip. - */ -#define USB_PD_PORT_TCPC 1 - /* * Macros for GPIO signals used in common code that don't match the * schematic names. Signal names in gpio.inc match the schematic and are diff --git a/board/helios/gpio.inc b/board/helios/gpio.inc index f00cbacbc7..c6e9b8b911 100644 --- a/board/helios/gpio.inc +++ b/board/helios/gpio.inc @@ -70,9 +70,6 @@ GPIO(LED_3_L, PIN(C, 2), GPIO_OUT_HIGH) GPIO(EC_KB_BL_EN, PIN(8, 6), GPIO_OUT_LOW) /* Keyboard backlight */ GPIO(EN_PP5000_FAN, PIN(6, 1), GPIO_OUT_LOW) -/* TODO (b/133788856): this is a NC on Helios, but Hatch baseboard depends on this signal */ -GPIO(EN_MST, PIN(9, 6), GPIO_INPUT | GPIO_PULL_DOWN) - /* I2C pins - Alternate function below configures I2C module on these pins */ GPIO(I2C0_SCL, PIN(B, 5), GPIO_INPUT | GPIO_SEL_1P8V) /* EC_I2C_SENSOR_1V8_SCL */ diff --git a/board/kindred/board.h b/board/kindred/board.h index 06a7e7945c..356d2883d8 100644 --- a/board/kindred/board.h +++ b/board/kindred/board.h @@ -100,7 +100,7 @@ * one Type C port. This will need to be chagned to support 2 Type C ports * connected to the same MST chip. */ -#define USB_PD_PORT_TCPC 1 +#define USB_PD_PORT_TCPC_MST USB_PD_PORT_TCPC_1 /* * Macros for GPIO signals used in common code that don't match the diff --git a/board/kohaku/board.h b/board/kohaku/board.h index cbdd9f5e22..8e37e56a0a 100644 --- a/board/kohaku/board.h +++ b/board/kohaku/board.h @@ -101,14 +101,6 @@ #define CONFIG_THROTTLE_AP #define CONFIG_STEINHART_HART_3V3_51K1_47K_4050B -/* MST */ -/* - * TDOD (b/124068003): This inherently assumes the MST chip is connected to only - * one Type C port. This will need to be chagned to support 2 Type C ports - * connected to the same MST chip. - */ -#define USB_PD_PORT_TCPC 1 - /* * Macros for GPIO signals used in common code that don't match the * schematic names. Signal names in gpio.inc match the schematic and are diff --git a/board/kohaku/gpio.inc b/board/kohaku/gpio.inc index c5acffe69b..dd60ba89b7 100644 --- a/board/kohaku/gpio.inc +++ b/board/kohaku/gpio.inc @@ -71,7 +71,6 @@ GPIO(LED_1_L, PIN(C, 4), GPIO_OUT_HIGH) /* Yellow (hatch GPIO(LED_2_L, PIN(C, 3), GPIO_OUT_HIGH) /* White (hatch) */ GPIO(LED_3_L, PIN(C, 2), GPIO_OUT_HIGH) GPIO(EC_KB_BL_EN, PIN(8, 6), GPIO_OUT_LOW) /* Keyboard backlight */ -GPIO(EN_MST, PIN(9, 6), GPIO_OUT_LOW) GPIO(LID_ACCEL_INT_L, PIN(5, 0), GPIO_INPUT | /* Lid accelerometer */ GPIO_SEL_1P8V) |