Re: [PATCH 5/5] lib/vsprintf: Add %pfw conversion specifier for printing fwnode names

From: Heikki Krogerus
Date: Tue Mar 26 2019 - 10:06:39 EST


Hi,

On Tue, Mar 26, 2019 at 03:13:53PM +0200, Andy Shevchenko wrote:
> > > > On ACPI based systems the resulting strings look like
> > > >
> > > > \_SB.PCI0.CIO2.port@xxxxxxxxxx@0
> > > >
> > > > where the nodes are separated by a dot (".") and the first three are
> > > > ACPI device nodes and the latter two ACPI data nodes.
> > >
> > > Do we support swnode here?
> >
> > Good question. The swnodes have no hierarchy at the moment (they're only
> > created for a struct device as a parent) and they do not have human-readable
> > names. So I'd say it's not relevant right now. Should these two change,
> > support for swnode could (and should) be added later on.
>
> Heikki, what do you think about this?

Well, the swnodes do have hierarchy. That was kind of the whole point
of introducing them. They now can also be named using "name" property.
See commit 344798206f171c5abea7ab1f9762fa526d7f539d.

thanks,

--
heikki