[coreboot-gerrit] Change in coreboot[master]: mb/google/dedede: Update EC_MKBP_INT_L configuration

[Patrick Georgi (Code Review)]

Patrick Georgi has submitted this change. ( https://review.coreboot.org/c/coreboot/+/39796 )

Change subject: mb/google/dedede: Update EC_MKBP_INT_L configuration
......................................................................

mb/google/dedede: Update EC_MKBP_INT_L configuration

The concerned GPIO is configured as an open drain at the Embedded
Controller side without an external pull-up. This causes leakage in the
PP3300_A rail. So configure the GPIO to have a weak internal pull-up at
the SoC side.

BUG=b:151680590
TEST=Build the mainboard. Ensure that there are no leakages in the
PP3300_A rail.

Change-Id: I5553cf40adb92edc0fecab5c875ec8d72063ba7b
Signed-off-by: Karthikeyan Ramasubramanian 
Reviewed-on: https://review.coreboot.org/c/coreboot/+/39796
Tested-by: build bot (Jenkins) 
Reviewed-by: Justin TerAvest 
Reviewed-by: Furquan Shaikh 
---
M src/mainboard/google/dedede/variants/baseboard/gpio.c
1 file changed, 1 insertion(+), 1 deletion(-)

Approvals:
  build bot (Jenkins): Verified
  Furquan Shaikh: Looks good to me, approved
  Justin TerAvest: Looks good to me, approved

diff --git a/src/mainboard/google/dedede/variants/baseboard/gpio.c b/src/mainboard/google/dedede/variants/baseboard/gpio.c
index bdc3de4..a012737 100644
--- a/src/mainboard/google/dedede/variants/baseboard/gpio.c
+++ b/src/mainboard/google/dedede/variants/baseboard/gpio.c
@@ -127,7 +127,7 @@
    /* C14 : EC_IN_RW_OD */
    PAD_CFG_GPI(GPP_C14, NONE, DEEP),
    /* C15 : EC_AP_MKBP_INT_L */
-	PAD_CFG_GPI_APIC(GPP_C15, NONE, PLTRST, LEVEL, INVERT),
+	PAD_CFG_GPI_APIC(GPP_C15, UP_20K, PLTRST, LEVEL, INVERT),
    /* C16 : AP_I2C_TRACKPAD_SDA_3V3 */
    PAD_CFG_NF(GPP_C16, NONE, DEEP, NF1),
    /* C17 : AP_I2C_TRACKPAD_SCL_3V3 */