Aaron Durbin (adurbin@google.com) just uploaded a new patch set to gerrit, which you can find at http://review.coreboot.org/9239
-gerrit
commit 891fc7f17270d6de6d6d18ad02da19d13a5ed8e4 Author: jinkun.hong jinkun.hong@rock-chips.com Date: Thu Sep 25 20:27:26 2014 -0700
coreboot: rk3288: add new ddr config and support ddr3 freq up to 800mhz
Add ddr3-samsung-2GB config and modify 533mhz linit. Support ddr3 freq up to 800mhz. Enable ODT at LPDDR3.
BUG=None TEST=Boot Veyron Pinky
Original-Change-Id: Ic02a381985796a00644c5c681b96f10ad1558936 Original-Signed-off-by: jinkun.hong jinkun.hong@rock-chips.com Original-Reviewed-on: https://chromium-review.googlesource.com/220113 Original-Reviewed-by: Julius Werner jwerner@chromium.org Original-Tested-by: Lin Huang hl@rock-chips.com Original-Commit-Queue: Julius Werner jwerner@chromium.org
Change-Id: I867753bc5d1eb301eb4975f5a945bfdba9b8f37d (cherry picked from commit e6689cbb0ec50317672c8ebe4e23555ca2f01005) Signed-off-by: Aaron Durbin adurbin@chromium.org --- src/mainboard/google/veyron_pinky/sdram_configs.c | 2 +- .../sdram_inf/sdram-ddr3-samsung-2GB.inc | 78 ++++++++++++++++++++++ src/soc/rockchip/rk3288/sdram.c | 37 +++++++--- 3 files changed, 105 insertions(+), 12 deletions(-)
diff --git a/src/mainboard/google/veyron_pinky/sdram_configs.c b/src/mainboard/google/veyron_pinky/sdram_configs.c index 21a483c..cc29de0 100644 --- a/src/mainboard/google/veyron_pinky/sdram_configs.c +++ b/src/mainboard/google/veyron_pinky/sdram_configs.c @@ -28,7 +28,7 @@ static struct rk3288_sdram_params sdram_configs[] = { #include "sdram_inf/sdram-unused.inc" /* ram_code = 0001 */ #include "sdram_inf/sdram-unused.inc" /* ram_code = 0010 */ #include "sdram_inf/sdram-unused.inc" /* ram_code = 0011 */ -#include "sdram_inf/sdram-unused.inc" /* ram_code = 0100 */ +#include "sdram_inf/sdram-ddr3-samsung-2GB.inc" /* ram_code = 0100 */ #include "sdram_inf/sdram-unused.inc" /* ram_code = 0101 */ #include "sdram_inf/sdram-unused.inc" /* ram_code = 0110 */ #include "sdram_inf/sdram-unused.inc" /* ram_code = 0111 */ diff --git a/src/mainboard/google/veyron_pinky/sdram_inf/sdram-ddr3-samsung-2GB.inc b/src/mainboard/google/veyron_pinky/sdram_inf/sdram-ddr3-samsung-2GB.inc new file mode 100644 index 0000000..3fdbecf --- /dev/null +++ b/src/mainboard/google/veyron_pinky/sdram_inf/sdram-ddr3-samsung-2GB.inc @@ -0,0 +1,78 @@ +{ + /* two Samsung K4B4G1646D-BYK0 chips */ + { + { + .rank = 0x1, + .col = 0xA, + .bk = 0x3, + .bw = 0x2, + .dbw = 0x1, + .row_3_4 = 0x0, + .cs0_row = 0xF, + .cs1_row = 0xF + }, + { + .rank = 0x1, + .col = 0xA, + .bk = 0x3, + .bw = 0x2, + .dbw = 0x1, + .row_3_4 = 0x0, + .cs0_row = 0xF, + .cs1_row = 0xF + } + }, + { + .togcnt1u = 0x29A, + .tinit = 0xC8, + .trsth = 0x1F4, + .togcnt100n = 0x42, + .trefi = 0x4E, + .tmrd = 0x4, + .trfc = 0xEA, + .trp = 0xA, + .trtw = 0x5, + .tal = 0x0, + .tcl = 0xA, + .tcwl = 0x7, + .tras = 0x19, + .trc = 0x24, + .trcd = 0xA, + .trrd = 0x7, + .trtp = 0x5, + .twr = 0xA, + .twtr = 0x5, + .texsr = 0x200, + .txp = 0x5, + .txpdll = 0x10, + .tzqcs = 0x40, + .tzqcsi = 0x0, + .tdqs = 0x1, + .tcksre = 0x7, + .tcksrx = 0x7, + .tcke = 0x4, + .tmod = 0xC, + .trstl = 0x43, + .tzqcl = 0x100, + .tmrr = 0x0, + .tckesr = 0x5, + .tdpd = 0x0 + }, + { + .dtpr0 = 0x48F9AAB4, + .dtpr1 = 0xEA0910, + .dtpr2 = 0x1002C200, + .mr[0] = 0xA60, + .mr[1] = 0x40, + .mr[2] = 0x10, + .mr[3] = 0x0 + }, + .noc_timing = 0x30B25564, + .noc_activate = 0x627, + .ddrconfig = 3, + .ddr_freq = 666000000, + .dramtype = DDR3, + .num_channels = 2, + .stride = 9, + .odt = 1 +}, diff --git a/src/soc/rockchip/rk3288/sdram.c b/src/soc/rockchip/rk3288/sdram.c index 7ac84f0..6c0c7fd 100644 --- a/src/soc/rockchip/rk3288/sdram.c +++ b/src/soc/rockchip/rk3288/sdram.c @@ -340,12 +340,12 @@ static struct rk3288_msch_regs * const rk3288_msch[2] = { #define PRT_DLLSRST(n) ((n) << 0)
/* PTR1 */ -#define PRT_DINIT1(n) ((n) << 19) #define PRT_DINIT0(n) ((n) << 0) +#define PRT_DINIT1(n) ((n) << 19)
/* PTR2 */ -#define PRT_DINIT3(n) ((n) << 17) #define PRT_DINIT2(n) ((n) << 0) +#define PRT_DINIT3(n) ((n) << 17)
/* DCR */ #define DDRMD_LPDDR 0 @@ -442,6 +442,7 @@ static struct rk3288_msch_regs * const rk3288_msch[2] = { #define DPDE_CMD (9)
#define LPDDR2_MA(n) (((n) & 0xff) << 4) +#define LPDDR2_OP(n) (((n) & 0xff) << 12)
#define START_CMD (1u << 31)
@@ -620,7 +621,7 @@ static void pctl_cfg(u32 channel, writel(sdram_params->pctl_timing.tcwl - 1, &ddr_pctl_regs->dfitphywrlat); writel(MDDR_LPDDR2_CLK_STOP_IDLE(0) | DDR3_EN - | DDR2_DDR3_BL_8 | TFAW_CFG(5) | PD_EXIT_SLOW + | DDR2_DDR3_BL_8 | TFAW_CFG(6) | PD_EXIT_SLOW | PD_TYPE(1) | PD_IDLE(0), &ddr_pctl_regs->mcfg); writel(MSCH_MAINDDR3(channel, 1), &rk3288_grf->soc_con0);
@@ -650,12 +651,19 @@ static void phy_cfg(u32 channel, const struct rk3288_sdram_params *sdram_params) writel(sdram_params->noc_activate, &msch_regs->activate); writel(BUSWRTORD(2) | BUSRDTOWR(2) | BUSRDTORD(1), &msch_regs->devtodev); - writel(PRT_ITMSRST(8) | PRT_DLLLOCK(2750) | PRT_DLLSRST(27), - &ddr_publ_regs->ptr[0]); - /* tDINIT1=400ns (533MHz), tDINIT0=500us (533MHz) */ - writel(PRT_DINIT1(213) | PRT_DINIT0(266525), &ddr_publ_regs->ptr[1]); - /* tDINIT3=1us (533MHz), tDINIT2=200us (533MHz) */ - writel(PRT_DINIT3(534) | PRT_DINIT2(106610), &ddr_publ_regs->ptr[2]); + writel(PRT_DLLLOCK(div_round_up(sdram_params->ddr_freq / 1000000 + * 5120, 1000)) + | PRT_DLLSRST(div_round_up(sdram_params->ddr_freq / 1000000 + * 50, 1000)) + | PRT_ITMSRST(8), &ddr_publ_regs->ptr[0]); + writel(PRT_DINIT0(div_round_up(sdram_params->ddr_freq / 1000000 + * 500000, 1000)) + | PRT_DINIT1(div_round_up(sdram_params->ddr_freq / 1000000 + * 400, 1000)), &ddr_publ_regs->ptr[1]); + writel(PRT_DINIT2(div_round_up(sdram_params->ddr_freq / 1000000 + * 200000, 1000)) + | PRT_DINIT3(div_round_up(sdram_params->ddr_freq / 1000000 + * 1000, 1000)), &ddr_publ_regs->ptr[2]);
switch (sdram_params->dramtype) { case LPDDR3: @@ -860,7 +868,7 @@ static int data_training(u32 channel, } } /* send some auto refresh to complement the lost while DTT */ - for (i = 0; i < (rank > 1 ? 4 : 2); i++) + for (i = 0; i < (rank > 1 ? 8 : 4); i++) send_command(ddr_pctl_regs, rank, REF_CMD, 0);
if (sdram_params->dramtype != LPDDR3) @@ -962,7 +970,10 @@ void sdram_init(const struct rk3288_sdram_params *sdram_params) int zqcr; printk(BIOS_INFO, "Starting SDRAM initialization...\n");
- if (sdram_params->ddr_freq > 533000000) + if ((sdram_params->dramtype == DDR3 + && sdram_params->ddr_freq > 800000000) + || (sdram_params->dramtype == LPDDR3 + && sdram_params->ddr_freq > 533000000)) die("SDRAM frequency is to high!");
rkclk_configure_ddr(sdram_params->ddr_freq); @@ -1011,6 +1022,10 @@ void sdram_init(const struct rk3288_sdram_params *sdram_params) if (sdram_params->dramtype == LPDDR3) { /* LPDDR2/LPDDR3 need to wait DAI complete, max 10us */ udelay(10); + send_command(ddr_pctl_regs, + (sdram_params->ch[channel].rank | 1), + MRS_CMD, LPDDR2_MA(11) | + sdram_params->odt ? LPDDR2_OP(3) : 0); if (channel == 0) { writel(0, &ddr_pctl_regs->mrrcfg0); send_command(ddr_pctl_regs, 1, MRR_CMD,