All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH v2] arm64: dts: renesas: r8a77990: ebisu: Fix EthernetAVB phy mode to rgmii
@ 2018-12-14 15:58 ` Simon Horman
  0 siblings, 0 replies; 6+ messages in thread
From: Simon Horman @ 2018-12-14 15:58 UTC (permalink / raw)
  To: linux-renesas-soc
  Cc: linux-arm-kernel, Magnus Damm, Biju Das, Kazuya Mizuguchi,
	Takeshi Kihara, Simon Horman

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 doesn't support
on R-Car E3. This patch fixes EthernetAVB phy mode to rgmii.

v2 [Simon Horman]
* Simply drop phy-mode property from r8a77990-ebisu.dts
  as the default property for this for r8a77990, as set in
  r8a77990.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/r8a77990-ebisu.dts | 1 -
 1 file changed, 1 deletion(-)

 Based on renesas-devel-20181213-v4.20-rc6

diff --git a/arch/arm64/boot/dts/renesas/r8a77990-ebisu.dts b/arch/arm64/boot/dts/renesas/r8a77990-ebisu.dts
index 62bdddcbbae7..b7082c5bab60 100644
--- a/arch/arm64/boot/dts/renesas/r8a77990-ebisu.dts
+++ b/arch/arm64/boot/dts/renesas/r8a77990-ebisu.dts
@@ -248,7 +248,6 @@
 	pinctrl-names = "default";
 	renesas,no-ether-link;
 	phy-handle = <&phy0>;
-	phy-mode = "rgmii-txid";
 	status = "okay";
 
 	phy0: ethernet-phy@0 {
-- 
2.11.0


^ permalink raw reply related	[flat|nested] 6+ messages in thread

* [PATCH v2] arm64: dts: renesas: r8a77990: ebisu: Fix EthernetAVB phy mode to rgmii
@ 2018-12-14 15:58 ` Simon Horman
  0 siblings, 0 replies; 6+ messages in thread
From: Simon Horman @ 2018-12-14 15:58 UTC (permalink / raw)
  To: linux-renesas-soc
  Cc: Takeshi Kihara, Magnus Damm, Biju Das, Simon Horman,
	linux-arm-kernel, Kazuya Mizuguchi

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 doesn't support
on R-Car E3. This patch fixes EthernetAVB phy mode to rgmii.

v2 [Simon Horman]
* Simply drop phy-mode property from r8a77990-ebisu.dts
  as the default property for this for r8a77990, as set in
  r8a77990.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/r8a77990-ebisu.dts | 1 -
 1 file changed, 1 deletion(-)

 Based on renesas-devel-20181213-v4.20-rc6

diff --git a/arch/arm64/boot/dts/renesas/r8a77990-ebisu.dts b/arch/arm64/boot/dts/renesas/r8a77990-ebisu.dts
index 62bdddcbbae7..b7082c5bab60 100644
--- a/arch/arm64/boot/dts/renesas/r8a77990-ebisu.dts
+++ b/arch/arm64/boot/dts/renesas/r8a77990-ebisu.dts
@@ -248,7 +248,6 @@
 	pinctrl-names = "default";
 	renesas,no-ether-link;
 	phy-handle = <&phy0>;
