On 10:08-20230824, Emanuele Ghidoli wrote: > From: Emanuele Ghidoli <emanuele.ghid...@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.ghid...@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