summary refs log tree commit diff
path: root/drivers/of
diff options
context:
space:
mode:
authorFrank Rowand <frank.rowand@sonymobile.com>2014-06-14 20:39:05 -0700
committerRob Herring <robh@kernel.org>2014-06-16 13:31:44 -0500
commit99de64984c3a7c9bf56a50e6dcc51006c9485620 (patch)
treeb01b4c17cef947bad5ea779622c8409d3ad20f42 /drivers/of
parent28d3ee40ef5ad488f4ad3376d8f237b9502cecfb (diff)
downloadlinux-99de64984c3a7c9bf56a50e6dcc51006c9485620.tar.gz
OF: fix of_find_node_by_path() assumption that of_allnodes is root
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>
Diffstat (limited to 'drivers/of')
-rw-r--r--drivers/of/base.c4
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);