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@nigauri.org CC: Franklin S Cooper Jr fcooper@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@ti.com

fdt_name = fdt_getprop(fit, node, FIT_FDT_PROP, &fdt_len);