[PATCH] binman: Support optional external blobs

[Tom Rini]

On Mon, Jan 02, 2023 at 11:52:17AM +0100, Quentin Schulz wrote:
> Hi Simon,
> 
> On 12/17/22 22:29, Simon Glass wrote:
> > Some blobs are actually not necessary for the board to work correctly. Add
> > a property to allow this to be indicated. Missing optional blobs do not
> > cause a build failure.
> > 
> > Signed-off-by: Simon Glass <sjg at chromium.org>
> > ---
> > 
> >   tools/binman/binman.rst                |  9 +++++++++
> >   tools/binman/control.py                | 11 +++++++++++
> >   tools/binman/entry.py                  | 25 +++++++++++++++++++++----
> >   tools/binman/etype/blob.py             |  2 +-
> >   tools/binman/etype/section.py          | 14 +++++++++++++-
> >   tools/binman/ftest.py                  | 10 ++++++++++
> >   tools/binman/test/266_blob_ext_opt.dts | 21 +++++++++++++++++++++
> >   7 files changed, 86 insertions(+), 6 deletions(-)
> >   create mode 100644 tools/binman/test/266_blob_ext_opt.dts
> > 
> > diff --git a/tools/binman/binman.rst b/tools/binman/binman.rst
> > index 1ab6d012b83..391494907ab 100644
> > --- a/tools/binman/binman.rst
> > +++ b/tools/binman/binman.rst
> > @@ -689,6 +689,15 @@ no-expanded:
> >       `no-expanded` property disables this just for a single entry. Put the
> >       `no-expanded` boolean property in the node to select this behaviour.
> > +optional:
> > +    External blobs are normally required to be present for the image to be
> > +    built (but see `External blobs`_). This properly allows an entry to be
> > +    optional, so that when it is cannot be found, this problem is ignored and
> > +    an empty file is used for this blob. This should be used only when the blob
> > +    is entirely optional and is not needed for correct operation of the image.
> 
> Could we provide such an example?

The example is the pmic firmware (SCP) for sunxi platforms. That can be
omitted safely (which results in some PM features not working).

-- 
Tom
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 659 bytes
Desc: not available
URL: <https://lists.denx.de/pipermail/u-boot/attachments/20230102/60a2ea59/attachment.sig>