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@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).