| Message ID | 20210427094831.545521-2-sr@denx.de |
|---|---|
| State | Accepted |
| Commit | 1c817c27e66fb17448a71f59fc70b182f0795ce0 |
| Delegated to: | Stefan Roese |
| Headers | show |
| Series | [1/4] arm: mvebu: armada-3720-uDPU.dts: Change back to phy-mode "2500base-x" | expand |
On 27.04.21 11:48, Stefan Roese wrote: > Because of the incorrectly supported SGMII_2500 mode, this patch > disables eth2 for now until this issue will be fixed in mainline. > > Also fix an incorrect comment. > > Signed-off-by: Stefan Roese <sr@denx.de> > Cc: Konstantin Porotchkin <kostap@marvell.com> > Cc: Stefan Chulski <stefanc@marvell.com> > Cc: Nadav Haklai <nadavh@marvell.com> > Cc: Marek Behun <marek.behun@nic.cz> > --- > This patch is targeted on-top of the latest Marvell SERDES, mvpp2 and > PHY patches to resolve the ongoing discussion of the incorrect usage of > SGMII_2500 for now. > > arch/arm/dts/cn9130-crb.dtsi | 4 +--- > 1 file changed, 1 insertion(+), 3 deletions(-) Applied to u-boot-marvell/master Thanks, Stefan > diff --git a/arch/arm/dts/cn9130-crb.dtsi b/arch/arm/dts/cn9130-crb.dtsi > index 657a934764ae..78b43b449b3e 100644 > --- a/arch/arm/dts/cn9130-crb.dtsi > +++ b/arch/arm/dts/cn9130-crb.dtsi > @@ -232,7 +232,6 @@ > }; > > &cp0_eth0 { > - /* Disable it for now, as mainline does not support this IF yet */ > status = "okay"; > phy-mode = "sfi"; > }; > @@ -247,7 +246,6 @@ > > &cp0_eth2 { > /* Disable it for now, as mainline does not support this IF yet */ > - status = "okay"; > + status = "disabled"; > phy = <&nbaset_phy0>; > - phy-mode = "sgmii-2500"; > }; > Viele Grüße, Stefan
diff --git a/arch/arm/dts/cn9130-crb.dtsi b/arch/arm/dts/cn9130-crb.dtsi index 657a934764ae..78b43b449b3e 100644 --- a/arch/arm/dts/cn9130-crb.dtsi +++ b/arch/arm/dts/cn9130-crb.dtsi @@ -232,7 +232,6 @@ }; &cp0_eth0 { - /* Disable it for now, as mainline does not support this IF yet */ status = "okay"; phy-mode = "sfi"; }; @@ -247,7 +246,6 @@ &cp0_eth2 { /* Disable it for now, as mainline does not support this IF yet */ - status = "okay"; + status = "disabled"; phy = <&nbaset_phy0>; - phy-mode = "sgmii-2500"; };
Because of the incorrectly supported SGMII_2500 mode, this patch disables eth2 for now until this issue will be fixed in mainline. Also fix an incorrect comment. Signed-off-by: Stefan Roese <sr@denx.de> Cc: Konstantin Porotchkin <kostap@marvell.com> Cc: Stefan Chulski <stefanc@marvell.com> Cc: Nadav Haklai <nadavh@marvell.com> Cc: Marek Behun <marek.behun@nic.cz> --- This patch is targeted on-top of the latest Marvell SERDES, mvpp2 and PHY patches to resolve the ongoing discussion of the incorrect usage of SGMII_2500 for now. arch/arm/dts/cn9130-crb.dtsi | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-)