@@ -613,9 +613,9 @@ static int mdiobus_scan_bus_c45(struct mii_bus *bus)
* stomping over the true devices reply, to performing a write to
* themselves which was intended for another device. Now that C22
* devices have been found, see if any of them are bad for C45, and if we
- * should skip the C45 scan.
+ * should prohibit any C45 transactions.
*/
-static bool mdiobus_prevent_c45_scan(struct mii_bus *bus)
+void mdiobus_scan_for_broken_c45_access(struct mii_bus *bus)
{
int i;
@@ -628,10 +628,11 @@ static bool mdiobus_prevent_c45_scan(struct mii_bus *bus)
continue;
oui = phydev->phy_id >> 10;
- if (oui == MICREL_OUI)
- return true;
+ if (oui == MICREL_OUI) {
+ bus->prevent_c45_access = true;
+ break;
+ }
}
- return false;
}
/**
@@ -652,7 +653,6 @@ int __mdiobus_register(struct mii_bus *bus, struct module *owner)
{
struct mdio_device *mdiodev;
struct gpio_desc *gpiod;
- bool prevent_c45_scan;
int i, err;
if (!bus || !bus->name)
@@ -724,9 +724,8 @@ int __mdiobus_register(struct mii_bus *bus, struct module *owner)
goto error;
}
- prevent_c45_scan = mdiobus_prevent_c45_scan(bus);
-
- if (!prevent_c45_scan && bus->read_c45) {
+ mdiobus_scan_for_broken_c45_access(bus);
+ if (!bus->prevent_c45_access && bus->read_c45) {
err = mdiobus_scan_bus_c45(bus);
if (err)
goto error;
@@ -410,6 +410,9 @@ struct mii_bus {
/** @phy_ignore_ta_mask: PHY addresses to ignore the TA/read failure */
u32 phy_ignore_ta_mask;
+ /** @prevent_c45_access: Don't do any C45 transactions on the bus */
+ unsigned prevent_c45_access:1;
+
/**
* @irq: An array of interrupts, each PHY's interrupt at the index
* matching its address
@@ -462,6 +465,8 @@ static inline struct mii_bus *devm_mdiobus_alloc(struct device *dev)
struct mii_bus *mdio_find_bus(const char *mdio_name);
struct phy_device *mdiobus_scan_c22(struct mii_bus *bus, int addr);
+void mdiobus_scan_for_broken_c45_access(struct mii_bus *bus);
+
#define PHY_INTERRUPT_DISABLED false
#define PHY_INTERRUPT_ENABLED true
The blacklist will also be used elsewhere in the kernel, e.g. in the DT scanning code. Make it a property of mii_bus and export the function. Signed-off-by: Michael Walle <mwalle@kernel.org> --- drivers/net/phy/mdio_bus.c | 17 ++++++++--------- include/linux/phy.h | 5 +++++ 2 files changed, 13 insertions(+), 9 deletions(-)