[U-Boot] [PATCH] SPL: fat: Fix spl_parse_image_header() return value handling

Peng Fan van.freenix at gmail.com
Fri May 20 11:31:37 CEST 2016


On Fri, May 20, 2016 at 01:45:24AM +0200, Marek Vasut wrote:
>The spl_parse_image_header() can return 0 and it is not an error.
>Only treat non-zero return value as an error.
>
>Signed-off-by: Marek Vasut <marex at denx.de>
>Cc: Fabio Estevam <fabio.estevam at nxp.com>
>Cc: Peng Fan <van.freenix at gmail.com>
>Cc: Stefano Babic <sbabic at denx.de>
>Cc: Tom Rini <trini at konsulko.com>

Reviewed-by: Peng Fan <van.freenix at gmail.com>

Regards,
Peng.
>---
> common/spl/spl_fat.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
>diff --git a/common/spl/spl_fat.c b/common/spl/spl_fat.c
>index 338ea2f..5b0d969 100644
>--- a/common/spl/spl_fat.c
>+++ b/common/spl/spl_fat.c
>@@ -58,7 +58,7 @@ int spl_load_image_fat(struct blk_desc *block_dev,
> 		goto end;
> 
> 	err = spl_parse_image_header(header);
>-	if (err <= 0)
>+	if (err)
> 		goto end;
> 
> 	err = file_fat_read(filename, (u8 *)spl_image.load_addr, 0);
>-- 
>2.7.0
>


More information about the U-Boot mailing list