Hi Simon,

Thanks for the feedback.

> -----Original Message-----
> From: devicetree-ow...@vger.kernel.org <devicetree-
> ow...@vger.kernel.org> On Behalf Of Simon Horman
> Sent: 17 August 2018 09:45
> To: Biju Das <biju....@bp.renesas.com>
> Cc: Rob Herring <robh...@kernel.org>; Mark Rutland
> <mark.rutl...@arm.com>; Magnus Damm <magnus.d...@gmail.com>;
> linux-renesas-soc@vger.kernel.org; devicet...@vger.kernel.org; Geert
> Uytterhoeven <geert+rene...@glider.be>; Chris Paterson
> <chris.paters...@renesas.com>; Fabrizio Castro
> <fabrizio.cas...@bp.renesas.com>
> Subject: Re: [PATCH v3 5/5] ARM: dts: iwg23s-sbc: Add pinctl support for
> EtherAVB
>
> On Fri, Aug 10, 2018 at 01:47:12PM +0000, Biju Das wrote:
> > Hello Simon,
> >
> > Thanks for the feedback.
> >
> > > Subject: Re: [PATCH v3 5/5] ARM: dts: iwg23s-sbc: Add pinctl support
> > > for EtherAVB
> > >
> > > On Tue, Aug 07, 2018 at 08:57:06AM +0100, Biju Das wrote:
> > > > Adding pinctrl support for EtherAVB interface.
> > > >
> > > > Signed-off-by: Biju Das <biju....@bp.renesas.com>
> > > > Reviewed-by: Fabrizio Castro <fabrizio.cas...@bp.renesas.com>
> > > > Reviewed-by: Geert Uytterhoeven <geert+rene...@glider.be>
> > > > ---
> > > > V1-->V2
> > > >      * No change
> > > > V2-->V3
> > > >      * No change
> > > >      Depend onthe below patch
> > > >        https://patchwork.kernel.org/patch/10546801/
> > >
> > > What is the effect of applying this patch without that dependency
> present?
> >
> > No build dependency. Only driver probe will fail during run time.
> > [    0.838091] ravb: probe of e6800000.ethernet failed with error -22
>
> Thanks, but does the driver probe fail without this patch?

Drive probe works ok without  this patch.

> If so I'd rather hold off on this until v4.21.



Renesas Electronics Europe Ltd, Dukes Meadow, Millboard Road, Bourne End, 
Buckinghamshire, SL8 5FH, UK. Registered in England & Wales under Registered 
No. 04586709.

Reply via email to