Hi Marek, On Sat, 22 Apr 2023 at 07:01, Marek Vasut <marek.vasut+rene...@mailbox.org> wrote: > > Add test for command bdinfo . > > Signed-off-by: Marek Vasut <marek.vasut+rene...@mailbox.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 >
Reviewed-by: Simon Glass <s...@chromium.org> see below [..] > + /* The gd->multi_dtb_fit may not be available, hence, #if below. */ > +#if CONFIG_IS_ENABLED(MULTI_DTB_FIT) > + bdinfo_test_num_l(uts, "multi_dtb_fit", (ulong)gd->multi_dtb_fit); You can avoid it if you use gd_multi_dtb_fit() > +#endif > + [..] Regards, Simon