Re: [PATCH net-next 1/8] net: dsa: mv88e6xxx: Implement external MDIO bus on mv88e6390

2017-01-21 Thread Andrew Lunn
On Fri, Jan 20, 2017 at 07:04:35PM -0500, Vivien Didelot wrote: > Hi Andrew, > > Andrew Lunn writes: > > > The mv88e6390 has two MDIO busses. The internal MDIO bus is used for > > the internal PHYs. The external MDIO can be used for external PHYs. > > The external MDIO bus will be instantiated i

Re: [PATCH net-next 1/8] net: dsa: mv88e6xxx: Implement external MDIO bus on mv88e6390

2017-01-21 Thread Andrew Lunn
> This looks fine, although I am not clear why we cannot utilize a > standard representation of a MDIO bus (with PHY devices as child nodes) > which has a specific compatible string, e.g: > marvell,mv88e6390-external-mdio, and that is a child node of the 6390 > Ethernet switch itself, something lik

Re: [PATCH net-next 1/8] net: dsa: mv88e6xxx: Implement external MDIO bus on mv88e6390

2017-01-20 Thread Vivien Didelot
Hi Andrew, Andrew Lunn writes: > The mv88e6390 has two MDIO busses. The internal MDIO bus is used for > the internal PHYs. The external MDIO can be used for external PHYs. > The external MDIO bus will be instantiated if there is an > "mdio-external" node in the device tree. Thanks for pushing t

Re: [PATCH net-next 1/8] net: dsa: mv88e6xxx: Implement external MDIO bus on mv88e6390

2017-01-20 Thread Florian Fainelli
On 01/20/2017 03:30 PM, Andrew Lunn wrote: > The mv88e6390 has two MDIO busses. The internal MDIO bus is used for > the internal PHYs. The external MDIO can be used for external PHYs. > The external MDIO bus will be instantiated if there is an > "mdio-external" node in the device tree. This looks

[PATCH net-next 1/8] net: dsa: mv88e6xxx: Implement external MDIO bus on mv88e6390

2017-01-20 Thread Andrew Lunn
The mv88e6390 has two MDIO busses. The internal MDIO bus is used for the internal PHYs. The external MDIO can be used for external PHYs. The external MDIO bus will be instantiated if there is an "mdio-external" node in the device tree. Signed-off-by: Andrew Lunn --- .../devicetree/bindings/net/d