From mboxrd@z Thu Jan 1 00:00:00 1970 From: Tom Rini Date: Fri, 1 May 2020 12:42:46 -0400 Subject: [PATCH] image: Add support for ZSTD decompression In-Reply-To: References: <20200425173721.2759955-1-robert.marko@sartura.hr> <20200501145605.GG4468@bill-the-cat> Message-ID: <20200501164246.GA12564@bill-the-cat> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de On Fri, May 01, 2020 at 05:15:41PM +0200, Robert Marko wrote: > On Fri, May 1, 2020 at 4:56 PM Tom Rini 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 > > > Cc: Luka Perkov > > > --- > > > 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 > > > #include > > > #include > > > +#include > > > > > > #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: