[PATCH v6 2/3] binman: collection: Set build_done on referenced entries

[Simon Glass]

Hi Yan,

On 2026-04-14T13:15:55, Yan WANG <yan.wang at softathome.com> wrote:
> binman: collection: Set build_done on referenced entries
>
> The collection etype uses phandles in the 'content' property to
> reference other entries. Mark each referenced entry with build_done
> to avoid rebuilding the same entry data multiple times.
>
> This is important for cases where rebuilding may change the data
> content, e.g. due to timestamps or random IVs in encryption.
>
> Refactor GetContentsByPhandle() to return both the entry object and
> its data.
>
> Signed-off-by: yan wang <yan.wang at softathome.com>
>
> tools/binman/etype/collection.py | 9 +++++++--
>  tools/binman/etype/section.py    | 5 +++--
>  2 files changed, 10 insertions(+), 4 deletions(-)

> diff --git a/tools/binman/etype/collection.py b/tools/binman/etype/collection.py
> @@ -45,12 +45,17 @@ class Entry_collection(Entry):
> +            # Mark referenced entries as build_done to avoid rebuilding
> +            if required:
> +                entry.build_done = True

I believe this should use mark_build_done() instead, as it recursively
marks child entries in sections.

Regards,
Simon