[PATCH v2 8/8] arm64: dts: renesas: eagle: add EtherAVB support

[PATCH v2 8/8] arm64: dts: renesas: eagle: add EtherAVB support

[Sergei Shtylyov]

[-- Attachment #1: arm64-dts-renesas-eagle-add-EtherAVB-support-v2.patch --]
[-- Type: text/plain, Size: 1261 bytes --]

Define the Eagle board  dependent part of the EtherAVB device node.
Enable DHCP  and NFS root for the kernel booting.

Based  on the original (and large) patch by Vladimir Barinov.

Signed-off-by: Vladimir Barinov <vladimir.barinov@cogentembedded.com>
Signed-off-by: Sergei Shtylyov <sergei.shtylyov@cogentembedded.com>

---
Changes in version 2:
- uncommented the "renesas,no-ether-link" property;
- removed the "max-speed" property from the PHY node.

 arch/arm64/boot/dts/renesas/r8a77970-eagle.dts |   14 +++++++++++++-
 1 file changed, 13 insertions(+), 1 deletion(-)

Index: renesas/arch/arm64/boot/dts/renesas/r8a77970-eagle.dts
===================================================================
--- renesas.orig/arch/arm64/boot/dts/renesas/r8a77970-eagle.dts
+++ renesas/arch/arm64/boot/dts/renesas/r8a77970-eagle.dts
@@ -18,10 +18,11 @@
 
 	aliases {
 		serial0 = &scif0;
+		ethernet0 = &avb;
 	};
 
 	chosen {
-		bootargs = "ignore_loglevel";
+		bootargs = "ignore_loglevel rw root=/dev/nfs ip=dhcp";
 		stdout-path = "serial0:115200n8";
 	};
 
@@ -43,3 +44,14 @@
 &scif0 {
 	status = "okay";
 };
+
+&avb {
+	renesas,no-ether-link;
+	phy-handle = <&phy0>;
+	status = "okay";
+
+	phy0: ethernet-phy@0 {
+		rxc-skew-ps = <1500>;
+		reg = <0>;
+	};
+};

[PATCH v2 8/8] arm64: dts: renesas: eagle: add EtherAVB support

[Sergei Shtylyov]

Define the Eagle board  dependent part of the EtherAVB device node.
Enable DHCP  and NFS root for the kernel booting.

Based  on the original (and large) patch by Vladimir Barinov.

Signed-off-by: Vladimir Barinov <vladimir.barinov@cogentembedded.com>
Signed-off-by: Sergei Shtylyov <sergei.shtylyov@cogentembedded.com>

---
Changes in version 2:
- uncommented the "renesas,no-ether-link" property;
- removed the "max-speed" property from the PHY node.

 arch/arm64/boot/dts/renesas/r8a77970-eagle.dts |   14 +++++++++++++-
 1 file changed, 13 insertions(+), 1 deletion(-)

Index: renesas/arch/arm64/boot/dts/renesas/r8a77970-eagle.dts
===================================================================
--- renesas.orig/arch/arm64/boot/dts/renesas/r8a77970-eagle.dts
+++ renesas/arch/arm64/boot/dts/renesas/r8a77970-eagle.dts
@@ -18,10 +18,11 @@
 
 	aliases {
 		serial0 = &scif0;
+		ethernet0 = &avb;
 	};
 
 	chosen {
-		bootargs = "ignore_loglevel";
+		bootargs = "ignore_loglevel rw root=/dev/nfs ip=dhcp";
 		stdout-path = "serial0:115200n8";
 	};
 
@@ -43,3 +44,14 @@
 &scif0 {
 	status = "okay";
 };
+
+&avb {
+	renesas,no-ether-link;
+	phy-handle = <&phy0>;
+	status = "okay";
+
+	phy0: ethernet-phy at 0 {
+		rxc-skew-ps = <1500>;
+		reg = <0>;
+	};
+};

Re: [PATCH v2 8/8] arm64: dts: renesas: eagle: add EtherAVB support

[Geert Uytterhoeven]

Hi Sergei,

On Fri, Sep 15, 2017 at 9:43 PM, Sergei Shtylyov
<sergei.shtylyov-M4DtvfQ/ZS1MRgGoP+s0PdBPR1lH4CV8@public.gmane.org> wrote:
> Define the Eagle board  dependent part of the EtherAVB device node.
> Enable DHCP  and NFS root for the kernel booting.
>
> Based  on the original (and large) patch by Vladimir Barinov.
>
> Signed-off-by: Vladimir Barinov <vladimir.barinov-M4DtvfQ/ZS1MRgGoP+s0PdBPR1lH4CV8@public.gmane.org>
> Signed-off-by: Sergei Shtylyov <sergei.shtylyov-M4DtvfQ/ZS1MRgGoP+s0PdBPR1lH4CV8@public.gmane.org>

> --- renesas.orig/arch/arm64/boot/dts/renesas/r8a77970-eagle.dts
> +++ renesas/arch/arm64/boot/dts/renesas/r8a77970-eagle.dts
> @@ -18,10 +18,11 @@
>
>         aliases {
>                 serial0 = &scif0;
> +               ethernet0 = &avb;
>         };
>
>         chosen {
> -               bootargs = "ignore_loglevel";
> +               bootargs = "ignore_loglevel rw root=/dev/nfs ip=dhcp";
>                 stdout-path = "serial0:115200n8";
>         };
>
> @@ -43,3 +44,14 @@
>  &scif0 {
>         status = "okay";
>  };
> +
> +&avb {
> +       renesas,no-ether-link;
> +       phy-handle = <&phy0>;
> +       status = "okay";
> +
> +       phy0: ethernet-phy@0 {
> +               rxc-skew-ps = <1500>;
> +               reg = <0>;

Any specific reason why you don't want to wire up the interrupt?

                interrupt-parent = <&gpio2>;
                interrupts = <11 IRQ_TYPE_LEVEL_LOW>;

> +       };
> +};

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 v2 8/8] arm64: dts: renesas: eagle: add EtherAVB support

[Geert Uytterhoeven]

Hi Sergei,

On Fri, Sep 15, 2017 at 9:43 PM, Sergei Shtylyov
<sergei.shtylyov@cogentembedded.com> wrote:
> Define the Eagle board  dependent part of the EtherAVB device node.
> Enable DHCP  and NFS root for the kernel booting.
>
> Based  on the original (and large) patch by Vladimir Barinov.
>
> Signed-off-by: Vladimir Barinov <vladimir.barinov@cogentembedded.com>
> Signed-off-by: Sergei Shtylyov <sergei.shtylyov@cogentembedded.com>

> --- renesas.orig/arch/arm64/boot/dts/renesas/r8a77970-eagle.dts
> +++ renesas/arch/arm64/boot/dts/renesas/r8a77970-eagle.dts
> @@ -18,10 +18,11 @@
>
>         aliases {
>                 serial0 = &scif0;
> +               ethernet0 = &avb;
>         };
>
>         chosen {
> -               bootargs = "ignore_loglevel";
> +               bootargs = "ignore_loglevel rw root=/dev/nfs ip=dhcp";
>                 stdout-path = "serial0:115200n8";
>         };
>
> @@ -43,3 +44,14 @@
>  &scif0 {
>         status = "okay";
>  };
> +
> +&avb {
> +       renesas,no-ether-link;
> +       phy-handle = <&phy0>;
> +       status = "okay";
> +
> +       phy0: ethernet-phy@0 {
> +               rxc-skew-ps = <1500>;
> +               reg = <0>;

Any specific reason why you don't want to wire up the interrupt?

                interrupt-parent = <&gpio2>;
                interrupts = <11 IRQ_TYPE_LEVEL_LOW>;

> +       };
> +};

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

[PATCH v2 8/8] arm64: dts: renesas: eagle: add EtherAVB support

[Geert Uytterhoeven]

Hi Sergei,

On Fri, Sep 15, 2017 at 9:43 PM, Sergei Shtylyov
<sergei.shtylyov@cogentembedded.com> wrote:
> Define the Eagle board  dependent part of the EtherAVB device node.
> Enable DHCP  and NFS root for the kernel booting.
>
> Based  on the original (and large) patch by Vladimir Barinov.
>
> Signed-off-by: Vladimir Barinov <vladimir.barinov@cogentembedded.com>
> Signed-off-by: Sergei Shtylyov <sergei.shtylyov@cogentembedded.com>

