Hi Marek, On Thu, 4 May 2023 at 10:01, Marek Vasut <marek.va...@mailbox.org> wrote: > > On 5/4/23 17:51, Tom Rini wrote: > > On Sat, Apr 22, 2023 at 03:01:34PM +0200, Marek Vasut wrote: > > > >> Add test for command bdinfo . > >> > >> Signed-off-by: Marek Vasut <marek.vasut+rene...@mailbox.org> > >> Reviewed-by: Simon Glass <s...@chromium.org> > >> --- > >> Cc: Jason Liu <jason.hui....@nxp.com> > >> Cc: Michal Simek <michal.si...@amd.com> > >> Cc: Ovidiu Panait <ovpan...@gmail.com> > >> Cc: Simon Glass <s...@chromium.org> > >> --- > >> include/test/suites.h | 1 + > >> test/cmd/Makefile | 1 + > >> test/cmd/bdinfo.c | 179 ++++++++++++++++++++++++++++++++++++++++++ > >> test/cmd_ut.c | 6 ++ > >> 4 files changed, 187 insertions(+) > >> create mode 100644 test/cmd/bdinfo.c > > > > This breaks building on "snow" which does not set CFG_SYS_SDRAM_SIZE > > seemingly. > > Isn't this macro mandatory ? > > Simon, can you elaborate what chromebook snow does with the DRAM there ?
It detects it at runtime, so doesn't have that symbol. Regards, Simon