of: Warn if of_graph_parse_endpoint is called with the root node

If of_graph_parse_endpoint is given a parentless node instead of an
endpoint node, it is clearly a bug.

Signed-off-by: Philipp Zabel <[email protected]>
Acked-by: Tomi Valkeinen <[email protected]>
Acked-by: Mauro Carvalho Chehab <[email protected]>
Acked-by: Sylwester Nawrocki <[email protected]>

diff --git a/drivers/of/base.c b/drivers/of/base.c
index 715144af3a83b2592cde314aae0148376ae361cd..fd4b9c2eaa158ae128eaa5f3e3a1d10a7457accb 100644 (file)
--- a/drivers/of/base.c
+++ b/drivers/of/base.c
@@ -1996,6 +1996,9 @@ int of_graph_parse_endpoint(const struct device_node *node,
 {
        struct device_node *port_node = of_get_parent(node);
 
+       WARN_ONCE(!port_node, "%s(): endpoint %s has no parent node\n",
+                 __func__, node->full_name);
+
        memset(endpoint, 0, sizeof(*endpoint));
 
        endpoint->local_node = node;