diff options
author | Philipp Zabel <p.zabel@pengutronix.de> | 2014-03-04 12:31:24 +0100 |
---|---|---|
committer | Philipp Zabel <p.zabel@pengutronix.de> | 2014-03-06 17:41:54 +0100 |
commit | d484700a36952c6675aa47dec4d7a536929aa922 (patch) | |
tree | e963ac4830c406395117d7aa2f9fd80d19b55a3e | |
parent | f2a575f67695dcba9062acd666ae5aab2380b95c (diff) | |
download | op-kernel-dev-d484700a36952c6675aa47dec4d7a536929aa922.zip op-kernel-dev-d484700a36952c6675aa47dec4d7a536929aa922.tar.gz |
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>
-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 715144af..fd4b9c2 100644 --- 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; |