[PATCH v2 14/38] test: cmd: Add a basic test for 'addrmap' command

[Simon Glass]

Hi Bin,

On Sat, 20 Feb 2021 at 06:01, Bin Meng <bmeng.cn at gmail.com> wrote:
>
> Hi Simon,
>
> On Sat, Feb 20, 2021 at 7:55 PM Simon Glass <sjg at chromium.org> wrote:
> >
> > On Thu, 18 Feb 2021 at 08:59, Bin Meng <bmeng.cn at gmail.com> wrote:
> > >
> > > This adds a basic test for the newly introduced 'addrmap' command.
> > >
> > > Signed-off-by: Bin Meng <bmeng.cn at gmail.com>
> > >
> > > ---
> > >
> > > Changes in v2:
> > > - new patch: test: cmd: Add a basic test for 'addrmap' command
> > >
> > >  include/test/suites.h |  2 ++
> > >  test/cmd/Makefile     |  1 +
> > >  test/cmd/addrmap.c    | 38 ++++++++++++++++++++++++++++++++++++++
> > >  test/cmd_ut.c         |  6 ++++++
> > >  4 files changed, 47 insertions(+)
> > >  create mode 100644 test/cmd/addrmap.c
> >
> > Reviewed-by: Simon Glass <sjg at chromium.org>
> >
> > Just checking this test is enabled for sandbox?
>
> Not yet. I don't think sandbox has enabled CONFIG_ADDR_MAP.

OK then can you please enable it so we have test coverage?

Regards,
Simon