diff options
| author | Philipp Zabel <p.zabel@pengutronix.de> | 2014-03-04 12:31:24 +0100 |
|---|---|---|
| committer | Laurent Pinchart <laurent.pinchart@linaro.org> | 2015-04-28 20:59:55 +0300 |
| commit | 0fa4d3d2420dae2db713862cdd5dda81e1a2e185 (patch) | |
| tree | c73efe7abed3c36b687128ff75fe6d30f4e8fbd4 | |
| parent | afcb03945c5e1637987ba3cdba2a2c095320bd90 (diff) | |
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 <p.zabel@pengutronix.de>
Acked-by: Tomi Valkeinen <tomi.valkeinen@ti.com>
Acked-by: Mauro Carvalho Chehab <m.chehab@samsung.com>
Acked-by: Sylwester Nawrocki <s.nawrocki@samsung.com>
Backports: d484700a36952c6675aa47dec4d7a536929aa922
| -rw-r--r-- | drivers/of/base.c | 3 |
1 files changed, 3 insertions, 0 deletions
diff --git a/drivers/of/base.c b/drivers/of/base.c index a7e0ea6ea5b0..d77a3661f183 100644 --- a/drivers/of/base.c +++ b/drivers/of/base.c @@ -1717,6 +1717,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; |