google/veyron: Work around RAM code strapping error
commit2d99f3b15886ff984ae469b50d850b360e19911e
authorJulius Werner <jwerner@chromium.org>
Thu, 16 Mar 2017 21:08:09 +0000 (16 14:08 -0700)
committerPatrick Georgi <pgeorgi@google.com>
Fri, 17 Mar 2017 10:38:22 +0000 (17 11:38 +0100)
treed92e9c0412284e2695663a71a8c57fb88a30ac11
parent336a34c81be7455ce2ec978b31fb04f5d4518ec0
google/veyron: Work around RAM code strapping error

With a recent patch (google/veyron_*: Add new Micron and Hynix modules)
we switched RAM codes for Veyron boards to tri-state since we were
running out of binary numbers. Unfortunately we only tested that change
on Minnie and Speedy, and it turns out that it broke Jaq, Jerry and
Mighty. The "high" RAM code pins on those boards were incorrectly
strapped with 100Kohm resistors (as opposed to 1Kohm on Minnie and
Speedy), which is too high to overpower the SoC-internal pull-down we
use to differentiate "high" from "tri-state". Since we already used
tri-state codes on some Minnie and Speedy SKUs we have to hack up the
code to work differently on these two groups of boards to keep
everything working.

BRANCH=veyron
BUG=b:36279493
TEST=Compiled, confirmed ram_code called the right function depending on
board.

Change-Id: I253b213ef7ca621ce47a7a55a5119a167d944078
Signed-off-by: Julius Werner <jwerner@chromium.org>
Reviewed-on: https://review.coreboot.org/18859
Reviewed-by: Aaron Durbin <adurbin@chromium.org>
Tested-by: build bot (Jenkins)
Reviewed-by: Patrick Georgi <pgeorgi@google.com>
src/mainboard/google/veyron/Kconfig
src/mainboard/google/veyron/boardid.c