[PATCH 6/9] binman: Add support for RISC-V OpenSBI fw_dynamic blob

[Simon Glass]

Hi Bin,

On Fri, 7 May 2021 at 18:29, Bin Meng <bmeng.cn at gmail.com> wrote:
>
> Hi Simon,
>
> On Thu, May 6, 2021 at 7:37 AM Simon Glass <sjg at chromium.org> wrote:
> >
> > Hi Bin,
> >
> > On Wed, 5 May 2021 at 07:16, Bin Meng <bmeng.cn at gmail.com> wrote:
> > >
> > > Add an entry for RISC-V OpenSBI's 'fw_dynamic' firmware payload.
> > >
> > > Signed-off-by: Bin Meng <bmeng.cn at gmail.com>
> > > ---
> > >
> > >  tools/binman/entries.rst          | 13 +++++++++++++
> > >  tools/binman/etype/opensbi.py     | 23 +++++++++++++++++++++++
> > >  tools/binman/ftest.py             |  7 +++++++
> > >  tools/binman/test/201_opensbi.dts | 16 ++++++++++++++++
> > >  4 files changed, 59 insertions(+)
> > >  create mode 100644 tools/binman/etype/opensbi.py
> > >  create mode 100644 tools/binman/test/201_opensbi.dts
> >
> > Is it possible to use a default filename for this, instead of having
> > to specify it?
>
> I don't get what you meant. Would you please clarify?

I mean adding a GetDefaultFilename() argument. But I see that you are
using a -a parameter, so perhaps there is no point.

Reviewed-by: Simon Glass <sjg at chromium.org>

Regards,
Simon