On Sat, Apr 18, 2020 at 04:46:06PM +0200, Andrew Lunn wrote: > > - ret = of_mdiobus_register(bus, np); > > So this is the interesting part. What you really want to be doing is > adding a device_mdiobus_register(bus, dev) to the core. And it needs > to share as much as possible with the of_mdiobus_register() > implementation. Sure, will take this approach and submit v3. Thanks Calvin