Re: [PATCH] arm64: dts: allwinner: Pine H64: Enable both RGMII RX/TX delay

[Maxime Ripard]

On Mon, Oct 19, 2020 at 06:34:49AM +, Corentin Labbe wrote:
> Since commit bbc4d71d6354 ("net: phy: realtek: fix rtl8211e rx/tx delay 
> config"),
> the network is unusable on PineH64 model A.
> 
> This is due to phy-mode incorrectly set to rgmii instead of rgmii-id.
> 
> Fixes: 729e1ffcf47e ("arm64: allwinner: h6: add support for the Ethernet on 
> Pine H64")
> Signed-off-by: Corentin Labbe 

Queued as a fix for 5.10, thanks!
Maxime


signature.asc
Description: PGP signature

[PATCH] arm64: dts: allwinner: Pine H64: Enable both RGMII RX/TX delay

[Corentin Labbe]

Since commit bbc4d71d6354 ("net: phy: realtek: fix rtl8211e rx/tx delay 
config"),
the network is unusable on PineH64 model A.

This is due to phy-mode incorrectly set to rgmii instead of rgmii-id.

Fixes: 729e1ffcf47e ("arm64: allwinner: h6: add support for the Ethernet on 
Pine H64")
Signed-off-by: Corentin Labbe 
---
 arch/arm64/boot/dts/allwinner/sun50i-h6-pine-h64.dts | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/arch/arm64/boot/dts/allwinner/sun50i-h6-pine-h64.dts 
b/arch/arm64/boot/dts/allwinner/sun50i-h6-pine-h64.dts
index af85b2074867..961732c52aa0 100644
--- a/arch/arm64/boot/dts/allwinner/sun50i-h6-pine-h64.dts
+++ b/arch/arm64/boot/dts/allwinner/sun50i-h6-pine-h64.dts
@@ -100,7 +100,7 @@  {
  {
pinctrl-names = "default";
pinctrl-0 = <_rgmii_pins>;
-   phy-mode = "rgmii";
+   phy-mode = "rgmii-id";
phy-handle = <_rgmii_phy>;
phy-supply = <_gmac_3v3>;
allwinner,rx-delay-ps = <200>;
-- 
2.26.2