Hi Tony, On 09/08/2018 05:33 PM, Grygorii Strashko wrote: > am335x-evm has only one CPSW external port physically wired, but DT defines > 2 ext. ports. As result, PHY connection failure reported for the second > ext. port. > > Update DT to reflect am335x-evm board HW configuration, and, while here, > switch to use phy-handle instead of phy_id. > > Signed-off-by: Grygorii Strashko <grygorii.strashko@xxxxxx> > --- This is fix. Seems I messed up with subj, should be "[PATCH] ARM: dts: am335x-evm: fix number of cpsw slaves" > arch/arm/boot/dts/am335x-evm.dts | 12 ++++++------ > 1 file changed, 6 insertions(+), 6 deletions(-) > > diff --git a/arch/arm/boot/dts/am335x-evm.dts b/arch/arm/boot/dts/am335x-evm.dts > index 1356fd6..d71f52c 100644 > --- a/arch/arm/boot/dts/am335x-evm.dts > +++ b/arch/arm/boot/dts/am335x-evm.dts > @@ -713,6 +713,7 @@ > pinctrl-0 = <&cpsw_default>; > pinctrl-1 = <&cpsw_sleep>; > status = "okay"; > + slaves = <1>; > }; > > &davinci_mdio { > @@ -720,15 +721,14 @@ > pinctrl-0 = <&davinci_mdio_default>; > pinctrl-1 = <&davinci_mdio_sleep>; > status = "okay"; > -}; > > -&cpsw_emac0 { > - phy_id = <&davinci_mdio>, <0>; > - phy-mode = "rgmii-txid"; > + ethphy0: ethernet-phy@0 { > + reg = <0>; > + }; > }; > > -&cpsw_emac1 { > - phy_id = <&davinci_mdio>, <1>; > +&cpsw_emac0 { > + phy-handle = <ðphy0>; > phy-mode = "rgmii-txid"; > }; > > -- regards, -grygorii