summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorTarun Tuli <taruntuli@google.com>2023-06-09 20:28:43 +0000
committerEric Lai <eric_lai@quanta.corp-partner.google.com>2023-06-13 15:55:16 +0000
commit6b89089b0ccf232f8e2d33793edd606592c89cf6 (patch)
tree28e051581390ab2f97ea5046301535611cebfeb4
parentaa1efece74aa9707ece96a76cf24c8068d604525 (diff)
downloadcoreboot-6b89089b0ccf232f8e2d33793edd606592c89cf6.tar.gz
coreboot-6b89089b0ccf232f8e2d33793edd606592c89cf6.tar.bz2
coreboot-6b89089b0ccf232f8e2d33793edd606592c89cf6.zip
mb/google/brya/variants/hades: Set WP signal to GPP_E12
Move the WP signal to GPP_E12 from the current GPP_E15 to match the design. BUG=b:285084125 TEST=WP signal reports as we expect Signed-off-by: Tarun Tuli <taruntuli@google.com> Change-Id: I8772173fcdcabf78b0c7d605cd495ebe04b63242 Reviewed-on: https://review.coreboot.org/c/coreboot/+/75536 Tested-by: build bot (Jenkins) <no-reply@coreboot.org> Reviewed-by: Eric Lai <eric_lai@quanta.corp-partner.google.com>
-rw-r--r--src/mainboard/google/brya/variants/baseboard/hades/include/baseboard/gpio.h2
-rw-r--r--src/mainboard/google/brya/variants/hades/gpio.c6
2 files changed, 3 insertions, 5 deletions
diff --git a/src/mainboard/google/brya/variants/baseboard/hades/include/baseboard/gpio.h b/src/mainboard/google/brya/variants/baseboard/hades/include/baseboard/gpio.h
index 6da54bf0374b..b974aa576a32 100644
--- a/src/mainboard/google/brya/variants/baseboard/hades/include/baseboard/gpio.h
+++ b/src/mainboard/google/brya/variants/baseboard/hades/include/baseboard/gpio.h
@@ -12,7 +12,7 @@
/* EC wake is EC_PCH_INT which is routed to GPP_F17 pin */
#define GPE_EC_WAKE GPE0_DW2_17
/* WP signal to PCH */
-#define GPIO_PCH_WP GPP_E15
+#define GPIO_PCH_WP GPP_E12
/* EC in RW or RO */
#define GPIO_EC_IN_RW GPP_F18
/* Used to gate SoC's SLP_S0# signal */
diff --git a/src/mainboard/google/brya/variants/hades/gpio.c b/src/mainboard/google/brya/variants/hades/gpio.c
index 1c210e82a1fc..f826a65bffae 100644
--- a/src/mainboard/google/brya/variants/hades/gpio.c
+++ b/src/mainboard/google/brya/variants/hades/gpio.c
@@ -403,17 +403,15 @@ static const struct pad_config early_gpio_table[] = {
PAD_CFG_GPO(GPP_D2, 1, DEEP),
/* GPP_D11 : [] ==> EN_PP3300_SSD */
PAD_CFG_GPO(GPP_D11, 1, PLTRST),
+ /* GPP_E12 : [] ==> PCH_WP_OD */
+ PAD_CFG_GPI_GPIO_DRIVER_LOCK(GPP_E12, NONE, LOCK_CONFIG),
/* GPP_E13 : [] ==> MEM_CH_SEL */
PAD_CFG_GPI(GPP_E13, NONE, DEEP),
- /* GPP_E15 : [] ==> PCH_WP_OD */
- PAD_CFG_GPI_GPIO_DRIVER(GPP_E15, NONE, DEEP),
/* GPP_E16 : [] ==> WWAN_RST_L
* To meet timing constrains - drive reset low.
* Deasserted in ramstage.
*/
PAD_CFG_GPO(GPP_E16, 0, DEEP),
- /* GPP_E15 : [] ==> PCH_WP_OD */
- PAD_CFG_GPI_GPIO_DRIVER_LOCK(GPP_E15, NONE, LOCK_CONFIG),
/* GPP_E18 : [] ==> EN_PP1800_GPU_X */
PAD_CFG_GPO(GPP_E18, 0, PLTRST),
/* GPP_F18 : [] ==> EC_IN_RW_OD */