[U-Boot] [PATCH] spl: remove redundant call to parse_image_header()

Simon Glass sjg at chromium.org
Fri Nov 18 02:15:08 CET 2016


On 16 November 2016 at 04:15, Tomas Melin <tomas.melin at vaisala.com> wrote:
> Image header was checked twice.
>
> Signed-off-by: Tomas Melin <tomas.melin at vaisala.com>
> ---
>  common/spl/spl_ymodem.c | 1 -
>  1 file changed, 1 deletion(-)
>
> diff --git a/common/spl/spl_ymodem.c b/common/spl/spl_ymodem.c
> index 13e8e51..957894d 100644
> --- a/common/spl/spl_ymodem.c
> +++ b/common/spl/spl_ymodem.c
> @@ -109,7 +109,6 @@ static int spl_ymodem_load_image(struct spl_image_info *spl_image,
>                 while ((res = xyzModem_stream_read(buf, BUF_SIZE, &err)) > 0)
>                         size += res;
>         } else {
> -               spl_parse_image_header(spl_image, (struct image_header *)buf);
>                 ret = spl_parse_image_header(spl_image,
>                                              (struct image_header *)buf);
>                 if (ret)
> --
> 2.1.4
>

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


More information about the U-Boot mailing list