From mboxrd@z Thu Jan 1 00:00:00 1970 From: Frank Rowand Subject: Re: [RFC PATCH] OF: fix of_find_node_by_path() assumption that of_allnodes is root Date: Sat, 14 Jun 2014 20:36:42 -0700 Message-ID: <539D14CA.1060105@gmail.com> References: <539A91E2.7000606@gmail.com> Reply-To: frowand.list-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org Mime-Version: 1.0 Content-Type: text/plain; charset=ISO-8859-1 Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: Sender: devicetree-owner-u79uwXL29TY76Z2rM5mHXA@public.gmane.org To: Grant Likely , Pantelis Antoniou Cc: Rob Herring , Rob Herring , "devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org" , Linux Kernel list List-Id: devicetree@vger.kernel.org On 6/14/2014 8:00 AM, Grant Likely wrote: > On Fri, Jun 13, 2014 at 4:06 PM, Grant Likely wrote: >> On Fri, Jun 13, 2014 at 2:52 PM, Rob Herring wrote: >>> On Fri, Jun 13, 2014 at 12:53 AM, Frank Rowand wrote: >>>> From: Frank Rowand >>>> >>>> Pantelis Antoniou reports that 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 >>> >>> Is it not possible to do a fix in of_find_node_by_path instead? I just >>> wonder if this could have any other unintended side-effects on >>> of_attach_node's behavior. >> >> Given that I'm going to replace the custom list with list_head which >> makes this problem go away, I think this solution is fine. >> >> Acked-by: Grant Likely > > That said, the following might be better because it would guarantee > that children always appear after parents... < snip > I agree, I like that better. Patch respun, v2 will be sent as a reply to this email. Pantelis, can you test v2 please? Thanks, Frank -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org More majordomo info at http://vger.kernel.org/majordomo-info.html