> --- renesas.orig/arch/arm64/boot/dts/renesas/r8a77970-eagle.dts
> +++ renesas/arch/arm64/boot/dts/renesas/r8a77970-eagle.dts
> @@ -18,10 +18,11 @@
>
>         aliases {
>                 serial0 = &scif0;
> +               ethernet0 = &avb;
>         };
>
>         chosen {
> -               bootargs = "ignore_loglevel";
> +               bootargs = "ignore_loglevel rw root=/dev/nfs ip=dhcp";
>                 stdout-path = "serial0:115200n8";
>         };
>
> @@ -43,3 +44,14 @@
>  &scif0 {
>         status = "okay";
>  };
> +
> +&avb {
> +       renesas,no-ether-link;
> +       phy-handle = <&phy0>;
> +       status = "okay";
> +
> +       phy0: ethernet-phy at 0 {
> +               rxc-skew-ps = <1500>;
> +               reg = <0>;

Any specific reason why you don't want to wire up the interrupt?

                interrupt-parent = <&gpio2>;
                interrupts = <11 IRQ_TYPE_LEVEL_LOW>;

> +       };
> +};

Gr{oetje,eeting}s,

                        Geert

--
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert at 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 v2 8/8] arm64: dts: renesas: eagle: add EtherAVB support

[Sergei Shtylyov]

On 9/21/2017 4:08 PM, Geert Uytterhoeven wrote:

>> Define the Eagle board  dependent part of the EtherAVB device node.
>> Enable DHCP  and NFS root for the kernel booting.
>>
>> Based  on the original (and large) patch by Vladimir Barinov.
>>
>> Signed-off-by: Vladimir Barinov <vladimir.barinov-M4DtvfQ/ZS1MRgGoP+s0PdBPR1lH4CV8@public.gmane.org>
>> Signed-off-by: Sergei Shtylyov <sergei.shtylyov-M4DtvfQ/ZS1MRgGoP+s0PdBPR1lH4CV8@public.gmane.org>
> 
>> --- renesas.orig/arch/arm64/boot/dts/renesas/r8a77970-eagle.dts
>> +++ renesas/arch/arm64/boot/dts/renesas/r8a77970-eagle.dts
[...]
>> @@ -43,3 +44,14 @@
>>   &scif0 {
>>          status = "okay";
>>   };
>> +
>> +&avb {
>> +       renesas,no-ether-link;
>> +       phy-handle = <&phy0>;
>> +       status = "okay";
>> +
>> +       phy0: ethernet-phy@0 {
>> +               rxc-skew-ps = <1500>;
>> +               reg = <0>;
> 
> Any specific reason why you don't want to wire up the interrupt?
> 
>                  interrupt-parent = <&gpio2>;

    I thought it's quite obvious -- we don't have GPIOs yet, and GPIOs seem to 
require PFC.

>                  interrupts = <11 IRQ_TYPE_LEVEL_LOW>;
> 
>> +       };
>> +};
> 
> Gr{oetje,eeting}s,
> 
>                          Geert

MBR, Sergei
--
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 v2 8/8] arm64: dts: renesas: eagle: add EtherAVB support

[Sergei Shtylyov]

On 9/21/2017 4:08 PM, Geert Uytterhoeven wrote:

>> Define the Eagle board  dependent part of the EtherAVB device node.
>> Enable DHCP  and NFS root for the kernel booting.
>>
>> Based  on the original (and large) patch by Vladimir Barinov.
>>
>> Signed-off-by: Vladimir Barinov <vladimir.barinov@cogentembedded.com>
>> Signed-off-by: Sergei Shtylyov <sergei.shtylyov@cogentembedded.com>
> 
>> --- renesas.orig/arch/arm64/boot/dts/renesas/r8a77970-eagle.dts
>> +++ renesas/arch/arm64/boot/dts/renesas/r8a77970-eagle.dts
[...]
>> @@ -43,3 +44,14 @@
>>   &scif0 {
>>          status = "okay";
>>   };
>> +
>> +&avb {
>> +       renesas,no-ether-link;
>> +       phy-handle = <&phy0>;
>> +       status = "okay";
>> +
>> +       phy0: ethernet-phy@0 {
>> +               rxc-skew-ps = <1500>;
>> +               reg = <0>;
> 
> Any specific reason why you don't want to wire up the interrupt?
> 
>                  interrupt-parent = <&gpio2>;

    I thought it's quite obvious -- we don't have GPIOs yet, and GPIOs seem to 
require PFC.

>                  interrupts = <11 IRQ_TYPE_LEVEL_LOW>;
> 
>> +       };
>> +};
> 
> Gr{oetje,eeting}s,
> 
>                          Geert

MBR, Sergei

[PATCH v2 8/8] arm64: dts: renesas: eagle: add EtherAVB support

[Sergei Shtylyov]

On 9/21/2017 4:08 PM, Geert Uytterhoeven wrote:

>> Define the Eagle board  dependent part of the EtherAVB device node.
>> Enable DHCP  and NFS root for the kernel booting.
>>
>> Based  on the original (and large) patch by Vladimir Barinov.
>>
>> Signed-off-by: Vladimir Barinov <vladimir.barinov@cogentembedded.com>
>> Signed-off-by: Sergei Shtylyov <sergei.shtylyov@cogentembedded.com>
> 
>> --- renesas.orig/arch/arm64/boot/dts/renesas/r8a77970-eagle.dts
>> +++ renesas/arch/arm64/boot/dts/renesas/r8a77970-eagle.dts
[...]
>> @@ -43,3 +44,14 @@
>>   &scif0 {
>>          status = "okay";
>>   };
>> +
>> +&avb {
>> +       renesas,no-ether-link;
>> +       phy-handle = <&phy0>;
>> +       status = "okay";
>> +
>> +       phy0: ethernet-phy at 0 {
>> +               rxc-skew-ps = <1500>;
>> +               reg = <0>;
> 
> Any specific reason why you don't want to wire up the interrupt?
> 
>                  interrupt-parent = <&gpio2>;

    I thought it's quite obvious -- we don't have GPIOs yet, and GPIOs seem to 
require PFC.

>                  interrupts = <11 IRQ_TYPE_LEVEL_LOW>;
> 
>> +       };
>> +};
> 
> Gr{oetje,eeting}s,
> 
>                          Geert

MBR, Sergei

Re: [PATCH v2 8/8] arm64: dts: renesas: eagle: add EtherAVB support

[Geert Uytterhoeven]

Hi Sergei,

