[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