-	phy-mode = "rgmii-txid";
 	status = "okay";
 
 	phy0: ethernet-phy@0 {
-- 
2.11.0


_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel

^ permalink raw reply related	[flat|nested] 6+ messages in thread

* Re: [PATCH v2] arm64: dts: renesas: r8a77990: ebisu: Fix EthernetAVB phy mode to rgmii
  2018-12-14 15:58 ` Simon Horman
@ 2018-12-18  9:25   ` Ulrich Hecht
  -1 siblings, 0 replies; 6+ messages in thread
From: Ulrich Hecht @ 2018-12-18  9:25 UTC (permalink / raw)
  To: Simon Horman, linux-renesas-soc
  Cc: linux-arm-kernel, Magnus Damm, Biju Das, Kazuya Mizuguchi,
	Takeshi Kihara


> On December 14, 2018 at 4:58 PM 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 doesn't support
> on R-Car E3. This patch fixes EthernetAVB phy mode to rgmii.
> 
> v2 [Simon Horman]
> * Simply drop phy-mode property from r8a77990-ebisu.dts
>   as the default property for this for r8a77990, as set in
>   r8a77990.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/r8a77990-ebisu.dts | 1 -
>  1 file changed, 1 deletion(-)
> 
>  Based on renesas-devel-20181213-v4.20-rc6
> 
> diff --git a/arch/arm64/boot/dts/renesas/r8a77990-ebisu.dts b/arch/arm64/boot/dts/renesas/r8a77990-ebisu.dts
> index 62bdddcbbae7..b7082c5bab60 100644
> --- a/arch/arm64/boot/dts/renesas/r8a77990-ebisu.dts
> +++ b/arch/arm64/boot/dts/renesas/r8a77990-ebisu.dts
> @@ -248,7 +248,6 @@
>  	pinctrl-names = "default";
>  	renesas,no-ether-link;
>  	phy-handle = <&phy0>;
> -	phy-mode = "rgmii-txid";
>  	status = "okay";
>  
>  	phy0: ethernet-phy@0 {
> -- 
> 2.11.0
>

Reviewed-by: Ulrich Hecht <uli+renesas@fpond.eu>

CU
Uli

^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: [PATCH v2] arm64: dts: renesas: r8a77990: ebisu: Fix EthernetAVB phy mode to rgmii
@ 2018-12-18  9:25   ` Ulrich Hecht
  0 siblings, 0 replies; 6+ messages in thread
From: Ulrich Hecht @ 2018-12-18  9:25 UTC (permalink / raw)
  To: Simon Horman, linux-renesas-soc
  Cc: Takeshi Kihara, Kazuya Mizuguchi, Magnus Damm, linux-arm-kernel,
	Biju Das


> On December 14, 2018 at 4:58 PM 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 doesn't support
> on R-Car E3. This patch fixes EthernetAVB phy mode to rgmii.
> 
> v2 [Simon Horman]
> * Simply drop phy-mode property from r8a77990-ebisu.dts
>   as the default property for this for r8a77990, as set in
>   r8a77990.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/r8a77990-ebisu.dts | 1 -
>  1 file changed, 1 deletion(-)
> 
>  Based on renesas-devel-20181213-v4.20-rc6
> 
> diff --git a/arch/arm64/boot/dts/renesas/r8a77990-ebisu.dts b/arch/arm64/boot/dts/renesas/r8a77990-ebisu.dts
> index 62bdddcbbae7..b7082c5bab60 100644
> --- a/arch/arm64/boot/dts/renesas/r8a77990-ebisu.dts
> +++ b/arch/arm64/boot/dts/renesas/r8a77990-ebisu.dts
> @@ -248,7 +248,6 @@
>  	pinctrl-names = "default";
>  	renesas,no-ether-link;
>  	phy-handle = <&phy0>;
> -	phy-mode = "rgmii-txid";
>  	status = "okay";
>  
>  	phy0: ethernet-phy@0 {
> -- 
> 2.11.0
>

Reviewed-by: Ulrich Hecht <uli+renesas@fpond.eu>

CU
Uli

_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel

^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: [PATCH v2] arm64: dts: renesas: r8a77990: ebisu: Fix EthernetAVB phy mode to rgmii
  2018-12-18  9:25   ` Ulrich Hecht
@ 2018-12-18 10:28     ` Simon Horman
  -1 siblings, 0 replies; 6+ messages in thread
From: Simon Horman @ 2018-12-18 10:28 UTC (permalink / raw)
  To: Ulrich Hecht
  Cc: linux-renesas-soc, linux-arm-kernel, Magnus Damm, Biju Das,
	Kazuya Mizuguchi, Takeshi Kihara

On Tue, Dec 18, 2018 at 10:25:09AM +0100, Ulrich Hecht wrote:
> 
> > On December 14, 2018 at 4:58 PM 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 doesn't support
> > on R-Car E3. This patch fixes EthernetAVB phy mode to rgmii.
> > 
> > v2 [Simon Horman]
> > * Simply drop phy-mode property from r8a77990-ebisu.dts
> >   as the default property for this for r8a77990, as set in
> >   r8a77990.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/r8a77990-ebisu.dts | 1 -
> >  1 file changed, 1 deletion(-)
> > 
> >  Based on renesas-devel-20181213-v4.20-rc6
> > 
> > diff --git a/arch/arm64/boot/dts/renesas/r8a77990-ebisu.dts b/arch/arm64/boot/dts/renesas/r8a77990-ebisu.dts
> > index 62bdddcbbae7..b7082c5bab60 100644
> > --- a/arch/arm64/boot/dts/renesas/r8a77990-ebisu.dts
> > +++ b/arch/arm64/boot/dts/renesas/r8a77990-ebisu.dts
> > @@ -248,7 +248,6 @@
> >  	pinctrl-names = "default";
> >  	renesas,no-ether-link;
> >  	phy-handle = <&phy0>;
> > -	phy-mode = "rgmii-txid";
> >  	status = "okay";
> >  
> >  	phy0: ethernet-phy@0 {
> > -- 
> > 2.11.0
> >
> 
> Reviewed-by: Ulrich Hecht <uli+renesas@fpond.eu>

Thanks, applied for v4.22.

^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: [PATCH v2] arm64: dts: renesas: r8a77990: ebisu: Fix EthernetAVB phy mode to rgmii
@ 2018-12-18 10:28     ` Simon Horman
  0 siblings, 0 replies; 6+ messages in thread
From: Simon Horman @ 2018-12-18 10:28 UTC (permalink / raw)
  To: Ulrich Hecht
  Cc: Takeshi Kihara, Magnus Damm, Biju Das, linux-renesas-soc,
	linux-arm-kernel, Kazuya Mizuguchi

On Tue, Dec 18, 2018 at 10:25:09AM +0100, Ulrich Hecht wrote:
> 
> > On December 14, 2018 at 4:58 PM 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 doesn't support
> > on R-Car E3. This patch fixes EthernetAVB phy mode to rgmii.
> > 
> > v2 [Simon Horman]
> > * Simply drop phy-mode property from r8a77990-ebisu.dts
> >   as the default property for this for r8a77990, as set in
> >   r8a77990.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/r8a77990-ebisu.dts | 1 -
> >  1 file changed, 1 deletion(-)
> > 
> >  Based on renesas-devel-20181213-v4.20-rc6
> > 
> > diff --git a/arch/arm64/boot/dts/renesas/r8a77990-ebisu.dts b/arch/arm64/boot/dts/renesas/r8a77990-ebisu.dts
> > index 62bdddcbbae7..b7082c5bab60 100644
> > --- a/arch/arm64/boot/dts/renesas/r8a77990-ebisu.dts
> > +++ b/arch/arm64/boot/dts/renesas/r8a77990-ebisu.dts
> > @@ -248,7 +248,6 @@
> >  	pinctrl-names = "default";
> >  	renesas,no-ether-link;
> >  	phy-handle = <&phy0>;
> > -	phy-mode = "rgmii-txid";
> >  	status = "okay";
> >  
> >  	phy0: ethernet-phy@0 {
> > -- 
> > 2.11.0
> >
> 
> Reviewed-by: Ulrich Hecht <uli+renesas@fpond.eu>

Thanks, applied for v4.22.

_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel

^ permalink raw reply	[flat|nested] 6+ messages in thread

end of thread, other threads:[~2018-12-18 10:29 UTC | newest]

Thread overview: 6+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2018-12-14 15:58 [PATCH v2] arm64: dts: renesas: r8a77990: ebisu: Fix EthernetAVB phy mode to rgmii Simon Horman
2018-12-14 15:58 ` Simon Horman
2018-12-18  9:25 ` Ulrich Hecht
2018-12-18  9:25   ` Ulrich Hecht
2018-12-18 10:28   ` Simon Horman
2018-12-18 10:28     ` Simon Horman

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.