Hi Andrew, > -----Original Message----- > From: Andrew Lunn [mailto:andrew@xxxxxxx] > Sent: 01 June 2016 18:32 > To: Pramod Kumar > Cc: Rob Herring; Pawel Moll; Mark Rutland; Ian Campbell; Kumar Gala; Catalin > Marinas; Will Deacon; Kishon Vijay Abraham I; David S. Miller; > devicetree@xxxxxxxxxxxxxxx; netdev@xxxxxxxxxxxxxxx; linux- > kernel@xxxxxxxxxxxxxxx; bcm-kernel-feedback-list@xxxxxxxxxxxx; linux-arm- > kernel@xxxxxxxxxxxxxxxxxxx > Subject: Re: [PATCH v2 1/7] mdio:mux: Enhanced MDIO mux framework for > integrated multiplexers > > > diff --git a/drivers/of/of_mdio.c b/drivers/of/of_mdio.c index > > 8453f08..cf40e7a 100644 > > --- a/drivers/of/of_mdio.c > > +++ b/drivers/of/of_mdio.c > > @@ -225,6 +225,9 @@ int of_mdiobus_register(struct mii_bus *mdio, struct > device_node *np) > > if (rc) > > return rc; > > > > + if (of_property_read_bool(np, "mdio-integrated-mux")) > > + return 0; > > + > > Hi > > Rather than doing this, did you try setting bus->phy_mask = 0 and calling > mdiobus_register(), rather than of_mdiobus_regsiter() for the parent bus? > After setting bus->phy_mask = ~0 and using mdiobus_regsiter() I could get rid of newly added Property. It works for me as I find same has been done inside of of_mdiobus_regsiter(). Thanks for pointing out this. > Andrew -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html