[U-Boot] [PATCH v2 02/38] binman: Allow selection of logging verbosity

[Bin Meng]

On Wed, Oct 2, 2019 at 9:56 PM Bin Meng <bmeng.cn at gmail.com> wrote:
>
> On Wed, Sep 25, 2019 at 10:12 PM Simon Glass <sjg at chromium.org> wrote:
> >
> > Support a new BINMAN_VERBOSE option to the build, to allow passing the
> > -v flag to binman.
> >
> > Signed-off-by: Simon Glass <sjg at chromium.org>
> > ---
> >
> > Changes in v2: None
> >
> >  Makefile            | 3 ++-
> >  tools/binman/README | 6 ++++++
> >  2 files changed, 8 insertions(+), 1 deletion(-)
> >
> > diff --git a/Makefile b/Makefile
> > index c5caec8267e..fd390e19458 100644
> > --- a/Makefile
> > +++ b/Makefile
> > @@ -1214,7 +1214,8 @@ u-boot.ldr:       u-boot
> >  quiet_cmd_binman = BINMAN  $@
> >  cmd_binman = $(srctree)/tools/binman/binman $(if $(BINMAN_DEBUG),-D) \
> >                  --toolpath $(objtree)/tools \
> > -                build -u -d u-boot.dtb -O . -m \
> > +               $(if $(BINMAN_VERBOSE),-v$(BINMAN_VERBOSE)) \
> > +               build -u -d u-boot.dtb -O . -m \
> >                 -I . -I $(srctree) -I $(srctree)/board/$(BOARDDIR) \
> >                 $(BINMAN_$(@F))
> >
> > diff --git a/tools/binman/README b/tools/binman/README
> > index b4f6392ab74..8e0f0a8c55b 100644
> > --- a/tools/binman/README
> > +++ b/tools/binman/README
> > @@ -934,6 +934,12 @@ BINMAN_DEBUG=1 to your build:
> >     make sandbox_defconfig
> >     make BINMAN_DEBUG=1
> >
> > +To enable verbose logging from binman, base BINMAN_VERBOSE to your build, which
> > +adds a -v<level> option to the call to binman:
> > +
> > +   make sandbox_defconfig
>
> I don't see sandbox_defconfig enables BINMAN. Shouldn't it be
> qemu-x86_defconfig?

I will send a separate patch to correct the defconfig.

>
> > +   make BINMAN_VERBOSE=5
> > +
> >
> >  History / Credits
> >  -----------------
> > --
>
> Other than that,
> Reviewed-by: Bin Meng <bmeng.cn at gmail.com>
> Tested-by: Bin Meng <bmeng.cn at gmail.com>
>

applied to u-boot-x86/next, thanks!