[PATCH V2 4/5] nvmem: u-boot-env: improve coding style
Miquel Raynal
miquel.raynal at bootlin.com
Wed Dec 20 08:19:24 CET 2023
Hi Rafał,
zajec5 at gmail.com wrote on Tue, 19 Dec 2023 18:40:24 +0100:
> From: Rafał Miłecki <rafal at milecki.pl>
>
> 1. Prefer kzalloc() over kcalloc()
> See memory-allocation.rst which says: "to be on the safe side it's
> best to use routines that set memory to zero, like kzalloc()"
> 2. Drop dev_err() for u_boot_env_add_cells() fail
> It can fail only on -ENOMEM. We don't want to print error then.
> 3. Add extra "crc32_addr" variable
> It makes code reading header's crc32 easier to understand / review.
>
> Signed-off-by: Rafał Miłecki <rafal at milecki.pl>
Looks much nicer :)
Reviewed-by: Miquel Raynal <miquel.raynal at bootlin.com>
Thanks,
Miquèl
More information about the U-Boot
mailing list