On Tue, Dec 07, 2021 at 05:41:43PM +0200, Andy Shevchenko wrote: > In order to make the underneath API easier to change in the future, > prevent users from dereferencing fwnode from struct device. > Instead, use the specific dev_fwnode() API for that. ... > /* It needs to reference this NHI */ > - if (nhi->pdev->dev.fwnode != args.fwnode) > + if (dev_fwnode(&nhi->pdev->dev) != args.fwnode) Alternatively: if (!device_match_fwnode(&nhi->pdev->dev, args.fwnode)) Tell me. which one you prefer. > goto out_put; -- With Best Regards, Andy Shevchenko