On Wed, May 07, 2025 at 09:12:48AM +0200, Herve Codina wrote: > Setting fwnode->dev is specific to fw_devlink. > > In order to avoid having a direct 'fwnode->dev = dev;' in several > place in the kernel, introduce fw_devlink_set_device() helper to perform > this operation. Makes sense, can you also mark that field as __private? So sparse can catch the abusers up. -- With Best Regards, Andy Shevchenko