[PATCH v1 5/5] board: verdin-imx8mp: change prints in spl_dram_init function

Fabio Estevam festevam at gmail.com
Wed Mar 29 13:41:10 CEST 2023


Hi Marcel,

The whole series looks good.

On Wed, Mar 29, 2023 at 8:24 AM Marcel Ziswiler <marcel at ziswiler.com> wrote:
>
> From: Emanuele Ghidoli <emanuele.ghidoli at toradex.com>
>
> change prints to show which DDR configuration (single/dual rank) is used
>
> Signed-off-by: Emanuele Ghidoli <emanuele.ghidoli at toradex.com>
> Signed-off-by: Marcel Ziswiler <marcel.ziswiler at toradex.com>
>
> ---
>
>  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 ab5bcbc0929..f43c6cc02f8 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");

Just a minor suggestion:
puts() can be used instead of printf() as it uses less resources and
can help inside a memory constrained environment, such as SPL.


More information about the U-Boot mailing list