[coreboot-gerrit] New patch to review for coreboot: 05967e9 rambi: configure SD card signals
Aaron Durbin (adurbin@google.com)
gerrit at coreboot.org
Tue Jan 28 03:57:00 CET 2014
Aaron Durbin (adurbin at google.com) just uploaded a new patch set to gerrit, which you can find at http://review.coreboot.org/4990
-gerrit
commit 05967e99ffc07c60c293c2cc0c71725ca3798ea6
Author: Aaron Durbin <adurbin at chromium.org>
Date: Thu Dec 5 12:40:08 2013 -0600
rambi: configure SD card signals
Rambi 1.5 boards use the native SD card controller on baytrail.
Therefore, enable those signals. The CLK, D*, and CMD pins use
2K pulls as these were shown to not exhibit any errors when
doing reads or writes to a DDR50 sd card.
Note that if a servo is connected on needs to enable the
sd_vref_sel rail to pp1800 as this causes issues with card
detect if it is not set to pp1800.
BUG=chrome-os-partner:24312
BRANCH=None
TEST=Built and booted. Tested sd card read and write works in kernel.
Also noted that write protect detection works as well.
Change-Id: I520e2808acbd8494534fcb710411dbc0e12fc874
Signed-off-by: Aaron Durbin <adurbin at chromium.org>
Reviewed-on: https://chromium-review.googlesource.com/178961
Reviewed-by: Shawn Nematbakhsh <shawnn at chromium.org>
---
src/mainboard/google/rambi/gpio.c | 20 ++++++++++----------
1 file changed, 10 insertions(+), 10 deletions(-)
diff --git a/src/mainboard/google/rambi/gpio.c b/src/mainboard/google/rambi/gpio.c
index 76002cf..68a5970 100644
--- a/src/mainboard/google/rambi/gpio.c
+++ b/src/mainboard/google/rambi/gpio.c
@@ -63,7 +63,7 @@ static const struct soc_gpio_map gpscore_gpio_map[] = {
GPIO_FUNC1, /* S0-SC004 - PCIE_CLKREQ_WLAN# */
GPIO_NC, /* S0-SC005 - PCIE_CLKREQ_LAN# (NC) */
GPIO_NC, /* S0-SC006 - PCIE_CLKREQ3# (NC) */
- GPIO_NC, /* S0-SC007 - SD3_WP (NC) */
+ GPIO_FUNC(2, PULL_DISABLE, 10K), /* S0-SC007 - SD3_WP external pull */
GPIO_NC, /* S0-SC008 - ACZ_RST# (NC) */
GPIO_NC, /* S0-SC009 - ACZ_SYNC (NC) */
GPIO_NC, /* S0-SC010 - ACZ_BCLK (NC) */
@@ -89,15 +89,15 @@ static const struct soc_gpio_map gpscore_gpio_map[] = {
GPIO_NC, /* S0-SC030 - NC */
GPIO_NC, /* S0-SC031 - NC */
GPIO_NC, /* S0-SC032 - NC */
- GPIO_NC, /* S0-SC033 - SD3_CLK (NC) */
- GPIO_NC, /* S0-SC034 - SD3_D0 (NC) */
- GPIO_NC, /* S0-SC035 - SD3_D1 (NC) */
- GPIO_NC, /* S0-SC036 - SD3_D2 (NC) */
- GPIO_NC, /* S0-SC037 - SD3_D3 (NC) */
- GPIO_NC, /* S0-SC038 - SD3_CD# (NC) */
- GPIO_NC, /* S0-SC039 - SD3_CMD (NC) */
- GPIO_NC, /* S0-SC040 - SDMMC3_1P8_EN (NC) */
- GPIO_NC, /* S0-SC041 - SDIO3_PWR_EN# (NC)*/
+ GPIO_FUNC(1, PULL_DOWN, 2K), /* S0-SC033 - SD3_CLK */
+ GPIO_FUNC(1, PULL_UP, 2K), /* S0-SC034 - SD3_D0 */
+ GPIO_FUNC(1, PULL_UP, 2K), /* S0-SC035 - SD3_D1 */
+ GPIO_FUNC(1, PULL_UP, 2K), /* S0-SC036 - SD3_D2 */
+ GPIO_FUNC(1, PULL_UP, 2K), /* S0-SC037 - SD3_D3 */
+ GPIO_FUNC(1, PULL_UP, 20K), /* S0-SC038 - SD3_CD# */
+ GPIO_FUNC(1, PULL_UP, 2K), /* S0-SC039 - SD3_CMD */
+ GPIO_NC, /* S0-SC040 - SDMMC3_1P8_EN - TP3 */
+ GPIO_FUNC(1, PULL_UP, 20K), /* S0-SC041 - SDIO3_PWR_EN# */
GPIO_FUNC1, /* S0-SC042 - LPC_LAD0 */
GPIO_FUNC1, /* S0-SC043 - LPC-LAD1 */
GPIO_FUNC1, /* S0-SC044 - LPC_LAD2 */
More information about the coreboot-gerrit
mailing list