On Tue, Aug 07, 2018 at 10:16:16AM +0200, Geert Uytterhoeven wrote:
> Hi Marek,
> 
> On Sat, Aug 4, 2018 at 6:38 PM Marek Vasut <marek.va...@gmail.com> wrote:
> > Add DA9063 PMIC node to the I2C bus.
> >
> > Signed-off-by: Marek Vasut <marek.vasut+rene...@gmail.com>
> > Cc: Geert Uytterhoeven <geert+rene...@glider.be>
> > Cc: Kuninori Morimoto <kuninori.morimoto...@renesas.com>
> > Cc: Simon Horman <horms+rene...@verge.net.au>
> > Cc: Wolfram Sang <wsa+rene...@sang-engineering.com>
> > Cc: linux-renesas-soc@vger.kernel.org
> > ---
> > V2: - Replace shmobile with dts since it's a DT patch in subject
> >     - Connect the 9063_IRQ# line to GP3_31
> >     - Since the DA9063 is connected to both i2c1 and i2c7, connect
> >       it to i2c7, which is the dedicated dvfs i2c.
> 
> Thanks for the update!
> 
> > --- a/arch/arm/boot/dts/r8a7794-silk.dts
> > +++ b/arch/arm/boot/dts/r8a7794-silk.dts
> > @@ -405,6 +405,23 @@
> >         clock-frequency = <400000>;
> >  };
> >
> > +&i2c7 {
> > +       status = "okay";
> > +       clock-frequency = <100000>;
> > +
> > +       pmic@58 {
> > +               compatible = "dlg,da9063";
> > +               reg = <0x58>;
> > +               interrupt-parent = <&gpio3>;
> > +               interrupts = <31 IRQ_TYPE_LEVEL_LOW>;
> > +               interrupt-controller;
> > +
> > +               wdt {
> > +                       compatible = "dlg,da9063-watchdog";
> > +               };
> 
> Given Silk has the full da9063 (unlike the "L" version on Porter), shouldn't
> you add an rtc subnode?
> 
> Oh, Silk also has the onkey pin wired, so perhaps you want to add an onkey
> subnode, too? Does that feature work?
> 
> BTW, Stout also has the onkey wired, but lacks the onkey subnode.
> 
> > +       };
> > +};
> > +
> 
> Nevertheless:
> Reviewed-by: Geert Uytterhoeven <geert+rene...@glider.be>

Thanks, I have applied this patch for v4.20.

Marek,

please consider some follow-up patches to address the issues
raised by Geert.

Reply via email to