fdt: check for fdt errors in fdt_create_phandle
authorTimur Tabi <timur@freescale.com>
Tue, 20 Sep 2011 23:24:34 +0000 (18:24 -0500)
committerGerald Van Baren <gvb@unssw.com>
Sat, 15 Oct 2011 13:35:00 +0000 (09:35 -0400)
fdt_create_phandle() was ignoring errors from fdt_set_phandle().  If an
error occurs, print an error message and return 0, which is an invalid
phandle.  We also need to change the return type for fdt_create_phandle()
to indicate that it cannot return an error code.

Signed-off-by: Timur Tabi <timur@freescale.com>
common/fdt_support.c
include/fdt_support.h

index 46aa842..698abf7 100644 (file)
@@ -1241,15 +1241,22 @@ int fdt_set_phandle(void *fdt, int nodeoffset, uint32_t phandle)
  * @fdt: ptr to device tree
  * @nodeoffset: node to update
  */
-int fdt_create_phandle(void *fdt, int nodeoffset)
+unsigned int fdt_create_phandle(void *fdt, int nodeoffset)
 {
        /* see if there is a phandle already */
        int phandle = fdt_get_phandle(fdt, nodeoffset);
 
        /* if we got 0, means no phandle so create one */
        if (phandle == 0) {
+               int ret;
+
                phandle = fdt_alloc_phandle(fdt);
-               fdt_set_phandle(fdt, nodeoffset, phandle);
+               ret = fdt_set_phandle(fdt, nodeoffset, phandle);
+               if (ret < 0) {
+                       printf("Can't set phandle %u: %s\n", phandle,
+                              fdt_strerror(ret));
+                       return 0;
+               }
        }
 
        return phandle;
index 0091dc3..4826eee 100644 (file)
@@ -97,7 +97,7 @@ int fdt_node_offset_by_compat_reg(void *blob, const char *compat,
                                        phys_addr_t compat_off);
 int fdt_alloc_phandle(void *blob);
 int fdt_set_phandle(void *fdt, int nodeoffset, uint32_t phandle);
-int fdt_create_phandle(void *fdt, int nodeoffset);
+unsigned int fdt_create_phandle(void *fdt, int nodeoffset);
 int fdt_add_edid(void *blob, const char *compat, unsigned char *buf);
 
 int fdt_verify_alias_address(void *fdt, int anode, const char *alias,