Hi Andy, On Fri, Jan 27, 2023 at 10:25 AM Andy Shevchenko <andriy.shevchenko@xxxxxxxxxxxxxxx> wrote: > On Thu, Jan 26, 2023 at 04:11:30PM -0800, Saravana Kannan wrote: > > The OF_POPULATED flag was set to let fw_devlink know that the device > > tree node will not have a struct device created for it. This information > > is used by fw_devlink to avoid deferring the probe of consumers of this > > device tree node. > > > > Let's use fwnode_dev_initialized() instead because it achieves the same > > effect without using OF specific flags. This allows more generic code to > > be written in driver core. > > ... > > > - of_node_set_flag(np, OF_POPULATED); > > + fwnode_dev_initialized(&np->fwnode, true); > > of_fwnode_handle(np) ? Or of_node_to_fwnode(). Looks like we have (at least) two of them... Gr{oetje,eeting}s, Geert -- Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@xxxxxxxxxxxxxx In personal conversations with technical people, I call myself a hacker. But when I'm talking to journalists I just say "programmer" or something like that. -- Linus Torvalds