Re: [PATCH 2/2] arm64: dts: renesas: r8a77995: draak: enable EthernetAVB

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Wed, Sep 13, 2017 at 11:58:47PM +0000, Yoshihiro Shimoda wrote:
> Hi Geert-san,
> 
> > From: Geert Uytterhoeven
> > Sent: Wednesday, September 13, 2017 11:55 PM
> > 
> > Hi Shimoda-san,
> > 
> > On Wed, Sep 13, 2017 at 2:18 PM, Yoshihiro Shimoda
> > <yoshihiro.shimoda.uh@xxxxxxxxxxx> wrote:
> > > This patch enables EthernetAVB for R-Car D3 draak board.
> > >
> > > Signed-off-by: Yoshihiro Shimoda <yoshihiro.shimoda.uh@xxxxxxxxxxx>
> > 
> > Thanks for your patch!
> > 
> > > --- a/arch/arm64/boot/dts/renesas/r8a77995-draak.dts
> > > +++ b/arch/arm64/boot/dts/renesas/r8a77995-draak.dts
> > 
> > > @@ -37,6 +39,14 @@
> > >  };
> > >
> > >  &pfc {
> > > +       avb0_pins: avb {
> > > +               mux {
> > > +                       groups = "avb0_link", "avb0_phy_int", "avb0_mdc",
> > > +                                "avb0_mii";
> > > +                       function = "avb0";
> > 
> > This part depends on (an updated version of) "[PATCH 5/8] pinctrl: sh-pfc:
> > r8a77995: Add EthernetAVB pins, groups and functions", so it may change?
> 
> Oops, I fotgot to submit v2 PFC patch yesterday...
> Yes, it is changed like other gen3 SoCs. I submitted it now. 
> 
> > And we will probably have to add driver-strengths later, to avoid a
> > dependency on bootloader setup?
> 
> Yes, I will do so, if needed.
> 
> > Apart from that:
> > Reviewed-by: Geert Uytterhoeven <geert+renesas@xxxxxxxxx>
> 
> Thank you for your review!

Thanks. I have marked this as "Deferred" pending acceptance of the PFC
patch. Please repost or ping me, f.e. via a reply-to-all to this email,
once that dependency is resolved.



[Index of Archives]     [Linux Samsung SOC]     [Linux Wireless]     [Linux Kernel]     [ATH6KL]     [Linux Bluetooth]     [Linux Netdev]     [Kernel Newbies]     [IDE]     [Security]     [Git]     [Netfilter]     [Bugtraq]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Linux ATA RAID]     [Samba]     [Device Mapper]

  Powered by Linux