[PATCH v3 1/4] fs/squashfs: Add init. and clean-up functions to decompression

Thomas Petazzoni thomas.petazzoni at bootlin.com
Tue Aug 18 16:34:24 CEST 2020


Hello,

In your commit title: spurious . after "init".

On Tue, 18 Aug 2020 16:31:15 +0200
Joao Marcos Costa <joaomarcos.costa at bootlin.com> wrote:

> -static struct squashfs_ctxt ctxt;
> +struct squashfs_ctxt sqfs_ctxt;

Why are you dropping the "static" here ? This symbol only needs to be
visible from that file. If you keep the "static" keyword, you can keep
the variable named "ctxt" and avoid a lengthy patch.

> +	ret = sqfs_decompressor_init(&sqfs_ctxt);

See you're passing a reference to the squashfs context here, so this
variable does not need to be visible by other files.

Best regards,

Thomas
-- 
Thomas Petazzoni, CTO, Bootlin
Embedded Linux and Kernel engineering
https://bootlin.com


More information about the U-Boot mailing list