Hi Andrew, > -----Original Message----- > From: Andrew Lunn [mailto:and...@lunn.ch] > 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; > devicet...@vger.kernel.org; netdev@vger.kernel.org; linux- > ker...@vger.kernel.org; bcm-kernel-feedback-l...@broadcom.com; linux-arm- > ker...@lists.infradead.org > 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
