diff options
author | Marcin Wojtas <mw@semihalf.com> | 2021-06-25 12:38:53 +0200 |
---|---|---|
committer | David S. Miller <davem@davemloft.net> | 2021-06-25 11:46:29 -0700 |
commit | ac53c26433b51f1835ce5a935970e427d83e3ec5 (patch) | |
tree | cdbef2f33dc8a68365c9bbc569d49d330c743002 /drivers/net/wireless/mediatek/mt7601u/mac.c | |
parent | f92e1869d74e1acc6551256eb084a1c14a054e19 (diff) | |
download | linux-ac53c26433b51f1835ce5a935970e427d83e3ec5.tar.bz2 |
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>
Diffstat (limited to 'drivers/net/wireless/mediatek/mt7601u/mac.c')
0 files changed, 0 insertions, 0 deletions