[PATCH] image: Add support for ZSTD decompression

[Tom Rini <trini@konsulko.com>]

On Fri, May 01, 2020 at 05:15:41PM +0200, Robert Marko wrote:
> On Fri, May 1, 2020 at 4:56 PM Tom Rini <trini@konsulko.com> wrote:
> >
> > On Sat, Apr 25, 2020 at 07:37:21PM +0200, Robert Marko wrote:
> >
> > > This patch adds support for ZSTD decompression of FIT images.
> > >
> > > Signed-off-by: Robert Marko <robert.marko@sartura.hr>
> > > Cc: Luka Perkov <luka.perkov@sartura.hr>
> > > ---
> > >  common/image.c  | 52 +++++++++++++++++++++++++++++++++++++++++++++++++
> > >  include/image.h |  1 +
> > >  2 files changed, 53 insertions(+)
> > >
> > > diff --git a/common/image.c b/common/image.c
> > > index 94873cb6ed..70ba0f4328 100644
> > > --- a/common/image.c
> > > +++ b/common/image.c
> > > @@ -42,6 +42,7 @@
> > >  #include <lzma/LzmaTypes.h>
> > >  #include <lzma/LzmaDec.h>
> > >  #include <lzma/LzmaTools.h>
> > > +#include <linux/zstd.h>
> > >
> > >  #ifdef CONFIG_CMD_BDI
> > >  extern int do_bdinfo(cmd_tbl_t *cmdtp, int flag, int argc, char * const argv[]);
> > > @@ -193,6 +194,7 @@ static const table_entry_t uimage_comp[] = {
> > >       {       IH_COMP_LZMA,   "lzma",         "lzma compressed",      },
> > >       {       IH_COMP_LZO,    "lzo",          "lzo compressed",       },
> > >       {       IH_COMP_LZ4,    "lz4",          "lz4 compressed",       },
> > > +     {       IH_COMP_ZSTD,   "zstd",         "zstd compressed",      },
> > >       {       -1,             "",             "",                     },
> > >  };
> > >
> > > @@ -480,6 +482,56 @@ int image_decomp(int comp, ulong load, ulong image_start, int type,
> > >               break;
> > >       }
> > >  #endif /* CONFIG_LZ4 */
> > > +#ifdef CONFIG_ZSTD
> >
> > We need to add SPL_ZSTD as a symbol to lib/Kconfig and then use
> > CONFIG_IS_ENABLED() tests here to avoid growth in SPL.  Thanks!
> Hi,
> is that something that I need to do or?

Yes.  You need to add the symbol, and then the code you're adding needs
to make use of '#if CONFIG_IS_ENABLED(ZSTD)' rather than '#ifdef
CONFIG_ZSTD'.  Sorry for not being clear enough.

-- 
Tom
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 659 bytes
Desc: not available
URL: <https://lists.denx.de/pipermail/u-boot/attachments/20200501/ddc2e77b/attachment.sig>