On Thu, Sep 21, 2017 at 6:03 PM, Sergei Shtylyov
<sergei.shtylyov-M4DtvfQ/ZS1MRgGoP+s0PdBPR1lH4CV8@public.gmane.org> wrote:
> On 9/21/2017 4:08 PM, Geert Uytterhoeven wrote:
>>> Define the Eagle board  dependent part of the EtherAVB device node.
>>> Enable DHCP  and NFS root for the kernel booting.
>>>
>>> Based  on the original (and large) patch by Vladimir Barinov.
>>>
>>> Signed-off-by: Vladimir Barinov <vladimir.barinov-M4DtvfQ/ZS1MRgGoP+s0PdBPR1lH4CV8@public.gmane.org>
>>> Signed-off-by: Sergei Shtylyov <sergei.shtylyov-M4DtvfQ/ZS1MRgGoP+s0PdBPR1lH4CV8@public.gmane.org>
>>
>>
>>> --- renesas.orig/arch/arm64/boot/dts/renesas/r8a77970-eagle.dts
>>> +++ renesas/arch/arm64/boot/dts/renesas/r8a77970-eagle.dts
>
> [...]
>>>
>>> @@ -43,3 +44,14 @@
>>>   &scif0 {
>>>          status = "okay";
>>>   };
>>> +
>>> +&avb {
>>> +       renesas,no-ether-link;
>>> +       phy-handle = <&phy0>;
>>> +       status = "okay";
>>> +
>>> +       phy0: ethernet-phy@0 {
>>> +               rxc-skew-ps = <1500>;
>>> +               reg = <0>;
>>
>>
>> Any specific reason why you don't want to wire up the interrupt?
>>
>>                  interrupt-parent = <&gpio2>;
>
>
>    I thought it's quite obvious -- we don't have GPIOs yet, and GPIOs seem
> to require PFC.

Of course. And these can be added later.

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 v2 8/8] arm64: dts: renesas: eagle: add EtherAVB support

[Geert Uytterhoeven]

Hi Sergei,

On Thu, Sep 21, 2017 at 6:03 PM, Sergei Shtylyov
<sergei.shtylyov@cogentembedded.com> wrote:
> On 9/21/2017 4:08 PM, Geert Uytterhoeven wrote:
>>> Define the Eagle board  dependent part of the EtherAVB device node.
>>> Enable DHCP  and NFS root for the kernel booting.
>>>
>>> Based  on the original (and large) patch by Vladimir Barinov.
>>>
>>> Signed-off-by: Vladimir Barinov <vladimir.barinov@cogentembedded.com>
>>> Signed-off-by: Sergei Shtylyov <sergei.shtylyov@cogentembedded.com>
>>
>>
>>> --- renesas.orig/arch/arm64/boot/dts/renesas/r8a77970-eagle.dts
>>> +++ renesas/arch/arm64/boot/dts/renesas/r8a77970-eagle.dts
>
> [...]
>>>
>>> @@ -43,3 +44,14 @@
>>>   &scif0 {
>>>          status = "okay";
>>>   };
>>> +
>>> +&avb {
>>> +       renesas,no-ether-link;
>>> +       phy-handle = <&phy0>;
>>> +       status = "okay";
>>> +
>>> +       phy0: ethernet-phy@0 {
>>> +               rxc-skew-ps = <1500>;
>>> +               reg = <0>;
>>
>>
>> Any specific reason why you don't want to wire up the interrupt?
>>
>>                  interrupt-parent = <&gpio2>;
>
>
>    I thought it's quite obvious -- we don't have GPIOs yet, and GPIOs seem
> to require PFC.

Of course. And these can be added later.

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

[PATCH v2 8/8] arm64: dts: renesas: eagle: add EtherAVB support

[Geert Uytterhoeven]

Hi Sergei,

On Thu, Sep 21, 2017 at 6:03 PM, Sergei Shtylyov
<sergei.shtylyov@cogentembedded.com> wrote:
> On 9/21/2017 4:08 PM, Geert Uytterhoeven wrote:
>>> Define the Eagle board  dependent part of the EtherAVB device node.
>>> Enable DHCP  and NFS root for the kernel booting.
>>>
>>> Based  on the original (and large) patch by Vladimir Barinov.
>>>
>>> Signed-off-by: Vladimir Barinov <vladimir.barinov@cogentembedded.com>
>>> Signed-off-by: Sergei Shtylyov <sergei.shtylyov@cogentembedded.com>
>>
>>
>>> --- renesas.orig/arch/arm64/boot/dts/renesas/r8a77970-eagle.dts
>>> +++ renesas/arch/arm64/boot/dts/renesas/r8a77970-eagle.dts
>
> [...]
>>>
>>> @@ -43,3 +44,14 @@
>>>   &scif0 {
>>>          status = "okay";
>>>   };
>>> +
>>> +&avb {
>>> +       renesas,no-ether-link;
>>> +       phy-handle = <&phy0>;
>>> +       status = "okay";
>>> +
>>> +       phy0: ethernet-phy at 0 {
>>> +               rxc-skew-ps = <1500>;
>>> +               reg = <0>;
>>
>>
>> Any specific reason why you don't want to wire up the interrupt?
>>
>>                  interrupt-parent = <&gpio2>;
>
>
>    I thought it's quite obvious -- we don't have GPIOs yet, and GPIOs seem
> to require PFC.

Of course. And these can be added later.

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

Gr{oetje,eeting}s,

                        Geert

--
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert at 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 v2 8/8] arm64: dts: renesas: eagle: add EtherAVB support

[Simon Horman]

On Thu, Sep 21, 2017 at 06:09:34PM +0200, Geert Uytterhoeven wrote:
> Hi Sergei,
> 
> On Thu, Sep 21, 2017 at 6:03 PM, Sergei Shtylyov
> <sergei.shtylyov@cogentembedded.com> wrote:
> > On 9/21/2017 4:08 PM, Geert Uytterhoeven wrote:
> >>> Define the Eagle board  dependent part of the EtherAVB device node.
> >>> Enable DHCP  and NFS root for the kernel booting.
> >>>
> >>> Based  on the original (and large) patch by Vladimir Barinov.
> >>>
> >>> Signed-off-by: Vladimir Barinov <vladimir.barinov@cogentembedded.com>
> >>> Signed-off-by: Sergei Shtylyov <sergei.shtylyov@cogentembedded.com>
> >>
> >>
> >>> --- renesas.orig/arch/arm64/boot/dts/renesas/r8a77970-eagle.dts
> >>> +++ renesas/arch/arm64/boot/dts/renesas/r8a77970-eagle.dts
> >
> > [...]
> >>>
> >>> @@ -43,3 +44,14 @@
> >>>   &scif0 {
> >>>          status = "okay";
> >>>   };
> >>> +
> >>> +&avb {
> >>> +       renesas,no-ether-link;
> >>> +       phy-handle = <&phy0>;
> >>> +       status = "okay";
> >>> +
> >>> +       phy0: ethernet-phy@0 {
> >>> +               rxc-skew-ps = <1500>;
> >>> +               reg = <0>;
> >>
> >>
> >> Any specific reason why you don't want to wire up the interrupt?
> >>
> >>                  interrupt-parent = <&gpio2>;
> >
> >
> >    I thought it's quite obvious -- we don't have GPIOs yet, and GPIOs seem
> > to require PFC.
> 
> Of course. And these can be added later.
> 
> Reviewed-by: Geert Uytterhoeven <geert+renesas@glider.be>

Thanks, applied.

[PATCH v2 8/8] arm64: dts: renesas: eagle: add EtherAVB support

[Simon Horman]

On Thu, Sep 21, 2017 at 06:09:34PM +0200, Geert Uytterhoeven wrote:
> Hi Sergei,
> 
> On Thu, Sep 21, 2017 at 6:03 PM, Sergei Shtylyov
> <sergei.shtylyov@cogentembedded.com> wrote:
> > On 9/21/2017 4:08 PM, Geert Uytterhoeven wrote:
> >>> Define the Eagle board  dependent part of the EtherAVB device node.
> >>> Enable DHCP  and NFS root for the kernel booting.
> >>>
> >>> Based  on the original (and large) patch by Vladimir Barinov.
> >>>
> >>> Signed-off-by: Vladimir Barinov <vladimir.barinov@cogentembedded.com>
> >>> Signed-off-by: Sergei Shtylyov <sergei.shtylyov@cogentembedded.com>
> >>
> >>
> >>> --- renesas.orig/arch/arm64/boot/dts/renesas/r8a77970-eagle.dts
> >>> +++ renesas/arch/arm64/boot/dts/renesas/r8a77970-eagle.dts
> >
> > [...]
> >>>
> >>> @@ -43,3 +44,14 @@
> >>>   &scif0 {
> >>>          status = "okay";
> >>>   };
> >>> +
> >>> +&avb {
> >>> +       renesas,no-ether-link;
> >>> +       phy-handle = <&phy0>;
> >>> +       status = "okay";
> >>> +
> >>> +       phy0: ethernet-phy at 0 {
> >>> +               rxc-skew-ps = <1500>;
> >>> +               reg = <0>;
> >>
> >>
> >> Any specific reason why you don't want to wire up the interrupt?
> >>
> >>                  interrupt-parent = <&gpio2>;
> >
> >
> >    I thought it's quite obvious -- we don't have GPIOs yet, and GPIOs seem
> > to require PFC.
> 
> Of course. And these can be added later.
> 
> Reviewed-by: Geert Uytterhoeven <geert+renesas@glider.be>

Thanks, applied.

Re: [PATCH v2 8/8] arm64: dts: renesas: eagle: add EtherAVB support

[Sergei Shtylyov]

On 10/05/2017 12:05 PM, Simon Horman wrote:

>>>>> Define the Eagle board  dependent part of the EtherAVB device node.
>>>>> Enable DHCP  and NFS root for the kernel booting.
>>>>>
>>>>> Based  on the original (and large) patch by Vladimir Barinov.
>>>>>
>>>>> Signed-off-by: Vladimir Barinov <vladimir.barinov-M4DtvfQ/ZS1MRgGoP+s0PdBPR1lH4CV8@public.gmane.org>
>>>>> Signed-off-by: Sergei Shtylyov <sergei.shtylyov-M4DtvfQ/ZS1MRgGoP+s0PdBPR1lH4CV8@public.gmane.org>
>>>>
>>>>
>>>>> --- renesas.orig/arch/arm64/boot/dts/renesas/r8a77970-eagle.dts
>>>>> +++ renesas/arch/arm64/boot/dts/renesas/r8a77970-eagle.dts
>>>
>>> [...]
>>>>>
>>>>> @@ -43,3 +44,14 @@
>>>>>    &scif0 {
>>>>>           status = "okay";
>>>>>    };
>>>>> +
>>>>> +&avb {
>>>>> +       renesas,no-ether-link;
>>>>> +       phy-handle = <&phy0>;
>>>>> +       status = "okay";
>>>>> +
>>>>> +       phy0: ethernet-phy@0 {
>>>>> +               rxc-skew-ps = <1500>;
>>>>> +               reg = <0>;
>>>>
>>>>
>>>> Any specific reason why you don't want to wire up the interrupt?
>>>>
>>>>                   interrupt-parent = <&gpio2>;
>>>
>>>
>>>     I thought it's quite obvious -- we don't have GPIOs yet, and GPIOs seem
>>> to require PFC.
>>
>> Of course. And these can be added later.
>>
>> Reviewed-by: Geert Uytterhoeven <geert+renesas-gXvu3+zWzMSzQB+pC5nmwQ@public.gmane.org>
> 
> Thanks, applied.

    Still not seeing any devel branch updates/tags -- have you pushed it?

MBR, Sergei
--
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 v2 8/8] arm64: dts: renesas: eagle: add EtherAVB support

[Sergei Shtylyov]

On 10/05/2017 12:05 PM, Simon Horman wrote:

>>>>> Define the Eagle board  dependent part of the EtherAVB device node.
>>>>> Enable DHCP  and NFS root for the kernel booting.
>>>>>
>>>>> Based  on the original (and large) patch by Vladimir Barinov.
>>>>>
>>>>> Signed-off-by: Vladimir Barinov <vladimir.barinov@cogentembedded.com>
>>>>> Signed-off-by: Sergei Shtylyov <sergei.shtylyov@cogentembedded.com>
>>>>
>>>>
>>>>> --- renesas.orig/arch/arm64/boot/dts/renesas/r8a77970-eagle.dts
>>>>> +++ renesas/arch/arm64/boot/dts/renesas/r8a77970-eagle.dts
>>>
>>> [...]
>>>>>
>>>>> @@ -43,3 +44,14 @@
>>>>>    &scif0 {
>>>>>           status = "okay";
>>>>>    };
>>>>> +
>>>>> +&avb {
>>>>> +       renesas,no-ether-link;
>>>>> +       phy-handle = <&phy0>;
>>>>> +       status = "okay";
>>>>> +
>>>>> +       phy0: ethernet-phy@0 {
>>>>> +               rxc-skew-ps = <1500>;
>>>>> +               reg = <0>;
>>>>
>>>>
>>>> Any specific reason why you don't want to wire up the interrupt?
>>>>
>>>>                   interrupt-parent = <&gpio2>;
>>>
>>>
>>>     I thought it's quite obvious -- we don't have GPIOs yet, and GPIOs seem
>>> to require PFC.
>>
>> Of course. And these can be added later.
>>
>> Reviewed-by: Geert Uytterhoeven <geert+renesas@glider.be>
> 
> Thanks, applied.

    Still not seeing any devel branch updates/tags -- have you pushed it?

MBR, Sergei

[PATCH v2 8/8] arm64: dts: renesas: eagle: add EtherAVB support

[Sergei Shtylyov]

On 10/05/2017 12:05 PM, Simon Horman wrote:

>>>>> Define the Eagle board  dependent part of the EtherAVB device node.
>>>>> Enable DHCP  and NFS root for the kernel booting.
>>>>>
>>>>> Based  on the original (and large) patch by Vladimir Barinov.
>>>>>
>>>>> Signed-off-by: Vladimir Barinov <vladimir.barinov@cogentembedded.com>
>>>>> Signed-off-by: Sergei Shtylyov <sergei.shtylyov@cogentembedded.com>
>>>>
>>>>
>>>>> --- renesas.orig/arch/arm64/boot/dts/renesas/r8a77970-eagle.dts
>>>>> +++ renesas/arch/arm64/boot/dts/renesas/r8a77970-eagle.dts
>>>
>>> [...]
>>>>>
>>>>> @@ -43,3 +44,14 @@
>>>>>    &scif0 {
>>>>>           status = "okay";
>>>>>    };
>>>>> +
>>>>> +&avb {
>>>>> +       renesas,no-ether-link;
>>>>> +       phy-handle = <&phy0>;
>>>>> +       status = "okay";
>>>>> +
>>>>> +       phy0: ethernet-phy at 0 {
>>>>> +               rxc-skew-ps = <1500>;
>>>>> +               reg = <0>;
>>>>
>>>>
>>>> Any specific reason why you don't want to wire up the interrupt?
>>>>
>>>>                   interrupt-parent = <&gpio2>;
>>>
>>>
>>>     I thought it's quite obvious -- we don't have GPIOs yet, and GPIOs seem
>>> to require PFC.
>>
>> Of course. And these can be added later.
>>
>> Reviewed-by: Geert Uytterhoeven <geert+renesas@glider.be>
> 
> Thanks, applied.

    Still not seeing any devel branch updates/tags -- have you pushed it?

MBR, Sergei

Re: [PATCH v2 8/8] arm64: dts: renesas: eagle: add EtherAVB support

[Simon Horman]

On Thu, Oct 05, 2017 at 08:18:38PM +0300, Sergei Shtylyov wrote:
> On 10/05/2017 12:05 PM, Simon Horman wrote:
> 
> >>>>>Define the Eagle board  dependent part of the EtherAVB device node.
> >>>>>Enable DHCP  and NFS root for the kernel booting.
> >>>>>
> >>>>>Based  on the original (and large) patch by Vladimir Barinov.
> >>>>>
> >>>>>Signed-off-by: Vladimir Barinov <vladimir.barinov@cogentembedded.com>
> >>>>>Signed-off-by: Sergei Shtylyov <sergei.shtylyov@cogentembedded.com>
> >>>>
> >>>>
> >>>>>--- renesas.orig/arch/arm64/boot/dts/renesas/r8a77970-eagle.dts
> >>>>>+++ renesas/arch/arm64/boot/dts/renesas/r8a77970-eagle.dts
> >>>
> >>>[...]
> >>>>>
> >>>>>@@ -43,3 +44,14 @@
> >>>>>   &scif0 {
> >>>>>          status = "okay";
> >>>>>   };
> >>>>>+
> >>>>>+&avb {
> >>>>>+       renesas,no-ether-link;
> >>>>>+       phy-handle = <&phy0>;
> >>>>>+       status = "okay";
> >>>>>+
> >>>>>+       phy0: ethernet-phy@0 {
> >>>>>+               rxc-skew-ps = <1500>;
> >>>>>+               reg = <0>;
> >>>>
> >>>>
> >>>>Any specific reason why you don't want to wire up the interrupt?
> >>>>
> >>>>                  interrupt-parent = <&gpio2>;
> >>>
> >>>
> >>>    I thought it's quite obvious -- we don't have GPIOs yet, and GPIOs seem
> >>>to require PFC.
> >>
> >>Of course. And these can be added later.
> >>
> >>Reviewed-by: Geert Uytterhoeven <geert+renesas@glider.be>
> >
> >Thanks, applied.
> 
>    Still not seeing any devel branch updates/tags -- have you pushed it?

Sorry, there was a bit of a delay. But it is there.

[PATCH v2 8/8] arm64: dts: renesas: eagle: add EtherAVB support

[Simon Horman]

On Thu, Oct 05, 2017 at 08:18:38PM +0300, Sergei Shtylyov wrote:
> On 10/05/2017 12:05 PM, Simon Horman wrote:
> 
> >>>>>Define the Eagle board  dependent part of the EtherAVB device node.
> >>>>>Enable DHCP  and NFS root for the kernel booting.
> >>>>>
> >>>>>Based  on the original (and large) patch by Vladimir Barinov.
> >>>>>
> >>>>>Signed-off-by: Vladimir Barinov <vladimir.barinov@cogentembedded.com>
> >>>>>Signed-off-by: Sergei Shtylyov <sergei.shtylyov@cogentembedded.com>
> >>>>
> >>>>
> >>>>>--- renesas.orig/arch/arm64/boot/dts/renesas/r8a77970-eagle.dts
> >>>>>+++ renesas/arch/arm64/boot/dts/renesas/r8a77970-eagle.dts
> >>>
> >>>[...]
> >>>>>
> >>>>>@@ -43,3 +44,14 @@
> >>>>>   &scif0 {
> >>>>>          status = "okay";
> >>>>>   };
> >>>>>+
> >>>>>+&avb {
> >>>>>+       renesas,no-ether-link;
> >>>>>+       phy-handle = <&phy0>;
> >>>>>+       status = "okay";
> >>>>>+
> >>>>>+       phy0: ethernet-phy at 0 {
> >>>>>+               rxc-skew-ps = <1500>;
> >>>>>+               reg = <0>;
> >>>>
> >>>>
> >>>>Any specific reason why you don't want to wire up the interrupt?
> >>>>
> >>>>                  interrupt-parent = <&gpio2>;
> >>>
> >>>
> >>>    I thought it's quite obvious -- we don't have GPIOs yet, and GPIOs seem
> >>>to require PFC.
> >>
> >>Of course. And these can be added later.
> >>
> >>Reviewed-by: Geert Uytterhoeven <geert+renesas@glider.be>
> >
> >Thanks, applied.
> 
>    Still not seeing any devel branch updates/tags -- have you pushed it?

Sorry, there was a bit of a delay. But it is there.

Re: [PATCH v2 8/8] arm64: dts: renesas: eagle: add EtherAVB support

[Sergei Shtylyov]

On 10/6/2017 12:31 PM, Simon Horman wrote:

>>>>>>> Define the Eagle board  dependent part of the EtherAVB device node.
>>>>>>> Enable DHCP  and NFS root for the kernel booting.
>>>>>>>
>>>>>>> Based  on the original (and large) patch by Vladimir Barinov.
>>>>>>>
>>>>>>> Signed-off-by: Vladimir Barinov <vladimir.barinov@cogentembedded.com>
>>>>>>> Signed-off-by: Sergei Shtylyov <sergei.shtylyov@cogentembedded.com>
>>>>>>
>>>>>>
>>>>>>> --- renesas.orig/arch/arm64/boot/dts/renesas/r8a77970-eagle.dts
>>>>>>> +++ renesas/arch/arm64/boot/dts/renesas/r8a77970-eagle.dts
>>>>>
>>>>> [...]
>>>>>>>
>>>>>>> @@ -43,3 +44,14 @@
>>>>>>>    &scif0 {
>>>>>>>           status = "okay";
>>>>>>>    };
>>>>>>> +
>>>>>>> +&avb {
>>>>>>> +       renesas,no-ether-link;
>>>>>>> +       phy-handle = <&phy0>;
>>>>>>> +       status = "okay";
>>>>>>> +
>>>>>>> +       phy0: ethernet-phy@0 {
>>>>>>> +               rxc-skew-ps = <1500>;
>>>>>>> +               reg = <0>;
>>>>>>
>>>>>>
>>>>>> Any specific reason why you don't want to wire up the interrupt?
>>>>>>
>>>>>>                   interrupt-parent = <&gpio2>;
>>>>>
>>>>>
>>>>>     I thought it's quite obvious -- we don't have GPIOs yet, and GPIOs seem
>>>>> to require PFC.
>>>>
>>>> Of course. And these can be added later.
>>>>
>>>> Reviewed-by: Geert Uytterhoeven <geert+renesas@glider.be>
>>>
>>> Thanks, applied.
>>
>>     Still not seeing any devel branch updates/tags -- have you pushed it?
> 
> Sorry, there was a bit of a delay. But it is there.

    Yeah, I saw it later yesterday, thanks!

MBR, Sergei

[PATCH v2 8/8] arm64: dts: renesas: eagle: add EtherAVB support

[Sergei Shtylyov]

On 10/6/2017 12:31 PM, Simon Horman wrote:

>>>>>>> Define the Eagle board  dependent part of the EtherAVB device node.
>>>>>>> Enable DHCP  and NFS root for the kernel booting.
>>>>>>>
>>>>>>> Based  on the original (and large) patch by Vladimir Barinov.
>>>>>>>
>>>>>>> Signed-off-by: Vladimir Barinov <vladimir.barinov@cogentembedded.com>
>>>>>>> Signed-off-by: Sergei Shtylyov <sergei.shtylyov@cogentembedded.com>
>>>>>>
>>>>>>
>>>>>>> --- renesas.orig/arch/arm64/boot/dts/renesas/r8a77970-eagle.dts
>>>>>>> +++ renesas/arch/arm64/boot/dts/renesas/r8a77970-eagle.dts
>>>>>
>>>>> [...]
>>>>>>>
>>>>>>> @@ -43,3 +44,14 @@
>>>>>>>    &scif0 {
>>>>>>>           status = "okay";
>>>>>>>    };
>>>>>>> +
>>>>>>> +&avb {
>>>>>>> +       renesas,no-ether-link;
>>>>>>> +       phy-handle = <&phy0>;
>>>>>>> +       status = "okay";
>>>>>>> +
>>>>>>> +       phy0: ethernet-phy at 0 {
>>>>>>> +               rxc-skew-ps = <1500>;
>>>>>>> +               reg = <0>;
>>>>>>
>>>>>>
>>>>>> Any specific reason why you don't want to wire up the interrupt?
>>>>>>
>>>>>>                   interrupt-parent = <&gpio2>;
>>>>>
>>>>>
>>>>>     I thought it's quite obvious -- we don't have GPIOs yet, and GPIOs seem
>>>>> to require PFC.
>>>>
>>>> Of course. And these can be added later.
>>>>
>>>> Reviewed-by: Geert Uytterhoeven <geert+renesas@glider.be>
>>>
>>> Thanks, applied.
>>
>>     Still not seeing any devel branch updates/tags -- have you pushed it?
> 
> Sorry, there was a bit of a delay. But it is there.

    Yeah, I saw it later yesterday, thanks!

MBR, Sergei

Re: [PATCH v2 8/8] arm64: dts: renesas: eagle: add EtherAVB support

[Sergei Shtylyov]

On 10/05/2017 12:05 PM, Simon Horman wrote:

>>> On 9/21/2017 4:08 PM, Geert Uytterhoeven wrote:
>>>>> Define the Eagle board  dependent part of the EtherAVB device node.
>>>>> Enable DHCP  and NFS root for the kernel booting.
>>>>>
>>>>> Based  on the original (and large) patch by Vladimir Barinov.
>>>>>
>>>>> Signed-off-by: Vladimir Barinov <vladimir.barinov-M4DtvfQ/ZS1MRgGoP+s0PdBPR1lH4CV8@public.gmane.org>
>>>>> Signed-off-by: Sergei Shtylyov <sergei.shtylyov-M4DtvfQ/ZS1MRgGoP+s0PdBPR1lH4CV8@public.gmane.org>
>>>>
>>>>
>>>>> --- renesas.orig/arch/arm64/boot/dts/renesas/r8a77970-eagle.dts
>>>>> +++ renesas/arch/arm64/boot/dts/renesas/r8a77970-eagle.dts
>>>
>>> [...]
>>>>>
>>>>> @@ -43,3 +44,14 @@
>>>>>    &scif0 {
>>>>>           status = "okay";
>>>>>    };
>>>>> +
>>>>> +&avb {
>>>>> +       renesas,no-ether-link;
>>>>> +       phy-handle = <&phy0>;
>>>>> +       status = "okay";
>>>>> +
>>>>> +       phy0: ethernet-phy@0 {
>>>>> +               rxc-skew-ps = <1500>;
>>>>> +               reg = <0>;
>>>>
>>>>
>>>> Any specific reason why you don't want to wire up the interrupt?
>>>>
>>>>                   interrupt-parent = <&gpio2>;
>>>
>>>
>>>     I thought it's quite obvious -- we don't have GPIOs yet, and GPIOs seem
>>> to require PFC.
>>
>> Of course. And these can be added later.
>>
>> Reviewed-by: Geert Uytterhoeven <geert+renesas-gXvu3+zWzMSzQB+pC5nmwQ@public.gmane.org>
> 
> Thanks, applied.

    Unfortunately, you applied an outdated version of this patch. :-(

MBR, Sergei
--
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 v2 8/8] arm64: dts: renesas: eagle: add EtherAVB support

[Sergei Shtylyov]

On 10/05/2017 12:05 PM, Simon Horman wrote:

>>> On 9/21/2017 4:08 PM, Geert Uytterhoeven wrote:
>>>>> Define the Eagle board  dependent part of the EtherAVB device node.
>>>>> Enable DHCP  and NFS root for the kernel booting.
>>>>>
>>>>> Based  on the original (and large) patch by Vladimir Barinov.
>>>>>
>>>>> Signed-off-by: Vladimir Barinov <vladimir.barinov@cogentembedded.com>
>>>>> Signed-off-by: Sergei Shtylyov <sergei.shtylyov@cogentembedded.com>
>>>>
>>>>
>>>>> --- renesas.orig/arch/arm64/boot/dts/renesas/r8a77970-eagle.dts
>>>>> +++ renesas/arch/arm64/boot/dts/renesas/r8a77970-eagle.dts
>>>
>>> [...]
>>>>>
>>>>> @@ -43,3 +44,14 @@
>>>>>    &scif0 {
>>>>>           status = "okay";
>>>>>    };
>>>>> +
>>>>> +&avb {
>>>>> +       renesas,no-ether-link;
>>>>> +       phy-handle = <&phy0>;
>>>>> +       status = "okay";
>>>>> +
>>>>> +       phy0: ethernet-phy@0 {
>>>>> +               rxc-skew-ps = <1500>;
>>>>> +               reg = <0>;
>>>>
>>>>
>>>> Any specific reason why you don't want to wire up the interrupt?
>>>>
>>>>                   interrupt-parent = <&gpio2>;
>>>
>>>
>>>     I thought it's quite obvious -- we don't have GPIOs yet, and GPIOs seem
>>> to require PFC.
>>
>> Of course. And these can be added later.
>>
>> Reviewed-by: Geert Uytterhoeven <geert+renesas@glider.be>
> 
> Thanks, applied.

    Unfortunately, you applied an outdated version of this patch. :-(

MBR, Sergei

[PATCH v2 8/8] arm64: dts: renesas: eagle: add EtherAVB support

[Sergei Shtylyov]

On 10/05/2017 12:05 PM, Simon Horman wrote:

>>> On 9/21/2017 4:08 PM, Geert Uytterhoeven wrote:
>>>>> Define the Eagle board  dependent part of the EtherAVB device node.
>>>>> Enable DHCP  and NFS root for the kernel booting.
>>>>>
>>>>> Based  on the original (and large) patch by Vladimir Barinov.
>>>>>
>>>>> Signed-off-by: Vladimir Barinov <vladimir.barinov@cogentembedded.com>
>>>>> Signed-off-by: Sergei Shtylyov <sergei.shtylyov@cogentembedded.com>
>>>>
>>>>
>>>>> --- renesas.orig/arch/arm64/boot/dts/renesas/r8a77970-eagle.dts
>>>>> +++ renesas/arch/arm64/boot/dts/renesas/r8a77970-eagle.dts
>>>
>>> [...]
>>>>>
>>>>> @@ -43,3 +44,14 @@
>>>>>    &scif0 {
>>>>>           status = "okay";
>>>>>    };
>>>>> +
>>>>> +&avb {
>>>>> +       renesas,no-ether-link;
>>>>> +       phy-handle = <&phy0>;
>>>>> +       status = "okay";
>>>>> +
>>>>> +       phy0: ethernet-phy at 0 {
>>>>> +               rxc-skew-ps = <1500>;
>>>>> +               reg = <0>;
>>>>
>>>>
>>>> Any specific reason why you don't want to wire up the interrupt?
>>>>
>>>>                   interrupt-parent = <&gpio2>;
>>>
>>>
>>>     I thought it's quite obvious -- we don't have GPIOs yet, and GPIOs seem
>>> to require PFC.
>>
>> Of course. And these can be added later.
>>
>> Reviewed-by: Geert Uytterhoeven <geert+renesas@glider.be>
> 
> Thanks, applied.

    Unfortunately, you applied an outdated version of this patch. :-(

MBR, Sergei

Re: [PATCH v2 8/8] arm64: dts: renesas: eagle: add EtherAVB support

[Simon Horman]

On Mon, Oct 09, 2017 at 08:57:42PM +0300, Sergei Shtylyov wrote:
> On 10/05/2017 12:05 PM, Simon Horman wrote:
> 
> >>>On 9/21/2017 4:08 PM, Geert Uytterhoeven wrote:
> >>>>>Define the Eagle board  dependent part of the EtherAVB device node.
> >>>>>Enable DHCP  and NFS root for the kernel booting.
> >>>>>
> >>>>>Based  on the original (and large) patch by Vladimir Barinov.
> >>>>>
> >>>>>Signed-off-by: Vladimir Barinov <vladimir.barinov-M4DtvfQ/ZS1MRgGoP+s0PdBPR1lH4CV8@public.gmane.org>
> >>>>>Signed-off-by: Sergei Shtylyov <sergei.shtylyov-M4DtvfQ/ZS1MRgGoP+s0PdBPR1lH4CV8@public.gmane.org>
> >>>>
> >>>>
> >>>>>--- renesas.orig/arch/arm64/boot/dts/renesas/r8a77970-eagle.dts
> >>>>>+++ renesas/arch/arm64/boot/dts/renesas/r8a77970-eagle.dts
> >>>
> >>>[...]
> >>>>>
> >>>>>@@ -43,3 +44,14 @@
> >>>>>   &scif0 {
> >>>>>          status = "okay";
> >>>>>   };
> >>>>>+
> >>>>>+&avb {
> >>>>>+       renesas,no-ether-link;
> >>>>>+       phy-handle = <&phy0>;
> >>>>>+       status = "okay";
> >>>>>+
> >>>>>+       phy0: ethernet-phy@0 {
> >>>>>+               rxc-skew-ps = <1500>;
> >>>>>+               reg = <0>;
> >>>>
> >>>>
> >>>>Any specific reason why you don't want to wire up the interrupt?
> >>>>
> >>>>                  interrupt-parent = <&gpio2>;
> >>>
> >>>
> >>>    I thought it's quite obvious -- we don't have GPIOs yet, and GPIOs seem
> >>>to require PFC.
> >>
> >>Of course. And these can be added later.
> >>
> >>Reviewed-by: Geert Uytterhoeven <geert+renesas-gXvu3+zWzMSzQB+pC5nmwQ@public.gmane.org>
> >
> >Thanks, applied.
> 
>    Unfortunately, you applied an outdated version of this patch. :-(

Sorry, I think patchwork has been playing up of late - or my use of it!

Could you repost the correct patch so I am sure to get the right one
the second time around?
--
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 v2 8/8] arm64: dts: renesas: eagle: add EtherAVB support

[Simon Horman]

On Mon, Oct 09, 2017 at 08:57:42PM +0300, Sergei Shtylyov wrote:
> On 10/05/2017 12:05 PM, Simon Horman wrote:
> 
> >>>On 9/21/2017 4:08 PM, Geert Uytterhoeven wrote:
> >>>>>Define the Eagle board  dependent part of the EtherAVB device node.
> >>>>>Enable DHCP  and NFS root for the kernel booting.
> >>>>>
> >>>>>Based  on the original (and large) patch by Vladimir Barinov.
> >>>>>
> >>>>>Signed-off-by: Vladimir Barinov <vladimir.barinov@cogentembedded.com>
> >>>>>Signed-off-by: Sergei Shtylyov <sergei.shtylyov@cogentembedded.com>
> >>>>
> >>>>
> >>>>>--- renesas.orig/arch/arm64/boot/dts/renesas/r8a77970-eagle.dts
> >>>>>+++ renesas/arch/arm64/boot/dts/renesas/r8a77970-eagle.dts
> >>>
> >>>[...]
> >>>>>
> >>>>>@@ -43,3 +44,14 @@
> >>>>>   &scif0 {
> >>>>>          status = "okay";
> >>>>>   };
> >>>>>+
> >>>>>+&avb {
> >>>>>+       renesas,no-ether-link;
> >>>>>+       phy-handle = <&phy0>;
> >>>>>+       status = "okay";
> >>>>>+
> >>>>>+       phy0: ethernet-phy@0 {
> >>>>>+               rxc-skew-ps = <1500>;
> >>>>>+               reg = <0>;
> >>>>
> >>>>
> >>>>Any specific reason why you don't want to wire up the interrupt?
> >>>>
> >>>>                  interrupt-parent = <&gpio2>;
> >>>
> >>>
> >>>    I thought it's quite obvious -- we don't have GPIOs yet, and GPIOs seem
> >>>to require PFC.
> >>
> >>Of course. And these can be added later.
> >>
> >>Reviewed-by: Geert Uytterhoeven <geert+renesas@glider.be>
> >
> >Thanks, applied.
> 
>    Unfortunately, you applied an outdated version of this patch. :-(

Sorry, I think patchwork has been playing up of late - or my use of it!

Could you repost the correct patch so I am sure to get the right one
the second time around?

[PATCH v2 8/8] arm64: dts: renesas: eagle: add EtherAVB support

[Simon Horman]

On Mon, Oct 09, 2017 at 08:57:42PM +0300, Sergei Shtylyov wrote:
> On 10/05/2017 12:05 PM, Simon Horman wrote:
> 
> >>>On 9/21/2017 4:08 PM, Geert Uytterhoeven wrote:
> >>>>>Define the Eagle board  dependent part of the EtherAVB device node.
> >>>>>Enable DHCP  and NFS root for the kernel booting.
> >>>>>
> >>>>>Based  on the original (and large) patch by Vladimir Barinov.
> >>>>>
> >>>>>Signed-off-by: Vladimir Barinov <vladimir.barinov@cogentembedded.com>
> >>>>>Signed-off-by: Sergei Shtylyov <sergei.shtylyov@cogentembedded.com>
> >>>>
> >>>>
> >>>>>--- renesas.orig/arch/arm64/boot/dts/renesas/r8a77970-eagle.dts
> >>>>>+++ renesas/arch/arm64/boot/dts/renesas/r8a77970-eagle.dts
> >>>
> >>>[...]
> >>>>>
> >>>>>@@ -43,3 +44,14 @@
> >>>>>   &scif0 {
> >>>>>          status = "okay";
> >>>>>   };
> >>>>>+
> >>>>>+&avb {
> >>>>>+       renesas,no-ether-link;
> >>>>>+       phy-handle = <&phy0>;
> >>>>>+       status = "okay";
> >>>>>+
> >>>>>+       phy0: ethernet-phy at 0 {
> >>>>>+               rxc-skew-ps = <1500>;
> >>>>>+               reg = <0>;
> >>>>
> >>>>
> >>>>Any specific reason why you don't want to wire up the interrupt?
> >>>>
> >>>>                  interrupt-parent = <&gpio2>;
> >>>
> >>>
> >>>    I thought it's quite obvious -- we don't have GPIOs yet, and GPIOs seem
> >>>to require PFC.
> >>
> >>Of course. And these can be added later.
> >>
> >>Reviewed-by: Geert Uytterhoeven <geert+renesas@glider.be>
> >
> >Thanks, applied.
> 
>    Unfortunately, you applied an outdated version of this patch. :-(

Sorry, I think patchwork has been playing up of late - or my use of it!

Could you repost the correct patch so I am sure to get the right one
the second time around?

Re: [PATCH v2 8/8] arm64: dts: renesas: eagle: add EtherAVB support

[Simon Horman]

On Tue, Oct 10, 2017 at 09:33:46AM +0200, Simon Horman wrote:
> On Mon, Oct 09, 2017 at 08:57:42PM +0300, Sergei Shtylyov wrote:
> > On 10/05/2017 12:05 PM, Simon Horman wrote:
> > 
> > >>>On 9/21/2017 4:08 PM, Geert Uytterhoeven wrote:
> > >>>>>Define the Eagle board  dependent part of the EtherAVB device node.
> > >>>>>Enable DHCP  and NFS root for the kernel booting.
> > >>>>>
> > >>>>>Based  on the original (and large) patch by Vladimir Barinov.
> > >>>>>
> > >>>>>Signed-off-by: Vladimir Barinov <vladimir.barinov-M4DtvfQ/ZS1MRgGoP+s0PdBPR1lH4CV8@public.gmane.org>
> > >>>>>Signed-off-by: Sergei Shtylyov <sergei.shtylyov-M4DtvfQ/ZS1MRgGoP+s0PdBPR1lH4CV8@public.gmane.org>
> > >>>>
> > >>>>
> > >>>>>--- renesas.orig/arch/arm64/boot/dts/renesas/r8a77970-eagle.dts
> > >>>>>+++ renesas/arch/arm64/boot/dts/renesas/r8a77970-eagle.dts
> > >>>
> > >>>[...]
> > >>>>>
> > >>>>>@@ -43,3 +44,14 @@
> > >>>>>   &scif0 {
> > >>>>>          status = "okay";
> > >>>>>   };
> > >>>>>+
> > >>>>>+&avb {
> > >>>>>+       renesas,no-ether-link;
> > >>>>>+       phy-handle = <&phy0>;
> > >>>>>+       status = "okay";
> > >>>>>+
> > >>>>>+       phy0: ethernet-phy@0 {
> > >>>>>+               rxc-skew-ps = <1500>;
> > >>>>>+               reg = <0>;
> > >>>>
> > >>>>
> > >>>>Any specific reason why you don't want to wire up the interrupt?
> > >>>>
> > >>>>                  interrupt-parent = <&gpio2>;
> > >>>
> > >>>
> > >>>    I thought it's quite obvious -- we don't have GPIOs yet, and GPIOs seem
> > >>>to require PFC.
> > >>
> > >>Of course. And these can be added later.
> > >>
> > >>Reviewed-by: Geert Uytterhoeven <geert+renesas-gXvu3+zWzMSzQB+pC5nmwQ@public.gmane.org>
> > >
> > >Thanks, applied.
> > 
> >    Unfortunately, you applied an outdated version of this patch. :-(
> 
> Sorry, I think patchwork has been playing up of late - or my use of it!
> 
> Could you repost the correct patch so I am sure to get the right one
> the second time around?

Please ignore that request. I'll go ahead and apply v2.
--
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 v2 8/8] arm64: dts: renesas: eagle: add EtherAVB support

[Simon Horman]

On Tue, Oct 10, 2017 at 09:33:46AM +0200, Simon Horman wrote:
> On Mon, Oct 09, 2017 at 08:57:42PM +0300, Sergei Shtylyov wrote:
> > On 10/05/2017 12:05 PM, Simon Horman wrote:
> > 
> > >>>On 9/21/2017 4:08 PM, Geert Uytterhoeven wrote:
> > >>>>>Define the Eagle board  dependent part of the EtherAVB device node.
> > >>>>>Enable DHCP  and NFS root for the kernel booting.
> > >>>>>
> > >>>>>Based  on the original (and large) patch by Vladimir Barinov.
> > >>>>>
> > >>>>>Signed-off-by: Vladimir Barinov <vladimir.barinov@cogentembedded.com>
> > >>>>>Signed-off-by: Sergei Shtylyov <sergei.shtylyov@cogentembedded.com>
> > >>>>
> > >>>>
> > >>>>>--- renesas.orig/arch/arm64/boot/dts/renesas/r8a77970-eagle.dts
> > >>>>>+++ renesas/arch/arm64/boot/dts/renesas/r8a77970-eagle.dts
> > >>>
> > >>>[...]
> > >>>>>
> > >>>>>@@ -43,3 +44,14 @@
> > >>>>>   &scif0 {
> > >>>>>          status = "okay";
> > >>>>>   };
> > >>>>>+
> > >>>>>+&avb {
> > >>>>>+       renesas,no-ether-link;
> > >>>>>+       phy-handle = <&phy0>;
> > >>>>>+       status = "okay";
> > >>>>>+
> > >>>>>+       phy0: ethernet-phy@0 {
> > >>>>>+               rxc-skew-ps = <1500>;
> > >>>>>+               reg = <0>;
> > >>>>
> > >>>>
> > >>>>Any specific reason why you don't want to wire up the interrupt?
> > >>>>
> > >>>>                  interrupt-parent = <&gpio2>;
> > >>>
> > >>>
> > >>>    I thought it's quite obvious -- we don't have GPIOs yet, and GPIOs seem
> > >>>to require PFC.
> > >>
> > >>Of course. And these can be added later.
> > >>
> > >>Reviewed-by: Geert Uytterhoeven <geert+renesas@glider.be>
> > >
> > >Thanks, applied.
> > 
> >    Unfortunately, you applied an outdated version of this patch. :-(
> 
> Sorry, I think patchwork has been playing up of late - or my use of it!
> 
> Could you repost the correct patch so I am sure to get the right one
> the second time around?

Please ignore that request. I'll go ahead and apply v2.

[PATCH v2 8/8] arm64: dts: renesas: eagle: add EtherAVB support

[Simon Horman]

On Tue, Oct 10, 2017 at 09:33:46AM +0200, Simon Horman wrote:
> On Mon, Oct 09, 2017 at 08:57:42PM +0300, Sergei Shtylyov wrote:
> > On 10/05/2017 12:05 PM, Simon Horman wrote:
> > 
> > >>>On 9/21/2017 4:08 PM, Geert Uytterhoeven wrote:
> > >>>>>Define the Eagle board  dependent part of the EtherAVB device node.
> > >>>>>Enable DHCP  and NFS root for the kernel booting.
> > >>>>>
> > >>>>>Based  on the original (and large) patch by Vladimir Barinov.
> > >>>>>
> > >>>>>Signed-off-by: Vladimir Barinov <vladimir.barinov@cogentembedded.com>
> > >>>>>Signed-off-by: Sergei Shtylyov <sergei.shtylyov@cogentembedded.com>
> > >>>>
> > >>>>
> > >>>>>--- renesas.orig/arch/arm64/boot/dts/renesas/r8a77970-eagle.dts
> > >>>>>+++ renesas/arch/arm64/boot/dts/renesas/r8a77970-eagle.dts
> > >>>
> > >>>[...]
> > >>>>>
> > >>>>>@@ -43,3 +44,14 @@
> > >>>>>   &scif0 {
> > >>>>>          status = "okay";
> > >>>>>   };
> > >>>>>+
> > >>>>>+&avb {
> > >>>>>+       renesas,no-ether-link;
> > >>>>>+       phy-handle = <&phy0>;
> > >>>>>+       status = "okay";
> > >>>>>+
> > >>>>>+       phy0: ethernet-phy at 0 {
> > >>>>>+               rxc-skew-ps = <1500>;
> > >>>>>+               reg = <0>;
> > >>>>
> > >>>>
> > >>>>Any specific reason why you don't want to wire up the interrupt?
> > >>>>
> > >>>>                  interrupt-parent = <&gpio2>;
> > >>>
> > >>>
> > >>>    I thought it's quite obvious -- we don't have GPIOs yet, and GPIOs seem
> > >>>to require PFC.
> > >>
> > >>Of course. And these can be added later.
> > >>
> > >>Reviewed-by: Geert Uytterhoeven <geert+renesas@glider.be>
> > >
> > >Thanks, applied.
> > 
> >    Unfortunately, you applied an outdated version of this patch. :-(
> 
> Sorry, I think patchwork has been playing up of late - or my use of it!
> 
> Could you repost the correct patch so I am sure to get the right one
> the second time around?

Please ignore that request. I'll go ahead and apply v2.

Re: [PATCH v2 8/8] arm64: dts: renesas: eagle: add EtherAVB support

[Simon Horman]

On Tue, Oct 10, 2017 at 09:48:42AM +0200, Simon Horman wrote:
> On Tue, Oct 10, 2017 at 09:33:46AM +0200, Simon Horman wrote:
> > On Mon, Oct 09, 2017 at 08:57:42PM +0300, Sergei Shtylyov wrote:
> > > On 10/05/2017 12:05 PM, Simon Horman wrote:
> > > 
> > > >>>On 9/21/2017 4:08 PM, Geert Uytterhoeven wrote:
> > > >>>>>Define the Eagle board  dependent part of the EtherAVB device node.
> > > >>>>>Enable DHCP  and NFS root for the kernel booting.
> > > >>>>>
> > > >>>>>Based  on the original (and large) patch by Vladimir Barinov.
> > > >>>>>
> > > >>>>>Signed-off-by: Vladimir Barinov <vladimir.barinov-M4DtvfQ/ZS1MRgGoP+s0PdBPR1lH4CV8@public.gmane.org>
> > > >>>>>Signed-off-by: Sergei Shtylyov <sergei.shtylyov-M4DtvfQ/ZS1MRgGoP+s0PdBPR1lH4CV8@public.gmane.org>
> > > >>>>
> > > >>>>
> > > >>>>>--- renesas.orig/arch/arm64/boot/dts/renesas/r8a77970-eagle.dts
> > > >>>>>+++ renesas/arch/arm64/boot/dts/renesas/r8a77970-eagle.dts
> > > >>>
> > > >>>[...]
> > > >>>>>
> > > >>>>>@@ -43,3 +44,14 @@
> > > >>>>>   &scif0 {
> > > >>>>>          status = "okay";
> > > >>>>>   };
> > > >>>>>+
> > > >>>>>+&avb {
> > > >>>>>+       renesas,no-ether-link;
> > > >>>>>+       phy-handle = <&phy0>;
> > > >>>>>+       status = "okay";
> > > >>>>>+
> > > >>>>>+       phy0: ethernet-phy@0 {
> > > >>>>>+               rxc-skew-ps = <1500>;
> > > >>>>>+               reg = <0>;
> > > >>>>
> > > >>>>
> > > >>>>Any specific reason why you don't want to wire up the interrupt?
> > > >>>>
> > > >>>>                  interrupt-parent = <&gpio2>;
> > > >>>
> > > >>>
> > > >>>    I thought it's quite obvious -- we don't have GPIOs yet, and GPIOs seem
> > > >>>to require PFC.
> > > >>
> > > >>Of course. And these can be added later.
> > > >>
> > > >>Reviewed-by: Geert Uytterhoeven <geert+renesas-gXvu3+zWzMSzQB+pC5nmwQ@public.gmane.org>
> > > >
> > > >Thanks, applied.
> > > 
> > >    Unfortunately, you applied an outdated version of this patch. :-(
> > 
> > Sorry, I think patchwork has been playing up of late - or my use of it!
> > 
> > Could you repost the correct patch so I am sure to get the right one
> > the second time around?
> 
> Please ignore that request. I'll go ahead and apply v2.

Done, please check renesas-next-20171010-v4.14-rc1
--
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 v2 8/8] arm64: dts: renesas: eagle: add EtherAVB support

[Simon Horman]

On Tue, Oct 10, 2017 at 09:48:42AM +0200, Simon Horman wrote:
> On Tue, Oct 10, 2017 at 09:33:46AM +0200, Simon Horman wrote:
> > On Mon, Oct 09, 2017 at 08:57:42PM +0300, Sergei Shtylyov wrote:
> > > On 10/05/2017 12:05 PM, Simon Horman wrote:
> > > 
> > > >>>On 9/21/2017 4:08 PM, Geert Uytterhoeven wrote:
> > > >>>>>Define the Eagle board  dependent part of the EtherAVB device node.
> > > >>>>>Enable DHCP  and NFS root for the kernel booting.
> > > >>>>>
> > > >>>>>Based  on the original (and large) patch by Vladimir Barinov.
> > > >>>>>
> > > >>>>>Signed-off-by: Vladimir Barinov <vladimir.barinov@cogentembedded.com>
> > > >>>>>Signed-off-by: Sergei Shtylyov <sergei.shtylyov@cogentembedded.com>
> > > >>>>
> > > >>>>
> > > >>>>>--- renesas.orig/arch/arm64/boot/dts/renesas/r8a77970-eagle.dts
> > > >>>>>+++ renesas/arch/arm64/boot/dts/renesas/r8a77970-eagle.dts
> > > >>>
> > > >>>[...]
> > > >>>>>
> > > >>>>>@@ -43,3 +44,14 @@
> > > >>>>>   &scif0 {
> > > >>>>>          status = "okay";
> > > >>>>>   };
> > > >>>>>+
> > > >>>>>+&avb {
> > > >>>>>+       renesas,no-ether-link;
> > > >>>>>+       phy-handle = <&phy0>;
> > > >>>>>+       status = "okay";
> > > >>>>>+
> > > >>>>>+       phy0: ethernet-phy@0 {
> > > >>>>>+               rxc-skew-ps = <1500>;
> > > >>>>>+               reg = <0>;
> > > >>>>
> > > >>>>
> > > >>>>Any specific reason why you don't want to wire up the interrupt?
> > > >>>>
> > > >>>>                  interrupt-parent = <&gpio2>;
> > > >>>
> > > >>>
> > > >>>    I thought it's quite obvious -- we don't have GPIOs yet, and GPIOs seem
> > > >>>to require PFC.
> > > >>
> > > >>Of course. And these can be added later.
> > > >>
> > > >>Reviewed-by: Geert Uytterhoeven <geert+renesas@glider.be>
> > > >
> > > >Thanks, applied.
> > > 
> > >    Unfortunately, you applied an outdated version of this patch. :-(
> > 
> > Sorry, I think patchwork has been playing up of late - or my use of it!
> > 
> > Could you repost the correct patch so I am sure to get the right one
> > the second time around?
> 
> Please ignore that request. I'll go ahead and apply v2.

Done, please check renesas-next-20171010-v4.14-rc1

[PATCH v2 8/8] arm64: dts: renesas: eagle: add EtherAVB support

[Simon Horman]

On Tue, Oct 10, 2017 at 09:48:42AM +0200, Simon Horman wrote:
> On Tue, Oct 10, 2017 at 09:33:46AM +0200, Simon Horman wrote:
> > On Mon, Oct 09, 2017 at 08:57:42PM +0300, Sergei Shtylyov wrote:
> > > On 10/05/2017 12:05 PM, Simon Horman wrote:
> > > 
> > > >>>On 9/21/2017 4:08 PM, Geert Uytterhoeven wrote:
> > > >>>>>Define the Eagle board  dependent part of the EtherAVB device node.
> > > >>>>>Enable DHCP  and NFS root for the kernel booting.
> > > >>>>>
> > > >>>>>Based  on the original (and large) patch by Vladimir Barinov.
> > > >>>>>
> > > >>>>>Signed-off-by: Vladimir Barinov <vladimir.barinov@cogentembedded.com>
> > > >>>>>Signed-off-by: Sergei Shtylyov <sergei.shtylyov@cogentembedded.com>
> > > >>>>
> > > >>>>
> > > >>>>>--- renesas.orig/arch/arm64/boot/dts/renesas/r8a77970-eagle.dts
> > > >>>>>+++ renesas/arch/arm64/boot/dts/renesas/r8a77970-eagle.dts
> > > >>>
> > > >>>[...]
> > > >>>>>
> > > >>>>>@@ -43,3 +44,14 @@
> > > >>>>>   &scif0 {
> > > >>>>>          status = "okay";
> > > >>>>>   };
> > > >>>>>+
> > > >>>>>+&avb {
> > > >>>>>+       renesas,no-ether-link;
> > > >>>>>+       phy-handle = <&phy0>;
> > > >>>>>+       status = "okay";
> > > >>>>>+
> > > >>>>>+       phy0: ethernet-phy at 0 {
> > > >>>>>+               rxc-skew-ps = <1500>;
> > > >>>>>+               reg = <0>;
> > > >>>>
> > > >>>>
> > > >>>>Any specific reason why you don't want to wire up the interrupt?
> > > >>>>
> > > >>>>                  interrupt-parent = <&gpio2>;
> > > >>>
> > > >>>
> > > >>>    I thought it's quite obvious -- we don't have GPIOs yet, and GPIOs seem
> > > >>>to require PFC.
> > > >>
> > > >>Of course. And these can be added later.
> > > >>
> > > >>Reviewed-by: Geert Uytterhoeven <geert+renesas@glider.be>
> > > >
> > > >Thanks, applied.
> > > 
> > >    Unfortunately, you applied an outdated version of this patch. :-(
> > 
> > Sorry, I think patchwork has been playing up of late - or my use of it!
> > 
> > Could you repost the correct patch so I am sure to get the right one
> > the second time around?
> 
> Please ignore that request. I'll go ahead and apply v2.

Done, please check renesas-next-20171010-v4.14-rc1