| Message ID | 20200724214221.28125-1-grygorii.strashko@ti.com |
|---|---|
| State | New |
| Headers | show |
| Series | [RESEND] ARM: dts: keystone-k2g-evm: fix rgmii phy-mode for ksz9031 phy | expand |
On Fri, Jul 24, 2020 at 11:57 PM santosh.shilimkar@oracle.com <santosh.shilimkar@oracle.com> wrote: > On 7/24/20 2:42 PM, Grygorii Strashko wrote: > > Since commit bcf3440c6dd7 ("net: phy: micrel: add phy-mode support for the > > KSZ9031 PHY") the networking is broken on keystone-k2g-evm board. > > > > The above board have phy-mode = "rgmii-id" and it is worked before because > > KSZ9031 PHY started with default RGMII internal delays configuration (TX > > off, RX on 1.2 ns) and MAC provided TX delay by default. > > After above commit, the KSZ9031 PHY starts handling phy mode properly and > > enables both RX and TX delays, as result networking is become broken. > > > > Fix it by switching to phy-mode = "rgmii-rxid" to reflect previous > > behavior. > > > > Cc: Oleksij Rempel <o.rempel@pengutronix.de> > > Cc: Andrew Lunn <andrew@lunn.ch> > > Cc: Philippe Schenker <philippe.schenker@toradex.com> > > Fixes: bcf3440c6dd7 ("net: phy: micrel: add phy-mode support for the KSZ9031 PHY") > > Signed-off-by: Grygorii Strashko <grygorii.strashko@ti.com> > > --- > > Fix for one more broken TI board with KSZ9031 PHY. > Can you please apply this patch to your v5.8 fixes branch and send it > upstream ? Without the fix K2G EVM board is broken with v5.8. > > Am hoping you can pick this up with pull request since it just one > patch. I've applied it now, but would point out that it's generally better if you could forward the patch to soc@kernel.org with your Signed-off-by if you come across a similar patch again. That way it ends up in patchwork, and we are more likely to pick it up quickly. Arnd
diff --git a/arch/arm/boot/dts/keystone-k2g-evm.dts b/arch/arm/boot/dts/keystone-k2g-evm.dts index db640bab8c1d..8b3d64c913d8 100644 --- a/arch/arm/boot/dts/keystone-k2g-evm.dts +++ b/arch/arm/boot/dts/keystone-k2g-evm.dts @@ -402,7 +402,7 @@ &gbe0 { phy-handle = <ðphy0>; - phy-mode = "rgmii-id"; + phy-mode = "rgmii-rxid"; status = "okay"; };
Since commit bcf3440c6dd7 ("net: phy: micrel: add phy-mode support for the KSZ9031 PHY") the networking is broken on keystone-k2g-evm board. The above board have phy-mode = "rgmii-id" and it is worked before because KSZ9031 PHY started with default RGMII internal delays configuration (TX off, RX on 1.2 ns) and MAC provided TX delay by default. After above commit, the KSZ9031 PHY starts handling phy mode properly and enables both RX and TX delays, as result networking is become broken. Fix it by switching to phy-mode = "rgmii-rxid" to reflect previous behavior. Cc: Oleksij Rempel <o.rempel@pengutronix.de> Cc: Andrew Lunn <andrew@lunn.ch> Cc: Philippe Schenker <philippe.schenker@toradex.com> Fixes: bcf3440c6dd7 ("net: phy: micrel: add phy-mode support for the KSZ9031 PHY") Signed-off-by: Grygorii Strashko <grygorii.strashko@ti.com> --- Fix for one more broken TI board with KSZ9031 PHY. arch/arm/boot/dts/keystone-k2g-evm.dts | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) -- 2.17.1