diff options
author | Vadim Bendebury <vbendeb@chromium.org> | 2016-05-12 16:54:00 +0800 |
---|---|---|
committer | Martin Roth <martinroth@google.com> | 2016-05-18 20:21:32 +0200 |
commit | ad6ee021466c7b670091de167fd5610dad419a54 (patch) | |
tree | e8231573d1660d2e4c19017a72c4f7dd33b7e81b /src/mainboard/google/gru/mainboard.c | |
parent | a9cd4a2f117f03abb4e4433b040c2225242fd35f (diff) | |
download | coreboot-ad6ee021466c7b670091de167fd5610dad419a54.tar.xz |
rk3399: set proper configuration of SDMMC interface
For proper interface operation the drive strength on all pins is set
to 8 mA and all pull ups/pull downs disabled, this matches the current
kernel configuration.
BRANCH=none
BUG=chrome-os-partner:53257
TEST=it is possible to boot Chrome OS on Gru from various micro SD
cards which were failing to boot before.
Change-Id: Ie43e52a52cd0513d48d0ecc8ac02fbb100baf9a4
Signed-off-by: Martin Roth <martinroth@google.com>
Original-Commit-Id: 6bb0549ed728ac3c5faab6cbe16e2487400e67ed
Original-Change-Id: I5180537d3ceb74a9a2f7b3982ca94d3e2daf0369
Original-Signed-off-by: Vadim Bendebury <vbendeb@chromium.org>
Original-Reviewed-on: https://chromium-review.googlesource.com/344491
Original-Reviewed-by: Patrick Georgi <pgeorgi@chromium.org>
Reviewed-on: https://review.coreboot.org/14853
Tested-by: build bot (Jenkins)
Reviewed-by: Vadim Bendebury <vbendeb@chromium.org>
Diffstat (limited to 'src/mainboard/google/gru/mainboard.c')
-rw-r--r-- | src/mainboard/google/gru/mainboard.c | 33 |
1 files changed, 33 insertions, 0 deletions
diff --git a/src/mainboard/google/gru/mainboard.c b/src/mainboard/google/gru/mainboard.c index b7a197ee8b..4a5ed4f687 100644 --- a/src/mainboard/google/gru/mainboard.c +++ b/src/mainboard/google/gru/mainboard.c @@ -33,6 +33,39 @@ static void configure_sdmmc(void) gpio_output(GPIO(2, D, 4), 0); /* Keep the max voltage */ + /* + * The SD card on this board is connected to port SDMMC0, which is + * multiplexed with GPIO4B pins 0..5. + * + * Disable all pullups on these pins. For pullup configuration + * register layout stacks banks 2 through 4 together, hence [2] means + * group 4, [1] means bank B. This register is described on page 342 + * of section 1 of the TRM. + * + * Each GPIO pin's pull config takes two bits, writing zero to the + * field disables pull ups/downs, as described on page 342 of rk3399 + * TRM Version 0.3 Part 1. + */ + write32(&rk3399_grf->gpio2_p[2][1], RK_CLRSETBITS(0xfff, 0)); + + /* + * Set all outputs' drive strength to 8 mA. Group 4 bank B driver + * strength requires three bits per pin. Value of 2 written in that + * three bit field means '8 mA', as deduced from the kernel code. + * + * Thus the six pins involved in SDMMC interface require 18 bits to + * configure drive strength, but each 32 bit register provides only 16 + * bits for this setting, this covers 5 pins fully and one bit from + * the 6th pin. Two more bits spill over to the next register. This is + * described on page 378 of rk3399 TRM Version 0.3 Part 1. + */ + write32(&rk3399_grf->gpio4b_e01, + RK_CLRSETBITS(0xffff, + (2 << 0) | (2 << 3) | + (2 << 6) | (2 << 9) | (2 << 12))); + write32(&rk3399_grf->gpio4b_e2, RK_CLRSETBITS(3, 1)); + + /* And now set the multiplexor to enable SDMMC0. */ write32(&rk3399_grf->iomux_sdmmc, IOMUX_SDMMC); } |