diff options
author | Frank Rowand <frank.rowand@sonymobile.com> | 2014-06-14 20:39:05 -0700 |
---|---|---|
committer | Rob Herring <robh@kernel.org> | 2014-06-16 13:31:44 -0500 |
commit | 99de64984c3a7c9bf56a50e6dcc51006c9485620 (patch) | |
tree | b01b4c17cef947bad5ea779622c8409d3ad20f42 | |
parent | 28d3ee40ef5ad488f4ad3376d8f237b9502cecfb (diff) |
OF: fix of_find_node_by_path() assumption that of_allnodes is rootdt-fixes-for-3.16
of_find_node_by_path() is borked because of_allnodes is not guaranteed to
contain the root of the tree after using any of the dynamic update functions
because some other nodes ends up as of_allnodes.
Fixes: c22e650e66b8 of: Make of_find_node_by_path() handle /aliases
Reported-by: pantelis.antoniou@konsulko.com
Signed-off-by: Frank Rowand <frank.rowand@sonymobile.com>
Signed-off-by: Rob Herring <robh@kernel.org>
-rw-r--r-- | drivers/of/base.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/drivers/of/base.c b/drivers/of/base.c index f27d922eaece..b9864806e9b8 100644 --- a/drivers/of/base.c +++ b/drivers/of/base.c @@ -1961,9 +1961,9 @@ int of_attach_node(struct device_node *np) raw_spin_lock_irqsave(&devtree_lock, flags); np->sibling = np->parent->child; - np->allnext = of_allnodes; + np->allnext = np->parent->allnext; + np->parent->allnext = np; np->parent->child = np; - of_allnodes = np; of_node_clear_flag(np, OF_DETACHED); raw_spin_unlock_irqrestore(&devtree_lock, flags); |