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

[Simon Horman <horms@verge.net.au>]

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.