[PATCH 1/5] binman: Fix subentry expansion for FIT entry type

[Simon Glass]

Hi Alper,

On Sun, 6 Feb 2022 at 14:03, Alper Nebi Yasak <alpernebiyasak at gmail.com> wrote:
>
> Binman tries to expand some entries into parts that make it up, e.g.
> 'u-boot' into a 'u-boot-expanded' section that contains 'u-boot-nodtb'
> and 'u-boot-dtb'. Entries with child entries must call ExpandEntries()
> on them to build a correct image, as it's possible that unexpanded child
> entries have no data of their own. The FIT entry type doesn't currently
> do this, which means putting a "u-boot" entry inside it doesn't work as
> expected.
>
> Implement ExpandEntries() for FIT and add a copy of a simple FIT image
> test that checks subentry expansion in FIT entries.
>
> Signed-off-by: Alper Nebi Yasak <alpernebiyasak at gmail.com>
> ---
>
>  tools/binman/etype/fit.py |  5 +++++
>  tools/binman/ftest.py     | 45 +++++++++++++++++++++++++++++++++++++++
>  2 files changed, 50 insertions(+)

Looks good.

But please try to create a helper function in ftest.py just above the
one you add, so you can call it from both tests and avoid so much code
duplication?

Regards,
Simon