Re: [PATCH 1/2] ARM: dts: iwg20d-q7: Add RTC support

2017-08-15 Thread Simon Horman 
On Mon, Aug 14, 2017 at 02:26:08PM +, Biju Das wrote:
> 
> 
> > -Original Message-
> > From: geert.uytterhoe...@gmail.com [mailto:geert.uytterhoe...@gmail.com]
> > On Behalf Of Geert Uytterhoeven
> > Sent: 14 August 2017 15:12
> > To: Biju Das <biju@bp.renesas.com>
> > Cc: Rob Herring <robh...@kernel.org>; Mark Rutland
> > <mark.rutl...@arm.com>; Simon Horman <ho...@verge.net.au>; Magnus
> > Damm <magnus.d...@gmail.com>; Russell King <li...@armlinux.org.uk>;
> > Chris Paterson <chris.paters...@renesas.com>; devicet...@vger.kernel.org;
> > Linux-Renesas <linux-renesas-soc@vger.kernel.org>; linux-arm-
> > ker...@lists.infradead.org
> > Subject: Re: [PATCH 1/2] ARM: dts: iwg20d-q7: Add RTC support
> >
> > On Tue, Aug 8, 2017 at 3:04 PM, Biju Das <biju@bp.renesas.com> wrote:
> > > Define the iWave RainboW-G20D-Qseven board dependent part of the RTC
> > > device node.
> > >
> > > Signed-off-by: Biju Das <biju@bp.renesas.com>
> >
> > > @@ -54,3 +59,16 @@
> > > micrel,led-mode = <1>;
> > > };
> > >  };
> > > +
> > > + {
> > > +   pinctrl-0 = <_pins>;
> > > +   pinctrl-names = "default";
> > > +
> > > +   status = "okay";
> > > +   clock-frequency = <40>;
> > > +
> > > +   rtc@68 {
> > > +   compatible = "bq32000";
> >
> > "ti,bq32000"
> 
> Thanks. I will change this.

Thanks, I have dropped this patch for now.
Please submit a v2.

RE: [PATCH 1/2] ARM: dts: iwg20d-q7: Add RTC support

2017-08-14 Thread Biju Das 


> -Original Message-
> From: geert.uytterhoe...@gmail.com [mailto:geert.uytterhoe...@gmail.com]
> On Behalf Of Geert Uytterhoeven
> Sent: 14 August 2017 15:12
> To: Biju Das <biju@bp.renesas.com>
> Cc: Rob Herring <robh...@kernel.org>; Mark Rutland
> <mark.rutl...@arm.com>; Simon Horman <ho...@verge.net.au>; Magnus
> Damm <magnus.d...@gmail.com>; Russell King <li...@armlinux.org.uk>;
> Chris Paterson <chris.paters...@renesas.com>; devicet...@vger.kernel.org;
> Linux-Renesas <linux-renesas-soc@vger.kernel.org>; linux-arm-
> ker...@lists.infradead.org
> Subject: Re: [PATCH 1/2] ARM: dts: iwg20d-q7: Add RTC support
>
> On Tue, Aug 8, 2017 at 3:04 PM, Biju Das <biju@bp.renesas.com> wrote:
> > Define the iWave RainboW-G20D-Qseven board dependent part of the RTC
> > device node.
> >
> > Signed-off-by: Biju Das <biju@bp.renesas.com>
>
> > @@ -54,3 +59,16 @@
> > micrel,led-mode = <1>;
> > };
> >  };
> > +
> > + {
> > +   pinctrl-0 = <_pins>;
> > +   pinctrl-names = "default";
> > +
> > +   status = "okay";
> > +   clock-frequency = <40>;
> > +
> > +   rtc@68 {
> > +   compatible = "bq32000";
>
> "ti,bq32000"

Thanks. I will change this.

> > +   reg = <0x68>;
> > +   };
> > +};
>
> Gr{oetje,eeting}s,
>
> Geert
>
> --
> Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@linux-
> m68k.org
>
> In personal conversations with technical people, I call myself a hacker. But
> when I'm talking to journalists I just say "programmer" or something like 
> that.
> -- Linus Torvalds



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

Re: [PATCH 1/2] ARM: dts: iwg20d-q7: Add RTC support

2017-08-14 Thread Geert Uytterhoeven 
On Tue, Aug 8, 2017 at 3:04 PM, Biju Das  wrote:
> Define the iWave RainboW-G20D-Qseven board dependent part of the
> RTC device node.
>
> Signed-off-by: Biju Das 

> @@ -54,3 +59,16 @@
> micrel,led-mode = <1>;
> };
>  };
> +
> + {
> +   pinctrl-0 = <_pins>;
> +   pinctrl-names = "default";
> +
> +   status = "okay";
> +   clock-frequency = <40>;
> +
> +   rtc@68 {
> +   compatible = "bq32000";

"ti,bq32000"

> +   reg = <0x68>;
> +   };
> +};

Gr{oetje,eeting}s,

Geert

--
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- ge...@linux-m68k.org

In personal conversations with technical people, I call myself a hacker. But
when I'm talking to journalists I just say "programmer" or something like that.
-- Linus Torvalds

Re: [PATCH 1/2] ARM: dts: iwg20d-q7: Add RTC support

2017-08-13 Thread Simon Horman 
On Wed, Aug 09, 2017 at 11:02:30AM +0200, Simon Horman wrote:
> On Tue, Aug 08, 2017 at 02:04:24PM +0100, Biju Das wrote:
> > Define the iWave RainboW-G20D-Qseven board dependent part of the
> > RTC device node.
> > 
> > Signed-off-by: Biju Das 
> > ---
> >  arch/arm/boot/dts/r8a7743-iwg20d-q7.dts | 18 ++
> >  1 file changed, 18 insertions(+)
> 
> This has a compile-time dependency on
> "[PATCH  2/2] ARM: dts: r8a7743: Add I2C DT support"
> for the i2c2 node, right?

As I have now applied the above dependency I have
also applied this patch for v4.14.