| Message ID | 20190213103255.1265-1-horms+renesas@verge.net.au (mailing list archive) |
|---|---|
| State | Accepted |
| Commit | 05f1d882d28b871f0c32a6307c7777fa834b2541 |
| Delegated to: | Simon Horman |
| Headers | show |
| Series | [PATCH/RFT] arm64: dts: renesas: r8a77995: draak: Fix EthernetAVB phy mode to rgmii | expand |
> On February 13, 2019 at 11:32 AM Simon Horman <horms+renesas@verge.net.au> wrote: > > > From: Kazuya Mizuguchi <kazuya.mizuguchi.ks@renesas.com> > > According to the R-Car Gen3 Hardware Manual Errata for Rev 1.00 of > August 24, 2018, the TX clock internal Delay mode does'nt support > on R-Car D3. This patch fixes EthernetAVB phy mode to rgmii. > > This is achieved by simply dropping the phy-mode property from > r8a77995-draak.dts as the default property for this for r8a77995, > as set in r8a77995.dtsi, is "rgmii". > > Signed-off-by: Kazuya Mizuguchi <kazuya.mizuguchi.ks@renesas.com> > Signed-off-by: Takeshi Kihara <takeshi.kihara.df@renesas.com> > Signed-off-by: Simon Horman <horms+renesas@verge.net.au> > --- > arch/arm64/boot/dts/renesas/r8a77995-draak.dts | 1 - > 1 file changed, 1 deletion(-) > > diff --git a/arch/arm64/boot/dts/renesas/r8a77995-draak.dts b/arch/arm64/boot/dts/renesas/r8a77995-draak.dts > index db2bed1751b8..bfa260c9f78c 100644 > --- a/arch/arm64/boot/dts/renesas/r8a77995-draak.dts > +++ b/arch/arm64/boot/dts/renesas/r8a77995-draak.dts > @@ -168,7 +168,6 @@ > pinctrl-names = "default"; > renesas,no-ether-link; > phy-handle = <&phy0>; > - phy-mode = "rgmii-txid"; > status = "okay"; > > phy0: ethernet-phy@0 { > -- > 2.11.0 > Works for me. Tested-by: Ulrich Hecht <uli+renesas@fpond.eu> CU Uli
On Tue, Mar 05, 2019 at 01:06:52PM +0100, Ulrich Hecht wrote: > > > On February 13, 2019 at 11:32 AM Simon Horman <horms+renesas@verge.net.au> wrote: > > > > > > From: Kazuya Mizuguchi <kazuya.mizuguchi.ks@renesas.com> > > > > According to the R-Car Gen3 Hardware Manual Errata for Rev 1.00 of > > August 24, 2018, the TX clock internal Delay mode does'nt support > > on R-Car D3. This patch fixes EthernetAVB phy mode to rgmii. > > > > This is achieved by simply dropping the phy-mode property from > > r8a77995-draak.dts as the default property for this for r8a77995, > > as set in r8a77995.dtsi, is "rgmii". > > > > Signed-off-by: Kazuya Mizuguchi <kazuya.mizuguchi.ks@renesas.com> > > Signed-off-by: Takeshi Kihara <takeshi.kihara.df@renesas.com> > > Signed-off-by: Simon Horman <horms+renesas@verge.net.au> > > --- > > arch/arm64/boot/dts/renesas/r8a77995-draak.dts | 1 - > > 1 file changed, 1 deletion(-) > > > > diff --git a/arch/arm64/boot/dts/renesas/r8a77995-draak.dts b/arch/arm64/boot/dts/renesas/r8a77995-draak.dts > > index db2bed1751b8..bfa260c9f78c 100644 > > --- a/arch/arm64/boot/dts/renesas/r8a77995-draak.dts > > +++ b/arch/arm64/boot/dts/renesas/r8a77995-draak.dts > > @@ -168,7 +168,6 @@ > > pinctrl-names = "default"; > > renesas,no-ether-link; > > phy-handle = <&phy0>; > > - phy-mode = "rgmii-txid"; > > status = "okay"; > > > > phy0: ethernet-phy@0 { > > -- > > 2.11.0 > > > > Works for me. > > Tested-by: Ulrich Hecht <uli+renesas@fpond.eu> Thanks, applied for v5.2.
diff --git a/arch/arm64/boot/dts/renesas/r8a77995-draak.dts b/arch/arm64/boot/dts/renesas/r8a77995-draak.dts index db2bed1751b8..bfa260c9f78c 100644 --- a/arch/arm64/boot/dts/renesas/r8a77995-draak.dts +++ b/arch/arm64/boot/dts/renesas/r8a77995-draak.dts @@ -168,7 +168,6 @@ pinctrl-names = "default"; renesas,no-ether-link; phy-handle = <&phy0>; - phy-mode = "rgmii-txid"; status = "okay"; phy0: ethernet-phy@0 {