[coreboot-gerrit] Change in coreboot[master]: google/oak: Configure SD card detect pin with a pull-up

[Patrick Georgi (Code Review)]

Patrick Georgi has uploaded a new change for review. ( https://review.coreboot.org/19432 )

Change subject: google/oak: Configure SD card detect pin with a pull-up
......................................................................

google/oak: Configure SD card detect pin with a pull-up

SD card detect pins should normally have a pull-up. It seems that for
micro-SD cards this doesn't really matter all that much, but for the
full-size slots we have on some Oak-derivatives (like Hana) it does.

BRANCH=oak
BUG=b:35854317
TEST=Booted Hana, confirmed that card detect no longer seemed stuck-on.
Booted Elm and confirmed that SD card behavior didn't change.

Change-Id: I9b20e0f6fe310e724d191e36ca0a81ab4fe5f593
Signed-off-by: Patrick Georgi <pgeorgi at chromium.org>
Original-Commit-Id: c2781eeef50f52c6f02ee9344274ddf4dcb0a946
Original-Change-Id: I428ac92efb07f94265673b04e0e0dd452649b9fd
Original-Signed-off-by: Julius Werner <jwerner at chromium.org>
Original-Reviewed-on: https://chromium-review.googlesource.com/452861
Original-Reviewed-by: Nicolas Boichat <drinkcat at chromium.org>
---
M src/mainboard/google/oak/mainboard.c
1 file changed, 1 insertion(+), 1 deletion(-)


  git pull ssh://review.coreboot.org:29418/coreboot refs/changes/32/19432/1

diff --git a/src/mainboard/google/oak/mainboard.c b/src/mainboard/google/oak/mainboard.c
index 7cb6a68..8c8d715 100644
--- a/src/mainboard/google/oak/mainboard.c
+++ b/src/mainboard/google/oak/mainboard.c
@@ -256,7 +256,7 @@
 		mt6391_gpio_output(MT6391_KP_ROW2, 1);
 
 	/* Config SD card detection pin */
-	gpio_input(PAD_EINT1); /* SD_DET */
+	gpio_input_pullup(PAD_EINT1); /* SD_DET */
 
 	configure_audio();
 

-- 
To view, visit https://review.coreboot.org/19432
To unsubscribe, visit https://review.coreboot.org/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: I9b20e0f6fe310e724d191e36ca0a81ab4fe5f593
Gerrit-PatchSet: 1
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Owner: Patrick Georgi <pgeorgi at google.com>
Gerrit-Reviewer: Julius Werner <jwerner at chromium.org>