On Fri, 19 Aug 2022 at 08:26, Stefan Herbrechtsmeier
<stefan.herbrechtsmeier-...@weidmueller.com> wrote:
>
> From: Stefan Herbrechtsmeier <stefan.herbrechtsme...@weidmueller.com>
>
> Add an optional length header attribute to the device tree blob entry
> class based on the compressed data header from the utilities to compress
> and decompress data.
>
> If needed the header could be enabled with the following
> attribute beside the compress attribute:
> prepend = "length";
>
> The header was introduced as part of commit eb0f4a4cb402 ("binman:
> Support replacing data in a cbfs") to allow device tree entries to be
> larger than the compressed contents. Regarding the commit "this is
> necessary to cope with a compressed device tree being updated in such a
> way that it shrinks after the entry size is already set (an obscure
> case)". This case need to be fixed without influence any compressed data
> by itself.
>
> Signed-off-by: Stefan Herbrechtsmeier <stefan.herbrechtsme...@weidmueller.com>
>
> ---
>
> (no changes since v5)
>
> Changes in v5:
> - Fix decompress
>
> Changes in v3:
> - Move comp_util.py changes (drop with_header) into separate commits.
>
> Changes in v2:
> - Reworked to make the feature optional.
>
> tools/binman/entries.rst | 3 ++
> tools/binman/etype/blob_dtb.py | 27 +++++++++++++
> tools/binman/ftest.py | 39 +++++++++++++++++++
> .../test/235_compress_dtb_prepend_invalid.dts | 17 ++++++++
> .../test/236_compress_dtb_prepend_length.dts | 19 +++++++++
> 5 files changed, 105 insertions(+)
> create mode 100644 tools/binman/test/235_compress_dtb_prepend_invalid.dts
> create mode 100644 tools/binman/test/236_compress_dtb_prepend_length.dts
>
Reviewed-by: Simon Glass <s...@chromium.org>
