On Mon, Nov 05, 2018 at 12:50:02PM -0600, Rob Herring wrote: > On Mon, Nov 5, 2018 at 3:17 AM Heikki Krogerus > <heikki.krogerus@xxxxxxxxxxxxxxx> wrote: > > +static const char *of_fwnode_name(const struct fwnode_handle *fwnode) > > +{ > > + return to_of_node(fwnode)->name; > > I'm trying to get rid of the DT name ptr, so please don't add one. You > can use of_node_full_name() here instead if "<name>@<unit-address>" > instead of <name> is fine. Otherwise, you've got to allocate your own > storage and use "%pOFn" printf specifier. If we do this here, we will change a behaviour of the entire set of of_fwnode_get_named_child_node() users. I think this is out of scope of the series. -- With Best Regards, Andy Shevchenko