linux-renesas-soc.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* [PATCH] ARM: dts: r9a06g032-rzn1d400-db: enable rtc0
@ 2022-06-08  9:08 Clément Léger
  2022-06-09  9:36 ` Geert Uytterhoeven
  0 siblings, 1 reply; 3+ messages in thread
From: Clément Léger @ 2022-06-08  9:08 UTC (permalink / raw)
  To: Geert Uytterhoeven, Magnus Damm, Rob Herring, Krzysztof Kozlowski
  Cc: Clément Léger, linux-renesas-soc, devicetree,
	linux-kernel, Miquèl Raynal

The RZ/N1D-DB board does have a battery to power the RTC. Enable the
RTC device on this board.

Signed-off-by: Clément Léger <clement.leger@bootlin.com>
---
 arch/arm/boot/dts/r9a06g032-rzn1d400-db.dts | 4 ++++
 1 file changed, 4 insertions(+)

diff --git a/arch/arm/boot/dts/r9a06g032-rzn1d400-db.dts b/arch/arm/boot/dts/r9a06g032-rzn1d400-db.dts
index 3f8f3ce87e12..c736d398325a 100644
--- a/arch/arm/boot/dts/r9a06g032-rzn1d400-db.dts
+++ b/arch/arm/boot/dts/r9a06g032-rzn1d400-db.dts
@@ -31,3 +31,7 @@ &wdt0 {
 	timeout-sec = <60>;
 	status = "okay";
 };
+
+&rtc0 {
+	status = "okay";
+};
-- 
2.36.1


^ permalink raw reply related	[flat|nested] 3+ messages in thread

* Re: [PATCH] ARM: dts: r9a06g032-rzn1d400-db: enable rtc0
  2022-06-08  9:08 [PATCH] ARM: dts: r9a06g032-rzn1d400-db: enable rtc0 Clément Léger
@ 2022-06-09  9:36 ` Geert Uytterhoeven
  2022-06-09  9:38   ` Clément Léger
  0 siblings, 1 reply; 3+ messages in thread
From: Geert Uytterhoeven @ 2022-06-09  9:36 UTC (permalink / raw)
  To: Clément Léger
  Cc: Geert Uytterhoeven, Magnus Damm, Rob Herring,
	Krzysztof Kozlowski, Linux-Renesas,
	open list:OPEN FIRMWARE AND FLATTENED DEVICE TREE BINDINGS,
	Linux Kernel Mailing List, Miquèl Raynal

Hi Clément,

On Wed, Jun 8, 2022 at 11:10 AM Clément Léger <clement.leger@bootlin.com> wrote:
> The RZ/N1D-DB board does have a battery to power the RTC. Enable the
> RTC device on this board.
>
> Signed-off-by: Clément Léger <clement.leger@bootlin.com>

Thanks for your patch!

> --- a/arch/arm/boot/dts/r9a06g032-rzn1d400-db.dts
> +++ b/arch/arm/boot/dts/r9a06g032-rzn1d400-db.dts
> @@ -31,3 +31,7 @@ &wdt0 {
>         timeout-sec = <60>;
>         status = "okay";
>  };
> +
> +&rtc0 {
> +       status = "okay";
> +};

Reviewed-by: Geert Uytterhoeven <geert+renesas@glider.be>
i.e. will queue in renesas-devel for v5.20, with rtc0 moved up to
preserve sort order (no need to resend).

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

^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [PATCH] ARM: dts: r9a06g032-rzn1d400-db: enable rtc0
  2022-06-09  9:36 ` Geert Uytterhoeven
@ 2022-06-09  9:38   ` Clément Léger
  0 siblings, 0 replies; 3+ messages in thread
From: Clément Léger @ 2022-06-09  9:38 UTC (permalink / raw)
  To: Geert Uytterhoeven
  Cc: Geert Uytterhoeven, Magnus Damm, Rob Herring,
	Krzysztof Kozlowski, Linux-Renesas,
	open list:OPEN FIRMWARE AND FLATTENED DEVICE TREE BINDINGS,
	Linux Kernel Mailing List, Miquèl Raynal

Le Thu, 9 Jun 2022 11:36:13 +0200,
Geert Uytterhoeven <geert@linux-m68k.org> a écrit :

> Hi Clément,
> 
> On Wed, Jun 8, 2022 at 11:10 AM Clément Léger <clement.leger@bootlin.com> wrote:
> > The RZ/N1D-DB board does have a battery to power the RTC. Enable the
> > RTC device on this board.
> >
> > Signed-off-by: Clément Léger <clement.leger@bootlin.com>  
> 
> Thanks for your patch!
> 
> > --- a/arch/arm/boot/dts/r9a06g032-rzn1d400-db.dts
> > +++ b/arch/arm/boot/dts/r9a06g032-rzn1d400-db.dts
> > @@ -31,3 +31,7 @@ &wdt0 {
> >         timeout-sec = <60>;
> >         status = "okay";
> >  };
> > +
> > +&rtc0 {
> > +       status = "okay";
> > +};  
> 
> Reviewed-by: Geert Uytterhoeven <geert+renesas@glider.be>
> i.e. will queue in renesas-devel for v5.20, with rtc0 moved up to
> preserve sort order (no need to resend).

Acked, I checked the order but the wdt0 has already been inserted after
the uart0 node so I assumed it was unsorted.

Thanks ! 

> 
> 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



-- 
Clément Léger,
Embedded Linux and Kernel engineer at Bootlin
https://bootlin.com

^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2022-06-09  9:39 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2022-06-08  9:08 [PATCH] ARM: dts: r9a06g032-rzn1d400-db: enable rtc0 Clément Léger
2022-06-09  9:36 ` Geert Uytterhoeven
2022-06-09  9:38   ` Clément Léger

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).