Change in coreboot[master]: mb/google/volteer: Pull up GPP_D16 instead of driving it

[Nick Vaccaro (Code Review)]

Nick Vaccaro has submitted this change. ( https://review.coreboot.org/c/coreboot/+/44116 )

Change subject: mb/google/volteer: Pull up GPP_D16 instead of driving it
......................................................................

mb/google/volteer: Pull up GPP_D16 instead of driving it

The latest realtek RTS5261 SD daughterboard exposes the PRSNT# pin to
GPP_D16 but there is a RTS5261 requirement to pull up this pin and not
drive it at power on. We can meet this requirement without breaking
other boards by changing GPP_D16 to be a no-connect with an internal
pull up. Other boards use this signal as an enable input, so changing
this to pull up is OK.

BUG=b:162722965
TEST=Verified RTS5261 and GL9755 daughterboards enumerate on PCI and
    can read SD cards.

Change-Id: I096d76ec12b7c3afaf02e621fd301b6704913d5d
Signed-off-by: Caveh Jalali 
Reviewed-on: https://review.coreboot.org/c/coreboot/+/44116
Tested-by: build bot (Jenkins) 
Reviewed-by: Tim Wawrzynczak 
---
M src/mainboard/google/volteer/variants/volteer/gpio.c
M src/mainboard/google/volteer/variants/volteer2/gpio.c
2 files changed, 2 insertions(+), 2 deletions(-)

Approvals:
  build bot (Jenkins): Verified
  Tim Wawrzynczak: Looks good to me, approved

diff --git a/src/mainboard/google/volteer/variants/volteer/gpio.c b/src/mainboard/google/volteer/variants/volteer/gpio.c
index 9c36138..8bbc447 100644
--- a/src/mainboard/google/volteer/variants/volteer/gpio.c
+++ b/src/mainboard/google/volteer/variants/volteer/gpio.c
@@ -91,7 +91,7 @@
    /* D14 : ISH_UART0_TXD ==> UART_ISH_TX_DEBUG_RX */
    PAD_CFG_NF(GPP_D14, NONE, DEEP, NF1),
    /* D16 : ISH_UART0_CTS# ==> EN_PP3300_SD */
-	PAD_CFG_GPO(GPP_D16, 1, DEEP),
+	PAD_NC(GPP_D16, UP_20K),
    /* D17 : ISH_GP4 ==> EN_FCAM_PWR */
    PAD_CFG_GPO(GPP_D17, 1, DEEP),
    /* D18 : ISH_GP5 ==> FCAM_SNRPWR_EN */
diff --git a/src/mainboard/google/volteer/variants/volteer2/gpio.c b/src/mainboard/google/volteer/variants/volteer2/gpio.c
index 2b99e52..96d940a 100644
--- a/src/mainboard/google/volteer/variants/volteer2/gpio.c
+++ b/src/mainboard/google/volteer/variants/volteer2/gpio.c
@@ -91,7 +91,7 @@
    /* D14 : ISH_UART0_TXD ==> UART_ISH_TX_DEBUG_RX */
    PAD_CFG_NF(GPP_D14, NONE, DEEP, NF1),
    /* D16 : ISH_UART0_CTS# ==> EN_PP3300_SD */
-	PAD_CFG_GPO(GPP_D16, 1, DEEP),
+	PAD_NC(GPP_D16, UP_20K),
    /* D17 : ISH_GP4 ==> EN_FCAM_PWR */
    PAD_CFG_GPO(GPP_D17, 1, DEEP),
    /* D18 : ISH_GP5 ==> FCAM_SNRPWR_EN */