On Wed, Jul 8, 2020 at 8:35 PM Calvin Johnson
[off-list ref] wrote:

Store fwnode for mdiobus in the bus structure so that it can
later be retrieved and used whenever mdiobus fwnode information
is required.

...

+       if (pdev->dev.fwnode)

But do you need this check?

+               bus->dev.fwnode = pdev->dev.fwnode;

Shouldn't be rather something like dev_fwnode().
And maybe set_primary_fwnode()? I'm not sure about the latter, though.

-- 
With Best Regards,
Andy Shevchenko