[U-Boot] [PATCH] image: fix getenv_bootm_size() function

Simon Glass sjg at chromium.org
Fri Jan 8 04:33:47 CET 2016


Hi Masahiro,

On 17 December 2015 at 22:17, Masahiro Yamada
<yamada.masahiro at socionext.com> wrote:
> Currently, this function returns wrong size if "bootm_low" is defined,
> but "bootm_size" is not.
>
> Signed-off-by: Masahiro Yamada <yamada.masahiro at socionext.com>
> ---
>
>  common/image.c | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/common/image.c b/common/image.c
> index d63d9e0..f4a1dc8 100644
> --- a/common/image.c
> +++ b/common/image.c
> @@ -472,9 +472,9 @@ phys_size_t getenv_bootm_size(void)
>
>
>  #if defined(CONFIG_ARM) && defined(CONFIG_NR_DRAM_BANKS)
> -       return gd->bd->bi_dram[0].size - tmp;
> +       return gd->bd->bi_dram[0].size - (tmp - gd->bd->bi_dram[0].start);
>  #else
> -       return gd->bd->bi_memsize - tmp;
> +       return gd->bd->bi_memsize - (tmp - gd->bd->bi_memstart);
>  #endif
>  }
>
> --
> 1.9.1
>

Reviewed-by: Simon Glass <sjg at chromium.org>

But can you please add a comment to the image.h header file as to what
this function is supposed to do?

Regards,
Simon


More information about the U-Boot mailing list