[PATCH] arm64: dts: marvell: 8040-mcbin: Enable 1GB Ethernet
Andrew Lunn
andrew at lunn.ch
Tue Jun 6 05:10:25 PDT 2017
On Tue, Jun 06, 2017 at 09:21:58AM +0100, Marc Zyngier wrote:
> Hi Andrew,
>
> On 05/06/17 22:16, Andrew Lunn wrote:
> >> +&cps_eth1 {
> >> + status = "okay";
> >> + phy = <&phy0>;
> >> + phy-mode = "sgmii";
> >> +};
> >
> > Hi Marc
> >
> > Documentation/devicetree/bindings/net/ethernet.txt says:
> >
> > - phy-handle: phandle, specifies a reference to a node representing a PHY
> > device; this property is described in ePAPR and so preferred;
> > - phy: the same as "phy-handle" property, not recommended for new bindings.
>
> Happy to amend the patch, though armada-8040-db.dts is already using
> "phy" (and not "phy-handle"). Should that one be fixed as well in the
> name of consistency? Also,
> Documentation/devicetree/bindings/net/marvell-pp2.txt only mentions the
> "phy" property. If feels a bit odd not to at least recommend the new
> property in the binding documentation.
Ah. As Russell pointed out, parsing this property is done by the
driver, not a central helper. mvpp2 only supports phy, not phy-handle.
This is not a new binding, it has been around since the middle of
2014. So "phy" is O.K.
Sometime in the future, it would be nice to centralise the parsing of
this and phy-mode.
Andrew
More information about the linux-arm-kernel
mailing list