2022-08-02 14:43:40

by Vladimir Oltean

[permalink] [raw]
Subject: [PATCH] arm64: dts: ls1028a-qds-65bb: don't use in-band autoneg for 2500base-x

The Lynx PCS integrated with ENETC port 0 does not support in-band
autoneg for the 2500base-x SERDES protocol, and prints errors from its
phylink methods. Furthermore, the AQR112 card used for these boards does
not expect in-band autoneg either. So delete the extraneous property.

Fixes: e426d63e752b ("arm64: dts: ls1028a-qds: add overlays for various serdes protocols")
Signed-off-by: Vladimir Oltean <[email protected]>
---
arch/arm64/boot/dts/freescale/fsl-ls1028a-qds-65bb.dts | 1 -
1 file changed, 1 deletion(-)

diff --git a/arch/arm64/boot/dts/freescale/fsl-ls1028a-qds-65bb.dts b/arch/arm64/boot/dts/freescale/fsl-ls1028a-qds-65bb.dts
index 40d34c8384a5..b949cac03742 100644
--- a/arch/arm64/boot/dts/freescale/fsl-ls1028a-qds-65bb.dts
+++ b/arch/arm64/boot/dts/freescale/fsl-ls1028a-qds-65bb.dts
@@ -25,7 +25,6 @@ slot1_sgmii: ethernet-phy@2 {
&enetc_port0 {
phy-handle = <&slot1_sgmii>;
phy-mode = "2500base-x";
- managed = "in-band-status";
status = "okay";
};

--
2.34.1



2022-08-21 13:57:09

by Shawn Guo

[permalink] [raw]
Subject: Re: [PATCH] arm64: dts: ls1028a-qds-65bb: don't use in-band autoneg for 2500base-x

On Tue, Aug 02, 2022 at 04:50:06PM +0300, Vladimir Oltean wrote:
> The Lynx PCS integrated with ENETC port 0 does not support in-band
> autoneg for the 2500base-x SERDES protocol, and prints errors from its
> phylink methods. Furthermore, the AQR112 card used for these boards does
> not expect in-band autoneg either. So delete the extraneous property.
>
> Fixes: e426d63e752b ("arm64: dts: ls1028a-qds: add overlays for various serdes protocols")
> Signed-off-by: Vladimir Oltean <[email protected]>

Applied, thanks!