On Fri, Apr 12, 2024 at 07:54:32AM -0500, Rob Herring wrote: > On Thu, Apr 11, 2024 at 6:56 PM Saravana Kannan <saravanak@xxxxxxxxxx> wrote: > I think it is better to not have this wrapper. We want it to be clear > when we're acquiring a ref. I know get_device() does that, but I have > to look up what get_dev_from_fwnode() does exactly. > > Side note: I didn't know fwnode has a ptr to the struct device. I > wonder if we can kill off of_find_device_by_node() using that. That's > for platform devices though. I don't like the idea because we already have a big design issue with fwnode that is used in struct device. Ideally, fwnode has to be a node in the linked list, head of which is provided by the user (struct device, for example). When it's done, it will be easy to handle. Have you read the comment in the struct fwnode_handle definition? -- With Best Regards, Andy Shevchenko
