diff options
author | Moritz Fischer <moritzf@google.com> | 2021-02-18 14:43:20 -0800 |
---|---|---|
committer | ron minnich <rminnich@gmail.com> | 2021-02-19 19:08:29 +0000 |
commit | c73102d0f51c7e83c4a1723acf08bf92ecaff603 (patch) | |
tree | 74170460715c169917a6fe0afbea1ca08098dd71 | |
parent | d8f352b4fd0e260494fdb14d04b73b11607bee9d (diff) |
soc/rockchip/rk3399/sdram: Add phy_ctrl_reset
Add support for resetting PHY PCTRL for both channel 0 and 1.
On the ROCKPro64 board this allows getting past a pctl_cfg() failure.
Signed-off-by: Moritz Fischer <moritzf@google.com>
Change-Id: I9f807e318ffc63c568d04518c3edd02c1064e185
Reviewed-on: https://review.coreboot.org/c/coreboot/+/50890
Reviewed-by: ron minnich <rminnich@gmail.com>
Reviewed-by: Angel Pons <th3fanbus@gmail.com>
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
-rw-r--r-- | src/soc/rockchip/rk3399/sdram.c | 13 |
1 files changed, 13 insertions, 0 deletions
diff --git a/src/soc/rockchip/rk3399/sdram.c b/src/soc/rockchip/rk3399/sdram.c index 89463aa174..9490c0c2e4 100644 --- a/src/soc/rockchip/rk3399/sdram.c +++ b/src/soc/rockchip/rk3399/sdram.c @@ -100,6 +100,18 @@ static void copy_to_reg(u32 *dest, const u32 *src, u32 n) } } +static void phy_pctrl_reset(u32 channel) +{ + rkclk_ddr_reset(channel, 1, 1); + udelay(10); + + rkclk_ddr_reset(channel, 1, 0); + udelay(10); + + rkclk_ddr_reset(channel, 0, 0); + udelay(10); +} + static void phy_dll_bypass_set(u32 channel, struct rk3399_ddr_publ_regs *ddr_publ_regs, u32 freq) { @@ -1083,6 +1095,7 @@ void sdram_init(const struct rk3399_sdram_params *params) rkclk_configure_ddr(ddr_freq); for (channel = 0; channel < 2; channel++) { + phy_pctrl_reset(channel); phy_dll_bypass_set(channel, rk3399_ddr_publ[channel], ddr_freq); if (channel >= params->num_channels) |