Hi Marek,
Thank you for the patch.
On Mon, Mar 18, 2024 at 04:00:45PM +0100, Marek Vasut wrote:
> Clean up tmp variable use and type cast in env_get_bootm_size().
> No functional change.
The code change looks fine, but you may want to expand the commit
message to explain why this patch improves the code.
> Signed-off-by: Marek Vasut <marek.vasut+rene...@mailbox.org>
> ---
> Cc: Heinrich Schuchardt <xypron.g...@gmx.de>
> Cc: Kuninori Morimoto <kuninori.morimoto...@renesas.com>
> Cc: Laurent Pinchart <laurent.pinch...@ideasonboard.com>
> Cc: Simon Glass <s...@chromium.org>
> Cc: Tom Rini <tr...@konsulko.com>
> ---
> V2: - New patch
> ---
> boot/image-board.c | 8 +++-----
> 1 file changed, 3 insertions(+), 5 deletions(-)
>
> diff --git a/boot/image-board.c b/boot/image-board.c
> index 3263497a1d5..e3d63745299 100644
> --- a/boot/image-board.c
> +++ b/boot/image-board.c
> @@ -129,10 +129,8 @@ phys_size_t env_get_bootm_size(void)
> phys_addr_t start;
> char *s = env_get("bootm_size");
>
> - if (s) {
> - tmp = (phys_size_t)simple_strtoull(s, NULL, 16);
> - return tmp;
> - }
> + if (s)
> + return simple_strtoull(s, NULL, 16);
>
> start = gd->ram_base;
> size = gd->ram_size;
> @@ -142,7 +140,7 @@ phys_size_t env_get_bootm_size(void)
>
> s = env_get("bootm_low");
> if (s)
> - tmp = (phys_size_t)simple_strtoull(s, NULL, 16);
> + tmp = simple_strtoull(s, NULL, 16);
> else
> tmp = start;
>
Maybe you could even drop the tmp variable completely by writing this
if (s)
size -= simple_strtoull(s, NULL, 16) - start;
return size;
I've never liked variables named tmp :-)
--
Regards,
Laurent Pinchart
