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

[Simon Horman <horms-/R6kz+dDXgpPR4JQBCEnsQ@public.gmane.org>]

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