From 5fb80b61a7fc72e4ac409e2ebd45dffb7118fa89 Mon Sep 17 00:00:00 2001 From: Emanuele Ghidoli Date: Mon, 6 Feb 2023 16:31:26 +0100 Subject: board: verdin-imx8mp: change prints in spl_dram_init function change prints to show which DDR configuration (single/dual rank) is used Upstream-Status: Pending Waiting for internal test and validation before sending to upstream. Signed-off-by: Emanuele Ghidoli --- board/toradex/verdin-imx8mp/spl.c | 14 ++++++++++---- 1 file changed, 10 insertions(+), 4 deletions(-) diff --git a/board/toradex/verdin-imx8mp/spl.c b/board/toradex/verdin-imx8mp/spl.c index a966e8c512..b901038e90 100644 --- a/board/toradex/verdin-imx8mp/spl.c +++ b/board/toradex/verdin-imx8mp/spl.c @@ -35,11 +35,17 @@ void spl_dram_init(void) /* * Try configuring for dual rank memory falling back to single rank */ - if (ddr_init(&dram_timing)) { - printf("Dual rank failed, attempting single rank configuration.\n"); - lpddr4_single_rank_training_patch(); - ddr_init(&dram_timing); + if (!ddr_init(&dram_timing)) { + printf("DDR configured as dual rank\n"); + return; } + + lpddr4_single_rank_training_patch(); + if (!ddr_init(&dram_timing)) { + printf("DDR configured as single rank\n"); + return; + } + printf("DDR configuration failed\n"); } void spl_board_init(void) -- cgit v1.2.3