On Fri, Mar 9, 2012 at 10:59 AM, David Daney <david.daney@xxxxxxxxxx> wrote: > On 03/08/2012 05:33 PM, Grant Likely wrote: >> >> On Wed, 29 Feb 2012 11:21:04 -0800, David Daney<ddaney.cavm@xxxxxxxxx> >> wrote: >>> >>> From: David Daney<david.daney@xxxxxxxxxx> >>> >>> Currently all paths passed to of_find_node_by_path() must begin with a >>> '/', indicating a full path to the desired node. >>> >>> Augment the look-up code so that if a path does *not* begin with '/', >>> the path is used as the name of an /aliases property. The value of >>> this alias is then used as the full node path to be found. >>> >>> Signed-off-by: David Daney<david.daney@xxxxxxxxxx> > > [...] > >> >> All the aliases are already decoded at boot time now. See >> of_alias_scan(). Instead of open-coding this, you can add an >> of_alias_lookup() function something like this (untested): >> > > After objections from davem, and a bit of thought, I already indicated on a > different branch of this thread that we should drop this patch. > > I have improved my code so that it is no longer needed. Okay.