On Mon, Mar 02, 2020 at 12:41:38AM +0800, Dajun Jin wrote: > when registers a phy_device successful, should terminate the loop > or the phy_device would be registered in other addr. > > Signed-off-by: Dajun Jin <adajunjin@xxxxxxxxx> > --- > drivers/of/of_mdio.c | 1 + > 1 file changed, 1 insertion(+) > > diff --git a/drivers/of/of_mdio.c b/drivers/of/of_mdio.c > index 8270bbf505fb..9f982c0627a0 100644 > --- a/drivers/of/of_mdio.c > +++ b/drivers/of/of_mdio.c > @@ -306,6 +306,7 @@ int of_mdiobus_register(struct mii_bus *mdio, struct device_node *np) > rc = of_mdiobus_register_phy(mdio, child, addr); > if (rc && rc != -ENODEV) > goto unregister; > + break; > } > } > } Hi Dajun What problem are you seeing? You explanation needs to be better. I'm guessing you have two or more PHYs on the bus, without reg properties? Andrew