Hi Michael,

On 2026-09-29T12:51:05, Michael Walle <[email protected]> wrote:
> binman: imx9image: respect BINMAN_INDIRS for external files
>
> We have the BINMAN_INDIRS for any external binaries, so they don't have
> to be copied into the source/build directory of u-boot. This is
> especially important for image builders like buildroot.
>
> Use the get_input_filename() helper which will also look up the paths in
> BINMAN_INDIRS. The append case only handle external files, thus we can
> use that helper unconditionally.
>
> While at it, update and extend the test case for this image format.
> That is, first create the image in the input directory and then check if
> there is actually an append comand with that image name in the resulting
> .cfgout file.
>
> Signed-off-by: Michael Walle <[email protected]>
>
> tools/binman/etype/nxp_imx9image.py | 22 ++++++++++++----------
>  tools/binman/ftest.py               |  9 ++++++---
>  2 files changed, 18 insertions(+), 13 deletions(-)

Test looks good. Two small typos if you respin for other reasons:
'only handle' should be 'only handles' and 'comand' should be
'command'.

Reviewed-by: Simon Glass <[email protected]>

Regards,
Simon

Reply via email to