[PATCH v1 2/2] board: verdin-am62: fix check for minimum memory size
Nishanth Menon
nm at ti.com
Thu Aug 24 14:14:12 CEST 2023
On 10:08-20230824, Emanuele Ghidoli wrote:
> From: Emanuele Ghidoli <emanuele.ghidoli at toradex.com>
>
> verdin am62 SKUs comes in multiple memory configuration, check that
> the detected memory is at least 512MB since we have some
> reserved memory just before this threshold and therefore
> the module cannot work with less memory.
>
> Fixes: 7d1a10659f5b ("board: toradex: add verdin am62 support")
> Signed-off-by: Emanuele Ghidoli <emanuele.ghidoli at toradex.com>
> ---
> board/toradex/verdin-am62/verdin-am62.c | 4 ++--
> 1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/board/toradex/verdin-am62/verdin-am62.c b/board/toradex/verdin-am62/verdin-am62.c
> index a3d1d07a0cbf..d09dda5bccc9 100644
> --- a/board/toradex/verdin-am62/verdin-am62.c
> +++ b/board/toradex/verdin-am62/verdin-am62.c
> @@ -28,8 +28,8 @@ int dram_init(void)
> {
> gd->ram_size = get_ram_size((long *)CFG_SYS_SDRAM_BASE, CFG_SYS_SDRAM_SIZE);
>
> - if (gd->ram_size < SZ_64M)
> - puts("## WARNING: Less than 64MB RAM detected\n");
> + if (gd->ram_size < SZ_512M)
> + puts("## WARNING: Less than 512MB RAM detected\n");
Have you considered fdtdec_setup_mem_size_base
fdtdec_setup_memory_banksize - in which case the reserved memory
regions can be changed in dt (will need corresponding custom binaries
ofcourse)..
>
> return 0;
> }
> --
> 2.34.1
>
--
Regards,
Nishanth Menon
Key (0xDDB5849D1736249D) / Fingerprint: F8A2 8693 54EB 8232 17A3 1A34 DDB5 849D 1736 249D
More information about the U-Boot
mailing list