[PATCH 0/2] arm64: dts: renesas: r8a77995 and draak: add support EthernetAVB

[PATCH 0/2] arm64: dts: renesas: r8a77995 and draak: add support EthernetAVB

[Yoshihiro Shimoda]

This patch is based on the renesas-drivers.git /
renesas-drivers-2017-09-05-v4.13 tag.

Yoshihiro Shimoda (2):
  arm64: dts: renesas: r8a77995: Add EthernetAVB device node
  arm64: dts: renesas: r8a77995: draak: enable EthernetAVB

 arch/arm64/boot/dts/renesas/r8a77995-draak.dts | 25 ++++++++++++++
 arch/arm64/boot/dts/renesas/r8a77995.dtsi      | 45 ++++++++++++++++++++++++++
 2 files changed, 70 insertions(+)

-- 
1.9.1

--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

[PATCH 0/2] arm64: dts: renesas: r8a77995 and draak: add support EthernetAVB

[Yoshihiro Shimoda]

This patch is based on the renesas-drivers.git /
renesas-drivers-2017-09-05-v4.13 tag.

Yoshihiro Shimoda (2):
  arm64: dts: renesas: r8a77995: Add EthernetAVB device node
  arm64: dts: renesas: r8a77995: draak: enable EthernetAVB

 arch/arm64/boot/dts/renesas/r8a77995-draak.dts | 25 ++++++++++++++
 arch/arm64/boot/dts/renesas/r8a77995.dtsi      | 45 ++++++++++++++++++++++++++
 2 files changed, 70 insertions(+)

-- 
1.9.1

[PATCH 1/2] arm64: dts: renesas: r8a77995: Add EthernetAVB device node

[Yoshihiro Shimoda]

This patch adds EthernetAVB device node for r8a77995.

Signed-off-by: Yoshihiro Shimoda <yoshihiro.shimoda.uh@renesas.com>
---
 arch/arm64/boot/dts/renesas/r8a77995.dtsi | 45 +++++++++++++++++++++++++++++++
 1 file changed, 45 insertions(+)

diff --git a/arch/arm64/boot/dts/renesas/r8a77995.dtsi b/arch/arm64/boot/dts/renesas/r8a77995.dtsi
index d7756256..72d04d7 100644
--- a/arch/arm64/boot/dts/renesas/r8a77995.dtsi
+++ b/arch/arm64/boot/dts/renesas/r8a77995.dtsi
@@ -251,6 +251,51 @@
 			resets = <&cpg 906>;
 		};
 
