dm: core: Supress dead-code warning in __of_get_next_child()
authorSimon Glass <sjg@chromium.org>
Wed, 7 Jun 2017 16:28:45 +0000 (10:28 -0600)
committerTom Rini <trini@konsulko.com>
Mon, 12 Jun 2017 12:38:10 +0000 (08:38 -0400)
Suppress a warning on next = next->sibling.

Signed-off-by: Simon Glass <sjg@chromium.org>
Reported-by: Coverity (CID: 163245)
Fixes 644ec0a (dm: core: Add livetree access functions)

drivers/core/of_access.c

index 94ef3cc251fb4b567e69bb485a73c8c07da85c28..93a65604967bfb0f2c74646fd13331346b8ec8e5 100644 (file)
@@ -244,6 +244,12 @@ static struct device_node *__of_get_next_child(const struct device_node *node,
                return NULL;
 
        next = prev ? prev->sibling : node->child;
+       /*
+        * coverity[dead_error_line : FALSE]
+        * Dead code here since our current implementation of of_node_get()
+        * always returns NULL (Coverity CID 163245). But we leave it as is
+        * since we may want to implement get/put later.
+        */
        for (; next; next = next->sibling)
                if (of_node_get(next))
                        break;