[PATCH v3 01/12] binman: Allow entry args to be required

Michal Simek michal.simek at xilinx.com
Mon Sep 7 08:31:15 CEST 2020


Hi Simon,

On 05. 09. 20 23:10, Simon Glass wrote:
> If an entry argument is needed by an entry but the entry argument is not
> present, then a strange error can occur when trying to read the file.
> 
> Fix this by allowing arguments to be required. Select this option for the
> cros-ec-rw entry. If a filename is provided in the node, allow that to be
> used.
> 
> Also tidy up a few related tests to make the error string easier to find,
> and fully ignore unused return values.
> 
> Signed-off-by: Simon Glass <sjg at chromium.org>
> ---
> 
> (no changes since v1)
> 
>  tools/binman/README.entries             |  7 ++++++-
>  tools/binman/etype/blob_named_by_arg.py | 10 ++++++----
>  tools/binman/etype/cros_ec_rw.py        |  3 +--
>  tools/binman/ftest.py                   | 15 +++++++++++----
>  4 files changed, 24 insertions(+), 11 deletions(-)
> 
> Applied to u-boot-dm/next, thanks!

Did you start to use patman for sending this?
Just curios because I am missing indentation in origin message.

Thanks,
Michal



More information about the U-Boot mailing list