[PATCH 5/7] binman: Workaround lz4 cli padding in test cases

[Simon Glass]

Hi Mattijs,

On Thu, 10 Apr 2025 at 03:27, Mattijs Korpershoek
<mkorpershoek at kernel.org> wrote:
>
> Hi Simon,
>
> Thank you for the patch.
>
> On mar., mars 04, 2025 at 06:09, Simon Glass <sjg at chromium.org> wrote:
>
> > From: Jiaxun Yang <jiaxun.yang at flygoat.com>
> >
> > Newer lz4 util is not happy with any padding at end of file,
> > it would abort with error message like:
> >
> > Stream followed by undecodable data at position 43.
> >
> > Workaround by skipping testCompUtilPadding test case and manually
> > strip padding in testCompressSectionSize test case.
> >
> > Signed-off-by: Jiaxun Yang <jiaxun.yang at flygoat.com>
> > Reviewed-by: Simon Glass <sjg at chromium.org>
> > Signed-off-by: Simon Glass <sjg at chromium.org>
>
> Tested that this solves issues when using lz4 1.9.4.
>
> This patch solves:
> https://source.denx.de/u-boot/custodians/u-boot-dm/-/issues/38
>
> Tested-by: Mattijs Korpershoek <mkorpershoek at kernel.org>
>

Ah, I'd forgotten about this series, as I applied it to my tree a
while back. I sent a v2 for Tom[1].

Regards,
SImon

[1] https://patchwork.ozlabs.org/project/uboot/list/?series=452114