[U-Boot] [PATCH] boot_fit: Change return value from FDT_ERROR to -EINVAL in fdt_offset()

Franklin S Cooper Jr fcooper at ti.com
Thu Aug 10 02:08:58 UTC 2017



On 08/04/2017 03:47 PM, Nobuhiro Iwamatsu wrote:
> FDT_ERROR is defined as unsigned long. However, since the return value of
> fdt_offset() is int, a warning will occur when compiling. Also, it is better
> to use -EINVAL than FDT_ERROR.
> This fixes this problem by change return value from FDT_ERROR to -EINVAL.
> 
> Signed-off-by: Nobuhiro Iwamatsu <iwamatsu at nigauri.org>
> CC: Franklin S Cooper Jr <fcooper at ti.com>
> ---
>  common/boot_fit.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/common/boot_fit.c b/common/boot_fit.c
> index 51440a6e6f..0a723150b5 100644
> --- a/common/boot_fit.c
> +++ b/common/boot_fit.c
> @@ -25,7 +25,7 @@ int fdt_offset(void *fit)
>  	images = fdt_path_offset(fit, FIT_IMAGES_PATH);
>  	if (images < 0) {
>  		debug("%s: Cannot find /images node: %d\n", __func__, images);
> -		return FDT_ERROR;
> +		return -EINVAL;
>  	}
>  

Reviewed-by: Franklin S Cooper Jr <fcooper at ti.com>
>  	fdt_name = fdt_getprop(fit, node, FIT_FDT_PROP, &fdt_len);
> 


More information about the U-Boot mailing list