[coreboot-gerrit] New patch to review for coreboot: c1df33b rk3288: support single channel ddr

[Patrick Georgi (pgeorgi@google.com)]

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

-gerrit

commit c1df33b069ae11a79fc1e1694a2ba5388786234e
Author: jinkun.hong <jinkun.hong at rock-chips.com>
Date:   Sat Feb 14 15:07:50 2015 +0800

    rk3288: support single channel ddr
    
    When using single-channel ddr, DMC channel 1 need to reset dll,
    otherwise it will lead to pmdomain idle request fails.
    
    BUG=chrome-os-partner:35654
    BRANCH=veyron
    TEST=boot rialto
    
    Change-Id: Id6b673187c688d238e9a391b3d98720c783e3af4
    Signed-off-by: Patrick Georgi <pgeorgi at chromium.org>
    Original-Commit-Id: 927e8426104f8869e139c3f60a04cd49bf726e61
    Original-Change-Id: I8be1567040ddb5f2a2b0d06568e517d794ead87a
    Original-Signed-off-by: jinkun.hong <jinkun.hong at rock-chips.com>
    Original-Reviewed-on: https://chromium-review.googlesource.com/250060
    Original-Reviewed-by: Julius Werner <jwerner at chromium.org>
---
 src/soc/rockchip/rk3288/sdram.c | 5 ++++-
 1 file changed, 4 insertions(+), 1 deletion(-)

diff --git a/src/soc/rockchip/rk3288/sdram.c b/src/soc/rockchip/rk3288/sdram.c
index 2698024..3d547cd 100644
--- a/src/soc/rockchip/rk3288/sdram.c
+++ b/src/soc/rockchip/rk3288/sdram.c
@@ -987,7 +987,7 @@ void sdram_init(const struct rk3288_sdram_params *sdram_params)
 
 	rkclk_configure_ddr(sdram_params->ddr_freq);
 
-	for (channel = 0; channel < sdram_params->num_channels; channel++) {
+	for (channel = 0; channel < 2; channel++) {
 		struct rk3288_ddr_pctl_regs *ddr_pctl_regs =
 		    rk3288_ddr_pctl[channel];
 		struct rk3288_ddr_publ_regs *ddr_publ_regs =
@@ -996,6 +996,9 @@ void sdram_init(const struct rk3288_sdram_params *sdram_params)
 		phy_pctrl_reset(ddr_publ_regs, channel);
 		phy_dll_bypass_set(ddr_publ_regs, sdram_params->ddr_freq);
 
+		if (channel >= sdram_params->num_channels)
+			continue;
+
 		dfi_cfg(ddr_pctl_regs, sdram_params->dramtype);
 
 		pctl_cfg(channel, sdram_params);