2020-10-19 10:23:46

by Corentin LABBE

[permalink] [raw]

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

---

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 <[email protected]>
---
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 @@ &ehci3 {
&emac {
pinctrl-names = "default";
pinctrl-0 = <&ext_rgmii_pins>;
- phy-mode = "rgmii";
+ phy-mode = "rgmii-id";
phy-handle = <&ext_rgmii_phy>;
phy-supply = <&reg_gmac_3v3>;
allwinner,rx-delay-ps = <200>;
--
2.26.2

2020-10-22 18:07:09

by Maxime Ripard

[permalink] [raw]

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

---

On Mon, Oct 19, 2020 at 06:34:49AM +0000, 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 <[email protected]>

Queued as a fix for 5.10, thanks!
Maxime

---

Attachments:

(No filename) (468.00 B)
signature.asc (235.00 B)
Download all attachments