* [PATCH v2 8/8] arm64: dts: renesas: eagle: add EtherAVB support
@ 2017-09-15 19:43 Sergei Shtylyov
2017-09-21 13:08 ` Geert Uytterhoeven
0 siblings, 1 reply; 12+ messages in thread
From: Sergei Shtylyov @ 2017-09-15 19:43 UTC (permalink / raw)
To: linux-arm-kernel
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>;
+ };
+};
^ permalink raw reply [flat|nested] 12+ messages in thread* [PATCH v2 8/8] arm64: dts: renesas: eagle: add EtherAVB support
2017-09-15 19:43 [PATCH v2 8/8] arm64: dts: renesas: eagle: add EtherAVB support Sergei Shtylyov
@ 2017-09-21 13:08 ` Geert Uytterhoeven
2017-09-21 16:03 ` Sergei Shtylyov
0 siblings, 1 reply; 12+ messages in thread
From: Geert Uytterhoeven @ 2017-09-21 13:08 UTC (permalink / raw)
To: linux-arm-kernel
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
^ permalink raw reply [flat|nested] 12+ messages in thread* [PATCH v2 8/8] arm64: dts: renesas: eagle: add EtherAVB support
2017-09-21 13:08 ` Geert Uytterhoeven
@ 2017-09-21 16:03 ` Sergei Shtylyov
2017-09-21 16:09 ` Geert Uytterhoeven
0 siblings, 1 reply; 12+ messages in thread
From: Sergei Shtylyov @ 2017-09-21 16:03 UTC (permalink / raw)
To: linux-arm-kernel
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
^ permalink raw reply [flat|nested] 12+ messages in thread* [PATCH v2 8/8] arm64: dts: renesas: eagle: add EtherAVB support
2017-09-21 16:03 ` Sergei Shtylyov
@ 2017-09-21 16:09 ` Geert Uytterhoeven
2017-10-05 9:05 ` Simon Horman
0 siblings, 1 reply; 12+ messages in thread
From: Geert Uytterhoeven @ 2017-09-21 16:09 UTC (permalink / raw)
To: linux-arm-kernel
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
^ permalink raw reply [flat|nested] 12+ messages in thread* [PATCH v2 8/8] arm64: dts: renesas: eagle: add EtherAVB support
2017-09-21 16:09 ` Geert Uytterhoeven
@ 2017-10-05 9:05 ` Simon Horman
2017-10-05 17:18 ` Sergei Shtylyov
2017-10-09 17:57 ` Sergei Shtylyov
0 siblings, 2 replies; 12+ messages in thread
From: Simon Horman @ 2017-10-05 9:05 UTC (permalink / raw)
To: linux-arm-kernel
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.
^ permalink raw reply [flat|nested] 12+ messages in thread* [PATCH v2 8/8] arm64: dts: renesas: eagle: add EtherAVB support
2017-10-05 9:05 ` Simon Horman
@ 2017-10-05 17:18 ` Sergei Shtylyov
2017-10-06 9:31 ` Simon Horman
2017-10-09 17:57 ` Sergei Shtylyov
1 sibling, 1 reply; 12+ messages in thread
From: Sergei Shtylyov @ 2017-10-05 17:18 UTC (permalink / raw)
To: linux-arm-kernel
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
^ permalink raw reply [flat|nested] 12+ messages in thread* [PATCH v2 8/8] arm64: dts: renesas: eagle: add EtherAVB support
2017-10-05 17:18 ` Sergei Shtylyov
@ 2017-10-06 9:31 ` Simon Horman
2017-10-06 9:34 ` Sergei Shtylyov
0 siblings, 1 reply; 12+ messages in thread
From: Simon Horman @ 2017-10-06 9:31 UTC (permalink / raw)
To: linux-arm-kernel
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.
^ permalink raw reply [flat|nested] 12+ messages in thread* [PATCH v2 8/8] arm64: dts: renesas: eagle: add EtherAVB support
2017-10-06 9:31 ` Simon Horman
@ 2017-10-06 9:34 ` Sergei Shtylyov
0 siblings, 0 replies; 12+ messages in thread
From: Sergei Shtylyov @ 2017-10-06 9:34 UTC (permalink / raw)
To: linux-arm-kernel
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
^ permalink raw reply [flat|nested] 12+ messages in thread
* [PATCH v2 8/8] arm64: dts: renesas: eagle: add EtherAVB support
2017-10-05 9:05 ` Simon Horman
2017-10-05 17:18 ` Sergei Shtylyov
@ 2017-10-09 17:57 ` Sergei Shtylyov
2017-10-10 7:33 ` Simon Horman
1 sibling, 1 reply; 12+ messages in thread
From: Sergei Shtylyov @ 2017-10-09 17:57 UTC (permalink / raw)
To: linux-arm-kernel
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
^ permalink raw reply [flat|nested] 12+ messages in thread* [PATCH v2 8/8] arm64: dts: renesas: eagle: add EtherAVB support
2017-10-09 17:57 ` Sergei Shtylyov
@ 2017-10-10 7:33 ` Simon Horman
2017-10-10 7:48 ` Simon Horman
0 siblings, 1 reply; 12+ messages in thread
From: Simon Horman @ 2017-10-10 7:33 UTC (permalink / raw)
To: linux-arm-kernel
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?
^ permalink raw reply [flat|nested] 12+ messages in thread* [PATCH v2 8/8] arm64: dts: renesas: eagle: add EtherAVB support
2017-10-10 7:33 ` Simon Horman
@ 2017-10-10 7:48 ` Simon Horman
2017-10-10 8:03 ` Simon Horman
0 siblings, 1 reply; 12+ messages in thread
From: Simon Horman @ 2017-10-10 7:48 UTC (permalink / raw)
To: linux-arm-kernel
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.
^ permalink raw reply [flat|nested] 12+ messages in thread* [PATCH v2 8/8] arm64: dts: renesas: eagle: add EtherAVB support
2017-10-10 7:48 ` Simon Horman
@ 2017-10-10 8:03 ` Simon Horman
0 siblings, 0 replies; 12+ messages in thread
From: Simon Horman @ 2017-10-10 8:03 UTC (permalink / raw)
To: linux-arm-kernel
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
^ permalink raw reply [flat|nested] 12+ messages in thread
end of thread, other threads:[~2017-10-10 8:03 UTC | newest]
Thread overview: 12+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2017-09-15 19:43 [PATCH v2 8/8] arm64: dts: renesas: eagle: add EtherAVB support Sergei Shtylyov
2017-09-21 13:08 ` Geert Uytterhoeven
2017-09-21 16:03 ` Sergei Shtylyov
2017-09-21 16:09 ` Geert Uytterhoeven
2017-10-05 9:05 ` Simon Horman
2017-10-05 17:18 ` Sergei Shtylyov
2017-10-06 9:31 ` Simon Horman
2017-10-06 9:34 ` Sergei Shtylyov
2017-10-09 17:57 ` Sergei Shtylyov
2017-10-10 7:33 ` Simon Horman
2017-10-10 7:48 ` Simon Horman
2017-10-10 8:03 ` Simon Horman
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).