Hi Bin, On Sat, 20 Feb 2021 at 06:01, Bin Meng <bmeng...@gmail.com> wrote: > > Hi Simon, > > On Sat, Feb 20, 2021 at 7:55 PM Simon Glass <s...@chromium.org> wrote: > > > > On Thu, 18 Feb 2021 at 08:59, Bin Meng <bmeng...@gmail.com> wrote: > > > > > > This adds a basic test for the newly introduced 'addrmap' command. > > > > > > Signed-off-by: Bin Meng <bmeng...@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 <s...@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