On Mon, Jun 20, 2022 at 05:02:18PM +0200, Marcin Wojtas wrote: > A helper function which allows getting the struct net_device pointer > associated with a given device tree node can be more generic and > also support alternative hardware description. Switch to fwnode_ > and update the only existing caller in DSA subsystem. ... > +static int fwnode_dev_node_match(struct device *dev, const void *data) > { > for (; dev; dev = dev->parent) { > - if (dev->of_node == data) > + if (dev->fwnode == data) We have a helper in device/bus.h (?) device_match_fwnode(). > return 1; > } But this all sounds like a good candidate to be generic. Do we have more users in the kernel of a such? -- With Best Regards, Andy Shevchenko