+		avb: ethernet@e6800000 {
+			compatible = "renesas,etheravb-r8a77995",
+				     "renesas,etheravb-rcar-gen3";
+			reg = <0 0xe6800000 0 0x800>, <0 0xe6a00000 0 0x10000>;
+			interrupts = <GIC_SPI 39 IRQ_TYPE_LEVEL_HIGH>,
+				     <GIC_SPI 40 IRQ_TYPE_LEVEL_HIGH>,
+				     <GIC_SPI 41 IRQ_TYPE_LEVEL_HIGH>,
+				     <GIC_SPI 42 IRQ_TYPE_LEVEL_HIGH>,
+				     <GIC_SPI 43 IRQ_TYPE_LEVEL_HIGH>,
+				     <GIC_SPI 44 IRQ_TYPE_LEVEL_HIGH>,
+				     <GIC_SPI 45 IRQ_TYPE_LEVEL_HIGH>,
+				     <GIC_SPI 46 IRQ_TYPE_LEVEL_HIGH>,
+				     <GIC_SPI 47 IRQ_TYPE_LEVEL_HIGH>,
+				     <GIC_SPI 48 IRQ_TYPE_LEVEL_HIGH>,
+				     <GIC_SPI 49 IRQ_TYPE_LEVEL_HIGH>,
+				     <GIC_SPI 50 IRQ_TYPE_LEVEL_HIGH>,
+				     <GIC_SPI 51 IRQ_TYPE_LEVEL_HIGH>,
+				     <GIC_SPI 52 IRQ_TYPE_LEVEL_HIGH>,
+				     <GIC_SPI 53 IRQ_TYPE_LEVEL_HIGH>,
+				     <GIC_SPI 54 IRQ_TYPE_LEVEL_HIGH>,
+				     <GIC_SPI 55 IRQ_TYPE_LEVEL_HIGH>,
+				     <GIC_SPI 56 IRQ_TYPE_LEVEL_HIGH>,
+				     <GIC_SPI 57 IRQ_TYPE_LEVEL_HIGH>,
+				     <GIC_SPI 58 IRQ_TYPE_LEVEL_HIGH>,
+				     <GIC_SPI 59 IRQ_TYPE_LEVEL_HIGH>,
+				     <GIC_SPI 60 IRQ_TYPE_LEVEL_HIGH>,
+				     <GIC_SPI 61 IRQ_TYPE_LEVEL_HIGH>,
+				     <GIC_SPI 62 IRQ_TYPE_LEVEL_HIGH>,
+				     <GIC_SPI 63 IRQ_TYPE_LEVEL_HIGH>;
+			interrupt-names = "ch0", "ch1", "ch2", "ch3",
+					  "ch4", "ch5", "ch6", "ch7",
+					  "ch8", "ch9", "ch10", "ch11",
+					  "ch12", "ch13", "ch14", "ch15",
+					  "ch16", "ch17", "ch18", "ch19",
+					  "ch20", "ch21", "ch22", "ch23",
+					  "ch24";
+			clocks = <&cpg CPG_MOD 812>;
+			power-domains = <&sysc R8A77995_PD_ALWAYS_ON>;
+			resets = <&cpg 812>;
+			phy-mode = "rgmii-txid";
+			#address-cells = <1>;
+			#size-cells = <0>;
+			status = "disabled";
+		};
+
 		scif2: serial@e6e88000 {
 			compatible = "renesas,scif-r8a77995",
 				     "renesas,rcar-gen3-scif", "renesas,scif";
-- 
1.9.1

[PATCH 2/2] arm64: dts: renesas: r8a77995: draak: enable EthernetAVB

[Yoshihiro Shimoda]

This patch enables EthernetAVB for R-Car D3 draak board.

Signed-off-by: Yoshihiro Shimoda <yoshihiro.shimoda.uh-zM6kxYcvzFBBDgjK7y7TUQ@public.gmane.org>
---
 arch/arm64/boot/dts/renesas/r8a77995-draak.dts | 25 +++++++++++++++++++++++++
 1 file changed, 25 insertions(+)

diff --git a/arch/arm64/boot/dts/renesas/r8a77995-draak.dts b/arch/arm64/boot/dts/renesas/r8a77995-draak.dts
index 19c5462..9c5a790 100644
--- a/arch/arm64/boot/dts/renesas/r8a77995-draak.dts
+++ b/arch/arm64/boot/dts/renesas/r8a77995-draak.dts
@@ -11,6 +11,7 @@
 
 /dts-v1/;
 #include "r8a77995.dtsi"
+#include <dt-bindings/gpio/gpio.h>
 
 / {
 	model = "Renesas Draak board based on r8a77995";
@@ -18,6 +19,7 @@
 
 	aliases {
 		serial0 = &scif2;
+		ethernet0 = &avb;
 	};
 
 	chosen {
@@ -37,6 +39,14 @@
 };
 
 &pfc {
+	avb0_pins: avb {
+		mux {
+			groups = "avb0_link", "avb0_phy_int", "avb0_mdc",
+				 "avb0_mii";
+			function = "avb0";
+		};
+	};
+
 	scif2_pins: scif2 {
 		groups = "scif2_data";
 		function = "scif2";
@@ -44,6 +54,21 @@
 
 };
 
+&avb {
+	pinctrl-0 = <&avb0_pins>;
+	pinctrl-names = "default";
+	renesas,no-ether-link;
+	phy-handle = <&phy0>;
+	status = "okay";
+
+	phy0: ethernet-phy@0 {
+		rxc-skew-ps = <1500>;
+		reg = <0>;
+		interrupt-parent = <&gpio5>;
+		interrupts = <19 IRQ_TYPE_LEVEL_LOW>;
+	};
+};
+
 &scif2 {
 	pinctrl-0 = <&scif2_pins>;
 	pinctrl-names = "default";
-- 
1.9.1

--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

[PATCH 2/2] arm64: dts: renesas: r8a77995: draak: enable EthernetAVB

[Yoshihiro Shimoda]

This patch enables EthernetAVB for R-Car D3 draak board.

Signed-off-by: Yoshihiro Shimoda <yoshihiro.shimoda.uh@renesas.com>
---
 arch/arm64/boot/dts/renesas/r8a77995-draak.dts | 25 +++++++++++++++++++++++++
 1 file changed, 25 insertions(+)

diff --git a/arch/arm64/boot/dts/renesas/r8a77995-draak.dts b/arch/arm64/boot/dts/renesas/r8a77995-draak.dts
index 19c5462..9c5a790 100644
--- a/arch/arm64/boot/dts/renesas/r8a77995-draak.dts
+++ b/arch/arm64/boot/dts/renesas/r8a77995-draak.dts
@@ -11,6 +11,7 @@
 
 /dts-v1/;
 #include "r8a77995.dtsi"
+#include <dt-bindings/gpio/gpio.h>
 
 / {
 	model = "Renesas Draak board based on r8a77995";
@@ -18,6 +19,7 @@
 
 	aliases {
 		serial0 = &scif2;
+		ethernet0 = &avb;
 	};
 
 	chosen {
@@ -37,6 +39,14 @@
 };
 
 &pfc {
+	avb0_pins: avb {
+		mux {
+			groups = "avb0_link", "avb0_phy_int", "avb0_mdc",
+				 "avb0_mii";
+			function = "avb0";
+		};
+	};
+
 	scif2_pins: scif2 {
 		groups = "scif2_data";
 		function = "scif2";
@@ -44,6 +54,21 @@
 
 };
 
+&avb {
+	pinctrl-0 = <&avb0_pins>;
+	pinctrl-names = "default";
+	renesas,no-ether-link;
+	phy-handle = <&phy0>;
+	status = "okay";
+
+	phy0: ethernet-phy@0 {
+		rxc-skew-ps = <1500>;
+		reg = <0>;
+		interrupt-parent = <&gpio5>;
+		interrupts = <19 IRQ_TYPE_LEVEL_LOW>;
+	};
+};
+
 &scif2 {
 	pinctrl-0 = <&scif2_pins>;
 	pinctrl-names = "default";
-- 
1.9.1

Re: [PATCH 1/2] arm64: dts: renesas: r8a77995: Add EthernetAVB device node

[Geert Uytterhoeven]

On Wed, Sep 13, 2017 at 2:18 PM, Yoshihiro Shimoda
<yoshihiro.shimoda.uh-zM6kxYcvzFBBDgjK7y7TUQ@public.gmane.org> wrote:
> This patch adds EthernetAVB device node for r8a77995.
>
> Signed-off-by: Yoshihiro Shimoda <yoshihiro.shimoda.uh-zM6kxYcvzFBBDgjK7y7TUQ@public.gmane.org>

Reviewed-by: Geert Uytterhoeven <geert+renesas-gXvu3+zWzMSzQB+pC5nmwQ@public.gmane.org>

Gr{oetje,eeting}s,

                        Geert

--
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert-Td1EMuHUCqxL1ZNQvxDV9g@public.gmane.org

In personal conversations with technical people, I call myself a hacker. But
when I'm talking to journalists I just say "programmer" or something like that.
                                -- Linus Torvalds
--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Re: [PATCH 1/2] arm64: dts: renesas: r8a77995: Add EthernetAVB device node

[Geert Uytterhoeven]

On Wed, Sep 13, 2017 at 2:18 PM, Yoshihiro Shimoda
<yoshihiro.shimoda.uh@renesas.com> wrote:
> This patch adds EthernetAVB device node for r8a77995.
>
> Signed-off-by: Yoshihiro Shimoda <yoshihiro.shimoda.uh@renesas.com>

Reviewed-by: Geert Uytterhoeven <geert+renesas@glider.be>

Gr{oetje,eeting}s,

                        Geert

--
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@linux-m68k.org

In personal conversations with technical people, I call myself a hacker. But
when I'm talking to journalists I just say "programmer" or something like that.
                                -- Linus Torvalds

Re: [PATCH 2/2] arm64: dts: renesas: r8a77995: draak: enable EthernetAVB

[Geert Uytterhoeven]

Hi Shimoda-san,

On Wed, Sep 13, 2017 at 2:18 PM, Yoshihiro Shimoda
<yoshihiro.shimoda.uh@renesas.com> wrote:
> This patch enables EthernetAVB for R-Car D3 draak board.
>
> Signed-off-by: Yoshihiro Shimoda <yoshihiro.shimoda.uh@renesas.com>

Thanks for your patch!

> --- a/arch/arm64/boot/dts/renesas/r8a77995-draak.dts
> +++ b/arch/arm64/boot/dts/renesas/r8a77995-draak.dts

> @@ -37,6 +39,14 @@
>  };
>
>  &pfc {
> +       avb0_pins: avb {
> +               mux {
> +                       groups = "avb0_link", "avb0_phy_int", "avb0_mdc",
> +                                "avb0_mii";
> +                       function = "avb0";

This part depends on (an updated version of) "[PATCH 5/8] pinctrl: sh-pfc:
r8a77995: Add EthernetAVB pins, groups and functions", so it may change?

And we will probably have to add driver-strengths later, to avoid a
dependency on bootloader setup?

Apart from that:
Reviewed-by: Geert Uytterhoeven <geert+renesas@glider.be>

Gr{oetje,eeting}s,

                        Geert

--
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@linux-m68k.org

In personal conversations with technical people, I call myself a hacker. But
when I'm talking to journalists I just say "programmer" or something like that.
                                -- Linus Torvalds

RE: [PATCH 2/2] arm64: dts: renesas: r8a77995: draak: enable EthernetAVB

[Yoshihiro Shimoda]

[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #1: Type: text/plain; charset="utf-8", Size: 1871 bytes --]

Hi Geert-san,

> From: Geert Uytterhoeven
> Sent: Wednesday, September 13, 2017 11:55 PM
> 
> Hi Shimoda-san,
> 
> On Wed, Sep 13, 2017 at 2:18 PM, Yoshihiro Shimoda
> <yoshihiro.shimoda.uh@renesas.com> wrote:
> > This patch enables EthernetAVB for R-Car D3 draak board.
> >
> > Signed-off-by: Yoshihiro Shimoda <yoshihiro.shimoda.uh@renesas.com>
> 
> Thanks for your patch!
> 
> > --- a/arch/arm64/boot/dts/renesas/r8a77995-draak.dts
> > +++ b/arch/arm64/boot/dts/renesas/r8a77995-draak.dts
> 
> > @@ -37,6 +39,14 @@
> >  };
> >
> >  &pfc {
> > +       avb0_pins: avb {
> > +               mux {
> > +                       groups = "avb0_link", "avb0_phy_int", "avb0_mdc",
> > +                                "avb0_mii";
> > +                       function = "avb0";
> 
> This part depends on (an updated version of) "[PATCH 5/8] pinctrl: sh-pfc:
> r8a77995: Add EthernetAVB pins, groups and functions", so it may change?

Oops, I fotgot to submit v2 PFC patch yesterday...
Yes, it is changed like other gen3 SoCs. I submitted it now. 

> And we will probably have to add driver-strengths later, to avoid a
> dependency on bootloader setup?

Yes, I will do so, if needed.

> Apart from that:
> Reviewed-by: Geert Uytterhoeven <geert+renesas@glider.be>

Thank you for your review!

Best regards,
Yoshihiro Shimoda

> Gr{oetje,eeting}s,
> 
>                         Geert
> 
> --
> Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@linux-m68k.org
> 
> In personal conversations with technical people, I call myself a hacker. But
> when I'm talking to journalists I just say "programmer" or something like that.
>                                 -- Linus Torvalds
N‹§²æìr¸›yúèšØb²X¬¶Ç§vØ^–)Þº{.nÇ+‰·zøœzÚÞz)í…æèw*\x1fjg¬±¨\x1e¶‰šŽŠÝ¢j.ïÛ°\½½MŽúgjÌæa×\x02››–' ™©Þ¢¸\f¢·¦j:+v‰¨ŠwèjØm¶Ÿÿ¾\a«‘êçzZ+ƒùšŽŠÝ¢j"ú!¶i

RE: [PATCH 2/2] arm64: dts: renesas: r8a77995: draak: enable EthernetAVB

[Yoshihiro Shimoda]

Hi Geert-san,

> From: Geert Uytterhoeven
> Sent: Wednesday, September 13, 2017 11:55 PM
> 
> Hi Shimoda-san,
> 
> On Wed, Sep 13, 2017 at 2:18 PM, Yoshihiro Shimoda
> <yoshihiro.shimoda.uh@renesas.com> wrote:
> > This patch enables EthernetAVB for R-Car D3 draak board.
> >
> > Signed-off-by: Yoshihiro Shimoda <yoshihiro.shimoda.uh@renesas.com>
> 
> Thanks for your patch!
> 
> > --- a/arch/arm64/boot/dts/renesas/r8a77995-draak.dts
> > +++ b/arch/arm64/boot/dts/renesas/r8a77995-draak.dts
> 
> > @@ -37,6 +39,14 @@
> >  };
> >
> >  &pfc {
> > +       avb0_pins: avb {
> > +               mux {
> > +                       groups = "avb0_link", "avb0_phy_int", "avb0_mdc",
> > +                                "avb0_mii";
> > +                       function = "avb0";
> 
> This part depends on (an updated version of) "[PATCH 5/8] pinctrl: sh-pfc:
> r8a77995: Add EthernetAVB pins, groups and functions", so it may change?

Oops, I fotgot to submit v2 PFC patch yesterday...
Yes, it is changed like other gen3 SoCs. I submitted it now. 

> And we will probably have to add driver-strengths later, to avoid a
> dependency on bootloader setup?

Yes, I will do so, if needed.

> Apart from that:
> Reviewed-by: Geert Uytterhoeven <geert+renesas@glider.be>

Thank you for your review!

Best regards,
Yoshihiro Shimoda

> Gr{oetje,eeting}s,
> 
>                         Geert
> 
> --
> Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@linux-m68k.org
> 
> In personal conversations with technical people, I call myself a hacker. But
> when I'm talking to journalists I just say "programmer" or something like that.
>                                 -- Linus Torvalds

Re: [PATCH 1/2] arm64: dts: renesas: r8a77995: Add EthernetAVB device node

[Simon Horman]

On Wed, Sep 13, 2017 at 04:47:09PM +0200, Geert Uytterhoeven wrote:
> On Wed, Sep 13, 2017 at 2:18 PM, Yoshihiro Shimoda
> <yoshihiro.shimoda.uh-zM6kxYcvzFBBDgjK7y7TUQ@public.gmane.org> wrote:
> > This patch adds EthernetAVB device node for r8a77995.
> >
> > Signed-off-by: Yoshihiro Shimoda <yoshihiro.shimoda.uh-zM6kxYcvzFBBDgjK7y7TUQ@public.gmane.org>
> 
> Reviewed-by: Geert Uytterhoeven <geert+renesas-gXvu3+zWzMSzQB+pC5nmwQ@public.gmane.org>

Thanks, applied.
--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Re: [PATCH 1/2] arm64: dts: renesas: r8a77995: Add EthernetAVB device node

[Simon Horman]

On Wed, Sep 13, 2017 at 04:47:09PM +0200, Geert Uytterhoeven wrote:
> On Wed, Sep 13, 2017 at 2:18 PM, Yoshihiro Shimoda
> <yoshihiro.shimoda.uh@renesas.com> wrote:
> > This patch adds EthernetAVB device node for r8a77995.
> >
> > Signed-off-by: Yoshihiro Shimoda <yoshihiro.shimoda.uh@renesas.com>
> 
> Reviewed-by: Geert Uytterhoeven <geert+renesas@glider.be>

Thanks, applied.

Re: [PATCH 2/2] arm64: dts: renesas: r8a77995: draak: enable EthernetAVB

[Simon Horman]

On Wed, Sep 13, 2017 at 11:58:47PM +0000, Yoshihiro Shimoda wrote:
> Hi Geert-san,
> 
> > From: Geert Uytterhoeven
> > Sent: Wednesday, September 13, 2017 11:55 PM
> > 
> > Hi Shimoda-san,
> > 
> > On Wed, Sep 13, 2017 at 2:18 PM, Yoshihiro Shimoda
> > <yoshihiro.shimoda.uh-zM6kxYcvzFBBDgjK7y7TUQ@public.gmane.org> wrote:
> > > This patch enables EthernetAVB for R-Car D3 draak board.
> > >
> > > Signed-off-by: Yoshihiro Shimoda <yoshihiro.shimoda.uh-zM6kxYcvzFBBDgjK7y7TUQ@public.gmane.org>
> > 
> > Thanks for your patch!
> > 
> > > --- a/arch/arm64/boot/dts/renesas/r8a77995-draak.dts
> > > +++ b/arch/arm64/boot/dts/renesas/r8a77995-draak.dts
> > 
> > > @@ -37,6 +39,14 @@
> > >  };
> > >
> > >  &pfc {
> > > +       avb0_pins: avb {
> > > +               mux {
> > > +                       groups = "avb0_link", "avb0_phy_int", "avb0_mdc",
> > > +                                "avb0_mii";
> > > +                       function = "avb0";
> > 
> > This part depends on (an updated version of) "[PATCH 5/8] pinctrl: sh-pfc:
> > r8a77995: Add EthernetAVB pins, groups and functions", so it may change?
> 
> Oops, I fotgot to submit v2 PFC patch yesterday...
> Yes, it is changed like other gen3 SoCs. I submitted it now. 
> 
> > And we will probably have to add driver-strengths later, to avoid a
> > dependency on bootloader setup?
> 
> Yes, I will do so, if needed.
> 
> > Apart from that:
> > Reviewed-by: Geert Uytterhoeven <geert+renesas-gXvu3+zWzMSzQB+pC5nmwQ@public.gmane.org>
> 
> Thank you for your review!

Thanks. I have marked this as "Deferred" pending acceptance of the PFC
patch. Please repost or ping me, f.e. via a reply-to-all to this email,
once that dependency is resolved.
--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Re: [PATCH 2/2] arm64: dts: renesas: r8a77995: draak: enable EthernetAVB

[Simon Horman]

On Wed, Sep 13, 2017 at 11:58:47PM +0000, Yoshihiro Shimoda wrote:
> Hi Geert-san,
> 
> > From: Geert Uytterhoeven
> > Sent: Wednesday, September 13, 2017 11:55 PM
> > 
> > Hi Shimoda-san,
> > 
> > On Wed, Sep 13, 2017 at 2:18 PM, Yoshihiro Shimoda
> > <yoshihiro.shimoda.uh@renesas.com> wrote:
> > > This patch enables EthernetAVB for R-Car D3 draak board.
> > >
> > > Signed-off-by: Yoshihiro Shimoda <yoshihiro.shimoda.uh@renesas.com>
> > 
> > Thanks for your patch!
> > 
> > > --- a/arch/arm64/boot/dts/renesas/r8a77995-draak.dts
> > > +++ b/arch/arm64/boot/dts/renesas/r8a77995-draak.dts
> > 
> > > @@ -37,6 +39,14 @@
> > >  };
> > >
> > >  &pfc {
> > > +       avb0_pins: avb {
> > > +               mux {
> > > +                       groups = "avb0_link", "avb0_phy_int", "avb0_mdc",
> > > +                                "avb0_mii";
> > > +                       function = "avb0";
> > 
> > This part depends on (an updated version of) "[PATCH 5/8] pinctrl: sh-pfc:
> > r8a77995: Add EthernetAVB pins, groups and functions", so it may change?
> 
> Oops, I fotgot to submit v2 PFC patch yesterday...
> Yes, it is changed like other gen3 SoCs. I submitted it now. 
> 
> > And we will probably have to add driver-strengths later, to avoid a
> > dependency on bootloader setup?
> 
> Yes, I will do so, if needed.
> 
> > Apart from that:
> > Reviewed-by: Geert Uytterhoeven <geert+renesas@glider.be>
> 
> Thank you for your review!

Thanks. I have marked this as "Deferred" pending acceptance of the PFC
patch. Please repost or ping me, f.e. via a reply-to-all to this email,
once that dependency is resolved.

RE: [PATCH 2/2] arm64: dts: renesas: r8a77995: draak: enable EthernetAVB

[Yoshihiro Shimoda]

Hi Simon-san,

> From: Simon Horman
> Sent: Friday, September 15, 2017 4:35 PM
> 
> On Wed, Sep 13, 2017 at 11:58:47PM +0000, Yoshihiro Shimoda wrote:
> > Hi Geert-san,
> >
> > > From: Geert Uytterhoeven
> > > Sent: Wednesday, September 13, 2017 11:55 PM
> > >
> > > Hi Shimoda-san,
> > >
> > > On Wed, Sep 13, 2017 at 2:18 PM, Yoshihiro Shimoda
> > > <yoshihiro.shimoda.uh@renesas.com> wrote:
> > > > This patch enables EthernetAVB for R-Car D3 draak board.
> > > >
> > > > Signed-off-by: Yoshihiro Shimoda <yoshihiro.shimoda.uh@renesas.com>
> > >
> > > Thanks for your patch!
> > >
> > > > --- a/arch/arm64/boot/dts/renesas/r8a77995-draak.dts
> > > > +++ b/arch/arm64/boot/dts/renesas/r8a77995-draak.dts
> > >
> > > > @@ -37,6 +39,14 @@
> > > >  };
> > > >
> > > >  &pfc {
> > > > +       avb0_pins: avb {
> > > > +               mux {
> > > > +                       groups = "avb0_link", "avb0_phy_int", "avb0_mdc",
> > > > +                                "avb0_mii";
> > > > +                       function = "avb0";
> > >
> > > This part depends on (an updated version of) "[PATCH 5/8] pinctrl: sh-pfc:
> > > r8a77995: Add EthernetAVB pins, groups and functions", so it may change?
> >
> > Oops, I fotgot to submit v2 PFC patch yesterday...
> > Yes, it is changed like other gen3 SoCs. I submitted it now.
> >
> > > And we will probably have to add driver-strengths later, to avoid a
> > > dependency on bootloader setup?
> >
> > Yes, I will do so, if needed.
> >
> > > Apart from that:
> > > Reviewed-by: Geert Uytterhoeven <geert+renesas@glider.be>
> >
> > Thank you for your review!
> 
> Thanks. I have marked this as "Deferred" pending acceptance of the PFC
> patch. Please repost or ping me, f.e. via a reply-to-all to this email,
> once that dependency is resolved.

Since the PFC patch is applied into renesas-drivers.git / sh-pfc-for-v4.15 branch,
I'd like to ping you :)
https://git.kernel.org/pub/scm/linux/kernel/git/geert/renesas-drivers.git/commit/?h=sh-pfc-for-v4.15&id=66abd968d0ef3eb10dea45b48a31321eb29258f8

Or, should I rebase this patch on latest devel tag?
If so, I will rebase it in next Monday because I'll take a day off tomorrow.

Best regards,
Yoshihiro Shimoda

Re: [PATCH 2/2] arm64: dts: renesas: r8a77995: draak: enable EthernetAVB

[Simon Horman]

On Thu, Sep 21, 2017 at 10:37:15AM +0000, Yoshihiro Shimoda wrote:
> Hi Simon-san,
> 
> > From: Simon Horman
> > Sent: Friday, September 15, 2017 4:35 PM
> > 
> > On Wed, Sep 13, 2017 at 11:58:47PM +0000, Yoshihiro Shimoda wrote:
> > > Hi Geert-san,
> > >
> > > > From: Geert Uytterhoeven
> > > > Sent: Wednesday, September 13, 2017 11:55 PM
> > > >
> > > > Hi Shimoda-san,
> > > >
> > > > On Wed, Sep 13, 2017 at 2:18 PM, Yoshihiro Shimoda
> > > > <yoshihiro.shimoda.uh@renesas.com> wrote:
> > > > > This patch enables EthernetAVB for R-Car D3 draak board.
> > > > >
> > > > > Signed-off-by: Yoshihiro Shimoda <yoshihiro.shimoda.uh@renesas.com>
> > > >
> > > > Thanks for your patch!
> > > >
> > > > > --- a/arch/arm64/boot/dts/renesas/r8a77995-draak.dts
> > > > > +++ b/arch/arm64/boot/dts/renesas/r8a77995-draak.dts
> > > >
> > > > > @@ -37,6 +39,14 @@
> > > > >  };
> > > > >
> > > > >  &pfc {
> > > > > +       avb0_pins: avb {
> > > > > +               mux {
> > > > > +                       groups = "avb0_link", "avb0_phy_int", "avb0_mdc",
> > > > > +                                "avb0_mii";
> > > > > +                       function = "avb0";
> > > >
> > > > This part depends on (an updated version of) "[PATCH 5/8] pinctrl: sh-pfc:
> > > > r8a77995: Add EthernetAVB pins, groups and functions", so it may change?
> > >
> > > Oops, I fotgot to submit v2 PFC patch yesterday...
> > > Yes, it is changed like other gen3 SoCs. I submitted it now.
> > >
> > > > And we will probably have to add driver-strengths later, to avoid a
> > > > dependency on bootloader setup?
> > >
> > > Yes, I will do so, if needed.
> > >
> > > > Apart from that:
> > > > Reviewed-by: Geert Uytterhoeven <geert+renesas@glider.be>
> > >
> > > Thank you for your review!
> > 
> > Thanks. I have marked this as "Deferred" pending acceptance of the PFC
> > patch. Please repost or ping me, f.e. via a reply-to-all to this email,
> > once that dependency is resolved.
> 
> Since the PFC patch is applied into renesas-drivers.git / sh-pfc-for-v4.15 branch,
> I'd like to ping you :)
> https://git.kernel.org/pub/scm/linux/kernel/git/geert/renesas-drivers.git/commit/?h=sh-pfc-for-v4.15&id=66abd968d0ef3eb10dea45b48a31321eb29258f8
> 
> Or, should I rebase this patch on latest devel tag?
> If so, I will rebase it in next Monday because I'll take a day off tomorrow.

Thanks for pinging me. I have applied this patch.
There was some fuzz when doing so. Please check that the applied
patch is correct once I have pushed an updated devel branch later today.

RE: [PATCH 2/2] arm64: dts: renesas: r8a77995: draak: enable EthernetAVB

[Yoshihiro Shimoda]

Hi Simon-san,

> From: Simon Horman
> Sent: Monday, September 25, 2017 4:08 PM
> 
> On Thu, Sep 21, 2017 at 10:37:15AM +0000, Yoshihiro Shimoda wrote:
> > Hi Simon-san,
> >
> > > From: Simon Horman
> > > Sent: Friday, September 15, 2017 4:35 PM
> > >
> > > On Wed, Sep 13, 2017 at 11:58:47PM +0000, Yoshihiro Shimoda wrote:
> > > > Hi Geert-san,
> > > >
> > > > > From: Geert Uytterhoeven
> > > > > Sent: Wednesday, September 13, 2017 11:55 PM
> > > > >
> > > > > Hi Shimoda-san,
> > > > >
> > > > > On Wed, Sep 13, 2017 at 2:18 PM, Yoshihiro Shimoda
> > > > > <yoshihiro.shimoda.uh-zM6kxYcvzFBBDgjK7y7TUQ@public.gmane.org> wrote:
> > > > > > This patch enables EthernetAVB for R-Car D3 draak board.
> > > > > >
> > > > > > Signed-off-by: Yoshihiro Shimoda <yoshihiro.shimoda.uh@renesas.com>
> > > > >
> > > > > Thanks for your patch!
> > > > >
> > > > > > --- a/arch/arm64/boot/dts/renesas/r8a77995-draak.dts
> > > > > > +++ b/arch/arm64/boot/dts/renesas/r8a77995-draak.dts
> > > > >
> > > > > > @@ -37,6 +39,14 @@
> > > > > >  };
> > > > > >
> > > > > >  &pfc {
> > > > > > +       avb0_pins: avb {
> > > > > > +               mux {
> > > > > > +                       groups = "avb0_link", "avb0_phy_int", "avb0_mdc",
> > > > > > +                                "avb0_mii";
> > > > > > +                       function = "avb0";
> > > > >
> > > > > This part depends on (an updated version of) "[PATCH 5/8] pinctrl: sh-pfc:
> > > > > r8a77995: Add EthernetAVB pins, groups and functions", so it may change?
> > > >
> > > > Oops, I fotgot to submit v2 PFC patch yesterday...
> > > > Yes, it is changed like other gen3 SoCs. I submitted it now.
> > > >
> > > > > And we will probably have to add driver-strengths later, to avoid a
> > > > > dependency on bootloader setup?
> > > >
> > > > Yes, I will do so, if needed.
> > > >
> > > > > Apart from that:
> > > > > Reviewed-by: Geert Uytterhoeven <geert+renesas-gXvu3+zWzMSzQB+pC5nmwQ@public.gmane.org>
> > > >
> > > > Thank you for your review!
> > >
> > > Thanks. I have marked this as "Deferred" pending acceptance of the PFC
> > > patch. Please repost or ping me, f.e. via a reply-to-all to this email,
> > > once that dependency is resolved.
> >
> > Since the PFC patch is applied into renesas-drivers.git / sh-pfc-for-v4.15 branch,
> > I'd like to ping you :)
> >
> https://git.kernel.org/pub/scm/linux/kernel/git/geert/renesas-drivers.git/commit/?h=sh-pfc-for-v4.15&id=66abd968d0ef
> 3eb10dea45b48a31321eb29258f8
> >
> > Or, should I rebase this patch on latest devel tag?
> > If so, I will rebase it in next Monday because I'll take a day off tomorrow.
> 
> Thanks for pinging me. I have applied this patch.
> There was some fuzz when doing so. Please check that the applied
> patch is correct once I have pushed an updated devel branch later today.

Thank you for applying!
I checked renesas.git / renesas-devel-20170925-v4.14-rc2 tag, and it's correct.

Best regards,
Yoshihiro Shimoda
--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

RE: [PATCH 2/2] arm64: dts: renesas: r8a77995: draak: enable EthernetAVB

[Yoshihiro Shimoda]

Hi Simon-san,

> From: Simon Horman
> Sent: Monday, September 25, 2017 4:08 PM
> 
> On Thu, Sep 21, 2017 at 10:37:15AM +0000, Yoshihiro Shimoda wrote:
> > Hi Simon-san,
> >
> > > From: Simon Horman
> > > Sent: Friday, September 15, 2017 4:35 PM
> > >
> > > On Wed, Sep 13, 2017 at 11:58:47PM +0000, Yoshihiro Shimoda wrote:
> > > > Hi Geert-san,
> > > >
> > > > > From: Geert Uytterhoeven
> > > > > Sent: Wednesday, September 13, 2017 11:55 PM
> > > > >
> > > > > Hi Shimoda-san,
> > > > >
> > > > > On Wed, Sep 13, 2017 at 2:18 PM, Yoshihiro Shimoda
> > > > > <yoshihiro.shimoda.uh@renesas.com> wrote:
> > > > > > This patch enables EthernetAVB for R-Car D3 draak board.
> > > > > >
> > > > > > Signed-off-by: Yoshihiro Shimoda <yoshihiro.shimoda.uh@renesas.com>
> > > > >
> > > > > Thanks for your patch!
> > > > >
> > > > > > --- a/arch/arm64/boot/dts/renesas/r8a77995-draak.dts
> > > > > > +++ b/arch/arm64/boot/dts/renesas/r8a77995-draak.dts
> > > > >
> > > > > > @@ -37,6 +39,14 @@
> > > > > >  };
> > > > > >
> > > > > >  &pfc {
> > > > > > +       avb0_pins: avb {
> > > > > > +               mux {
> > > > > > +                       groups = "avb0_link", "avb0_phy_int", "avb0_mdc",
> > > > > > +                                "avb0_mii";
> > > > > > +                       function = "avb0";
> > > > >
> > > > > This part depends on (an updated version of) "[PATCH 5/8] pinctrl: sh-pfc:
> > > > > r8a77995: Add EthernetAVB pins, groups and functions", so it may change?
> > > >
> > > > Oops, I fotgot to submit v2 PFC patch yesterday...
> > > > Yes, it is changed like other gen3 SoCs. I submitted it now.
> > > >
> > > > > And we will probably have to add driver-strengths later, to avoid a
> > > > > dependency on bootloader setup?
> > > >
> > > > Yes, I will do so, if needed.
> > > >
> > > > > Apart from that:
> > > > > Reviewed-by: Geert Uytterhoeven <geert+renesas@glider.be>
> > > >
> > > > Thank you for your review!
> > >
> > > Thanks. I have marked this as "Deferred" pending acceptance of the PFC
> > > patch. Please repost or ping me, f.e. via a reply-to-all to this email,
> > > once that dependency is resolved.
> >
> > Since the PFC patch is applied into renesas-drivers.git / sh-pfc-for-v4.15 branch,
> > I'd like to ping you :)
> >
> https://git.kernel.org/pub/scm/linux/kernel/git/geert/renesas-drivers.git/commit/?h=sh-pfc-for-v4.15&id=66abd968d0ef
> 3eb10dea45b48a31321eb29258f8
> >
> > Or, should I rebase this patch on latest devel tag?
> > If so, I will rebase it in next Monday because I'll take a day off tomorrow.
> 
> Thanks for pinging me. I have applied this patch.
> There was some fuzz when doing so. Please check that the applied
> patch is correct once I have pushed an updated devel branch later today.

Thank you for applying!
I checked renesas.git / renesas-devel-20170925-v4.14-rc2 tag, and it's correct.

Best regards,
Yoshihiro Shimoda

Re: [PATCH 2/2] arm64: dts: renesas: r8a77995: draak: enable EthernetAVB

[Simon Horman]

On Wed, Sep 27, 2017 at 02:03:14AM +0000, Yoshihiro Shimoda wrote:
> Hi Simon-san,
> > From: Simon Horman
> > Sent: Monday, September 25, 2017 4:08 PM
> > On Thu, Sep 21, 2017 at 10:37:15AM +0000, Yoshihiro Shimoda wrote:
> > > Hi Simon-san,
> > > > From: Simon Horman
> > > > Sent: Friday, September 15, 2017 4:35 PM

...

> > > > Thanks. I have marked this as "Deferred" pending acceptance of the PFC
> > > > patch. Please repost or ping me, f.e. via a reply-to-all to this email,
> > > > once that dependency is resolved.
> > >
> > > Since the PFC patch is applied into renesas-drivers.git / sh-pfc-for-v4.15 branch,
> > > I'd like to ping you :)
> > >
> > https://git.kernel.org/pub/scm/linux/kernel/git/geert/renesas-drivers.git/commit/?h=sh-pfc-for-v4.15&id=66abd968d0ef
> > 3eb10dea45b48a31321eb29258f8
> > >
> > > Or, should I rebase this patch on latest devel tag?
> > > If so, I will rebase it in next Monday because I'll take a day off tomorrow.
> > 
> > Thanks for pinging me. I have applied this patch.
> > There was some fuzz when doing so. Please check that the applied
> > patch is correct once I have pushed an updated devel branch later today.
> 
> Thank you for applying!
> I checked renesas.git / renesas-devel-20170925-v4.14-rc2 tag, and it's correct.

Thanks for checking, much appreciated.
--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Re: [PATCH 2/2] arm64: dts: renesas: r8a77995: draak: enable EthernetAVB

[Simon Horman]

On Wed, Sep 27, 2017 at 02:03:14AM +0000, Yoshihiro Shimoda wrote:
> Hi Simon-san,
> > From: Simon Horman
> > Sent: Monday, September 25, 2017 4:08 PM
> > On Thu, Sep 21, 2017 at 10:37:15AM +0000, Yoshihiro Shimoda wrote:
> > > Hi Simon-san,
> > > > From: Simon Horman
> > > > Sent: Friday, September 15, 2017 4:35 PM

...

> > > > Thanks. I have marked this as "Deferred" pending acceptance of the PFC
> > > > patch. Please repost or ping me, f.e. via a reply-to-all to this email,
> > > > once that dependency is resolved.
> > >
> > > Since the PFC patch is applied into renesas-drivers.git / sh-pfc-for-v4.15 branch,
> > > I'd like to ping you :)
> > >
> > https://git.kernel.org/pub/scm/linux/kernel/git/geert/renesas-drivers.git/commit/?h=sh-pfc-for-v4.15&id=66abd968d0ef
> > 3eb10dea45b48a31321eb29258f8
> > >
> > > Or, should I rebase this patch on latest devel tag?
> > > If so, I will rebase it in next Monday because I'll take a day off tomorrow.
> > 
> > Thanks for pinging me. I have applied this patch.
> > There was some fuzz when doing so. Please check that the applied
> > patch is correct once I have pushed an updated devel branch later today.
> 
> Thank you for applying!
> I checked renesas.git / renesas-devel-20170925-v4.14-rc2 tag, and it's correct.

Thanks for checking, much appreciated.