Hi Pali,
Could you review this patch?
Thanks, Tony
On Sun, Apr 2, 2023 at 9:42 PM Tony Dinh mibodhi@gmail.com wrote:
- DDR Training sequence happens very fast. The speedup in boot time is
negligible by skipping the training sequence during 2nd boot or after. So remove the check and skip.
- This change improves the robustness of DDR training. If u-boot crashed
during DDR training, the training could be left in a limbo state, where the BootROM has recorded that it is already in a 2nd boot. The training must be repeated in this scenario to get out of this limbo state, but due to the check it cannot be performed.
Signed-off-by: Tony Dinh mibodhi@gmail.com
drivers/ddr/marvell/a38x/mv_ddr_plat.c | 7 ------- 1 file changed, 7 deletions(-)
diff --git a/drivers/ddr/marvell/a38x/mv_ddr_plat.c b/drivers/ddr/marvell/a38x/mv_ddr_plat.c index 6e7949ac72..8ec9fb0874 100644 --- a/drivers/ddr/marvell/a38x/mv_ddr_plat.c +++ b/drivers/ddr/marvell/a38x/mv_ddr_plat.c @@ -1363,13 +1363,6 @@ int mv_ddr_pre_training_soc_config(const char *ddr_type) DRAM_RESET_MASK_MASKED << DRAM_RESET_MASK_OFFS); }
/* Check if DRAM is already initialized */if (reg_read(REG_BOOTROM_ROUTINE_ADDR) &(1 << REG_BOOTROM_ROUTINE_DRAM_INIT_OFFS)) {printf("%s Training Sequence - 2nd boot - Skip\n", ddr_type);return MV_OK;}/* Fix read ready phases for all SOC in reg 0x15c8 */ reg_val = reg_read(TRAINING_DBG_3_REG);-- 2.30.2