[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <a91d2bad-b794-fe07-679a-e5096aa5ace8@oracle.com>
Date: Fri, 24 Jul 2020 14:55:19 -0700
From: "santosh.shilimkar@...cle.com" <santosh.shilimkar@...cle.com>
To: "arm@...nel.org" <arm@...nel.org>, Arnd Bergmann <arnd@...db.de>,
Olof Johansson <olof@...om.net>
Cc: Grygorii Strashko <grygorii.strashko@...com>,
Murali Karicheri <m-karicheri2@...com>,
Santosh Shilimkar <ssantosh@...nel.org>,
Sekhar Nori <nsekhar@...com>,
linux-arm-kernel@...ts.infradead.org,
"David S . Miller" <davem@...emloft.net>,
linux-kernel@...r.kernel.org, netdev@...r.kernel.org,
devicetree@...r.kernel.org,
Oleksij Rempel <o.rempel@...gutronix.de>,
Andrew Lunn <andrew@...n.ch>,
Philippe Schenker <philippe.schenker@...adex.com>
Subject: Re: [RESEND PATCH] ARM: dts: keystone-k2g-evm: fix rgmii phy-mode for
ksz9031 phy
Hi Arnd, Olof,
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@...gutronix.de>
> Cc: Andrew Lunn <andrew@...n.ch>
> Cc: Philippe Schenker <philippe.schenker@...adex.com>
> Fixes: bcf3440c6dd7 ("net: phy: micrel: add phy-mode support for the KSZ9031 PHY")
> Signed-off-by: Grygorii Strashko <grygorii.strashko@...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.
Regards,
Santosh
Powered by blists - more mailing lists