[PATCH] fs: ubifs: Add support for ZSTD decompression
Heiko Schocher
hs at denx.de
Mon May 27 08:13:51 CEST 2024
Hello Piotr,
On 06.05.24 16:07, Heiko Schocher wrote:
> Hello Piotr,
>
> On 30.04.24 12:23, Piotr Wojtaszczyk wrote:
>> Signed-off-by: Piotr Wojtaszczyk <piotr.wojtaszczyk at timesys.com>
>> ---
>>
>> fs/ubifs/ubifs-media.h | 2 ++
>> fs/ubifs/ubifs.c | 55 ++++++++++++++++++++++++++++++++++++++++--
>> 2 files changed, 55 insertions(+), 2 deletions(-)
>
> Looks good to me, thanks!
>
> Acked-by: Heiko Schocher <hs at denx.de>
Just tried to apply your patch (sorry for being so late) and checkpatch
dropped some warnings, see below. Could you please fix them?
Thanks!
bye,
Heiko
[1] checkpatch warnings
WARNING: Missing commit description - Add an appropriate one
WARNING: Use 'if (IS_ENABLED(CONFIG...))' instead of '#if or #ifdef' where possible
#160: FILE: fs/ubifs/ubifs.c:30:
+#if IS_ENABLED(CONFIG_ZSTD)
WARNING: Use 'if (IS_ENABLED(CONFIG...))' instead of '#if or #ifdef' where possible
#172: FILE: fs/ubifs/ubifs.c:50:
+#if IS_ENABLED(CONFIG_ZSTD)
CHECK: Alignment should match open parenthesis
#174: FILE: fs/ubifs/ubifs.c:52:
+static int zstd_decompress_wrapper(const unsigned char *in, size_t in_len,
+ unsigned char *out, size_t *out_len)
WARNING: braces {} are not necessary for single statement blocks
#183: FILE: fs/ubifs/ubifs.c:61:
+ if (ret < 0) {
+ return ret;
+ }
WARNING: Use 'if (IS_ENABLED(CONFIG...))' instead of '#if or #ifdef' where possible
#199: FILE: fs/ubifs/ubifs.c:99:
+#if IS_ENABLED(CONFIG_ZSTD)
CHECK: Please don't use multiple blank lines
#212: FILE: fs/ubifs/ubifs.c:112:
+
+
CHECK: Alignment should match open parenthesis
#231: FILE: fs/ubifs/ubifs.c:215:
+ ubifs_err(c, "%s compression is not compiled in",
+ compr->name ? compr->name : "unknown");
WARNING: Use 'if (IS_ENABLED(CONFIG...))' instead of '#if or #ifdef' where possible
#239: FILE: fs/ubifs/ubifs.c:280:
+#if IS_ENABLED(CONFIG_ZSTD)
total: 0 errors, 6 warnings, 3 checks, 101 lines checked
NOTE: For some of the reported defects, checkpatch may be able to
mechanically convert to the typical style using --fix or --fix-inplace.
mbox has style problems, please review.
NOTE: Ignored message types: COMPLEX_MACRO CONSIDER_KSTRTO ENOSYS MINMAX
MULTISTATEMENT_MACRO_USE_DO_WHILE NETWORKING_BLOCK_COMMENT_STYLE PREFER_ETHER_ADDR_COPY USLEEP_RANGE
NOTE: If any of the errors are false positives, please report
them to the maintainer, see CHECKPATCH in MAINTAINERS.
Wende an: fs: ubifs: Add support for ZSTD decompression
--
DENX Software Engineering GmbH, Managing Director: Erika Unter
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: +49-8142-66989-52 Fax: +49-8142-66989-80 Email: hs at denx.de
More information about the U-Boot
mailing list