[coreboot-gerrit] Patch set updated for coreboot: 2dfd65a rk3288: LPDDR3 reboot test fail

Patrick Georgi (pgeorgi@google.com) gerrit at coreboot.org
Tue Apr 14 09:45:47 CEST 2015


Patrick Georgi (pgeorgi at google.com) just uploaded a new patch set to gerrit, which you can find at http://review.coreboot.org/9654

-gerrit

commit 2dfd65aee3eb169f476a7e807dd16bda2f093bc5
Author: jinkun.hong <jinkun.hong at rock-chips.com>
Date:   Wed Jan 21 15:47:25 2015 +0800

    rk3288: LPDDR3 reboot test fail
    
    tMRD request 10nCK in LPDDR3, we set the DDR_PCTL_TMRD BIT0~BIT2 to generate
    this single, but the max value we can set is 7, can not meet the standard.So,
    now we send the Mode Register Set command manual,and we can add the delay
    manual.
    
    BUG=chrome-os-partner:34608
    TEST=loop reboot
    BRANCH=veyron
    
    Change-Id: Id974ab935c2df6ea35dcdd240378ffc68de0204d
    Signed-off-by: Stefan Reinauer <reinauer at chromium.org>
    Original-Commit-Id: b60a4de6ff3ad3720c2c06ed7de03ed942360e6c
    Original-Change-Id: I0d29ea9cd82ef018e835ae53090a47d0299ef61d
    Original-Signed-off-by: jinkun.hong <jinkun.hong at rock-chips.com>
    Original-Reviewed-on: https://chromium-review.googlesource.com/242176
    Original-Reviewed-by: Julius Werner <jwerner at chromium.org>
---
 src/soc/rockchip/rk3288/sdram.c | 23 ++++++++++++++++++++---
 1 file changed, 20 insertions(+), 3 deletions(-)

diff --git a/src/soc/rockchip/rk3288/sdram.c b/src/soc/rockchip/rk3288/sdram.c
index d330f99..cda36be 100644
--- a/src/soc/rockchip/rk3288/sdram.c
+++ b/src/soc/rockchip/rk3288/sdram.c
@@ -998,12 +998,29 @@ void sdram_init(const struct rk3288_sdram_params *sdram_params)
 		writel(POWER_UP_START, &ddr_pctl_regs->powctl);
 		while (!(readl(&ddr_pctl_regs->powstat) & POWER_UP_DONE))
 			;
-		send_command(ddr_pctl_regs, 3, DESELECT_CMD, 0);
-		udelay(1);
-		send_command(ddr_pctl_regs, 3, PREA_CMD, 0);
 
 		memory_init(ddr_publ_regs, sdram_params->dramtype);
 		move_to_config_state(ddr_publ_regs, ddr_pctl_regs);
+
+		if (sdram_params->dramtype == LPDDR3) {
+			send_command(ddr_pctl_regs, 3, DESELECT_CMD, 0);
+			udelay(1);
+			send_command(ddr_pctl_regs, 3, PREA_CMD, 0);
+			udelay(1);
+			send_command(ddr_pctl_regs, 3, MRS_CMD, LPDDR2_MA(63) |
+				LPDDR2_OP(0xFC));
+			udelay(1);
+			send_command(ddr_pctl_regs, 3, MRS_CMD, LPDDR2_MA(1) |
+				LPDDR2_OP(sdram_params->phy_timing.mr[1]));
+			udelay(1);
+			send_command(ddr_pctl_regs, 3, MRS_CMD, LPDDR2_MA(2) |
+				LPDDR2_OP(sdram_params->phy_timing.mr[2]));
+			udelay(1);
+			send_command(ddr_pctl_regs, 3, MRS_CMD, LPDDR2_MA(3) |
+				LPDDR2_OP(sdram_params->phy_timing.mr[3]));
+			udelay(1);
+		}
+
 		set_bandwidth_ratio(channel, sdram_params->ch[channel].bw);
 		/*
 		 * set cs



More information about the coreboot-gerrit mailing list