Hi Tobias, On 3/21/2020 1:24 PM, Tobias Waldekranz wrote: > An MDIO controller present on development boards for Marvell switches > from the Link Street (88E6xxx) family. > > Using this module, you can use the following setup as a development > platform for switchdev and DSA related work. > > .-------. .-----------------. > | USB----USB | > | SoC | | 88E6390X-DB ETH1-10 > | ETH----ETH0 | > '-------' '-----------------' > > Signed-off-by: Tobias Waldekranz <tobias@xxxxxxxxxxxxxx> > --- [snip] > +config MDIO_MVUSB > + tristate "Marvell USB to MDIO Adapter" > + depends on OF_MDIO && USB You could drop OF_MDIO here since of_mdiobus_register() does default to mdiobus_register() when CONFIG_OF is disabled. Other than that, LGTM! Reviewed-by: Florian Fainelli <f.fainelli@xxxxxxxxx> -- Florian