net: mdiobus: withdraw fwnode_mdbiobus_register
authorMarcin Wojtas <mw@semihalf.com>
Fri, 25 Jun 2021 10:38:53 +0000 (12:38 +0200)
committerDavid S. Miller <davem@davemloft.net>
Fri, 25 Jun 2021 18:46:29 +0000 (11:46 -0700)
commitac53c26433b51f1835ce5a935970e427d83e3ec5
treecdbef2f33dc8a68365c9bbc569d49d330c743002
parentf92e1869d74e1acc6551256eb084a1c14a054e19
net: mdiobus: withdraw fwnode_mdbiobus_register

The newly implemented fwnode_mdbiobus_register turned out to be
problematic - in case the fwnode_/of_/acpi_mdio are built as
modules, a dependency cycle can be observed during the depmod phase of
modules_install, eg.:

depmod: ERROR: Cycle detected: fwnode_mdio -> of_mdio -> fwnode_mdio
depmod: ERROR: Found 2 modules in dependency cycles!

OR:

depmod: ERROR: Cycle detected: acpi_mdio -> fwnode_mdio -> acpi_mdio
depmod: ERROR: Found 2 modules in dependency cycles!

A possible solution could be to rework fwnode_mdiobus_register,
so that to merge the contents of acpi_mdiobus_register and
of_mdiobus_register. However feasible, such change would
be very intrusive and affect huge amount of the of_mdiobus_register
users.

Since there are currently 2 users of ACPI and MDIO
(xgmac_mdio and mvmdio), withdraw the fwnode_mdbiobus_register
and roll back to a simple 'if' condition in affected drivers.

Fixes: 62a6ef6a996f ("net: mdiobus: Introduce fwnode_mdbiobus_register()")
Signed-off-by: Marcin Wojtas <mw@semihalf.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
drivers/net/ethernet/freescale/Kconfig
drivers/net/ethernet/freescale/xgmac_mdio.c
drivers/net/ethernet/marvell/mvmdio.c
drivers/net/mdio/fwnode_mdio.c
include/linux/fwnode_mdio.h