* [PATCH] dt-bindings: watchdog: renesas,wdt: Document RZ/G3L support
@ 2026-02-06 11:22 Biju
2026-02-23 17:12 ` Rob Herring (Arm)
` (2 more replies)
0 siblings, 3 replies; 5+ messages in thread
From: Biju @ 2026-02-06 11:22 UTC (permalink / raw)
To: Wim Van Sebroeck, Guenter Roeck, Rob Herring,
Krzysztof Kozlowski, Conor Dooley, Geert Uytterhoeven,
Magnus Damm
Cc: Biju Das, linux-watchdog, devicetree, linux-kernel,
linux-renesas-soc, Prabhakar Mahadev Lad, Biju Das
From: Biju Das <biju.das.jz@bp.renesas.com>
Document the support for the watchdog IP available on RZ/G3L SoC. The
watchdog IP available on RZ/G3L SoC is identical to the one found on
RZ/G2L SoC.
Signed-off-by: Biju Das <biju.das.jz@bp.renesas.com>
---
.../devicetree/bindings/watchdog/renesas,rzg2l-wdt.yaml | 1 +
1 file changed, 1 insertion(+)
diff --git a/Documentation/devicetree/bindings/watchdog/renesas,rzg2l-wdt.yaml b/Documentation/devicetree/bindings/watchdog/renesas,rzg2l-wdt.yaml
index a4d06c9c8b86..40b4fb26d9c5 100644
--- a/Documentation/devicetree/bindings/watchdog/renesas,rzg2l-wdt.yaml
+++ b/Documentation/devicetree/bindings/watchdog/renesas,rzg2l-wdt.yaml
@@ -18,6 +18,7 @@ properties:
- renesas,r9a07g044-wdt # RZ/G2{L,LC}
- renesas,r9a07g054-wdt # RZ/V2L
- renesas,r9a08g045-wdt # RZ/G3S
+ - renesas,r9a08g046-wdt # RZ/G3L
- const: renesas,rzg2l-wdt
- items:
--
2.43.0
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH] dt-bindings: watchdog: renesas,wdt: Document RZ/G3L support
2026-02-06 11:22 [PATCH] dt-bindings: watchdog: renesas,wdt: Document RZ/G3L support Biju
@ 2026-02-23 17:12 ` Rob Herring (Arm)
2026-05-05 6:04 ` Biju Das
2026-05-05 7:31 ` Geert Uytterhoeven
2 siblings, 0 replies; 5+ messages in thread
From: Rob Herring (Arm) @ 2026-02-23 17:12 UTC (permalink / raw)
To: Biju
Cc: Biju Das, Geert Uytterhoeven, devicetree, Krzysztof Kozlowski,
linux-renesas-soc, linux-kernel, Prabhakar Mahadev Lad,
Guenter Roeck, Magnus Damm, Conor Dooley, Wim Van Sebroeck,
linux-watchdog
On Fri, 06 Feb 2026 11:22:25 +0000, Biju wrote:
> From: Biju Das <biju.das.jz@bp.renesas.com>
>
> Document the support for the watchdog IP available on RZ/G3L SoC. The
> watchdog IP available on RZ/G3L SoC is identical to the one found on
> RZ/G2L SoC.
>
> Signed-off-by: Biju Das <biju.das.jz@bp.renesas.com>
> ---
> .../devicetree/bindings/watchdog/renesas,rzg2l-wdt.yaml | 1 +
> 1 file changed, 1 insertion(+)
>
Acked-by: Rob Herring (Arm) <robh@kernel.org>
^ permalink raw reply [flat|nested] 5+ messages in thread
* RE: [PATCH] dt-bindings: watchdog: renesas,wdt: Document RZ/G3L support
2026-02-06 11:22 [PATCH] dt-bindings: watchdog: renesas,wdt: Document RZ/G3L support Biju
2026-02-23 17:12 ` Rob Herring (Arm)
@ 2026-05-05 6:04 ` Biju Das
2026-05-05 7:31 ` Geert Uytterhoeven
2 siblings, 0 replies; 5+ messages in thread
From: Biju Das @ 2026-05-05 6:04 UTC (permalink / raw)
To: Wim Van Sebroeck, Guenter Roeck, Geert Uytterhoeven
Cc: linux-watchdog, devicetree, linux-kernel, linux-renesas-soc,
Prabhakar Mahadev Lad, biju.das.au, Rob Herring,
Krzysztof Kozlowski, Conor Dooley, magnus.damm, biju.das.au
Hi Wim/Guenter/Geert,
> -----Original Message-----
> From: Biju <biju.das.au@gmail.com>
> Sent: 06 February 2026 11:22
> Subject: [PATCH] dt-bindings: watchdog: renesas,wdt: Document RZ/G3L support
>
> From: Biju Das <biju.das.jz@bp.renesas.com>
>
> Document the support for the watchdog IP available on RZ/G3L SoC. The watchdog IP available on RZ/G3L
> SoC is identical to the one found on RZ/G2L SoC.
>
> Signed-off-by: Biju Das <biju.das.jz@bp.renesas.com>
> ---
> .../devicetree/bindings/watchdog/renesas,rzg2l-wdt.yaml | 1 +
> 1 file changed, 1 insertion(+)
>
> diff --git a/Documentation/devicetree/bindings/watchdog/renesas,rzg2l-wdt.yaml
> b/Documentation/devicetree/bindings/watchdog/renesas,rzg2l-wdt.yaml
> index a4d06c9c8b86..40b4fb26d9c5 100644
> --- a/Documentation/devicetree/bindings/watchdog/renesas,rzg2l-wdt.yaml
> +++ b/Documentation/devicetree/bindings/watchdog/renesas,rzg2l-wdt.yaml
> @@ -18,6 +18,7 @@ properties:
> - renesas,r9a07g044-wdt # RZ/G2{L,LC}
> - renesas,r9a07g054-wdt # RZ/V2L
> - renesas,r9a08g045-wdt # RZ/G3S
> + - renesas,r9a08g046-wdt # RZ/G3L
> - const: renesas,rzg2l-wdt
>
> - items:
> --
> 2.43.0
Gentle ping.
Cheers,
Biju
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH] dt-bindings: watchdog: renesas,wdt: Document RZ/G3L support
2026-02-06 11:22 [PATCH] dt-bindings: watchdog: renesas,wdt: Document RZ/G3L support Biju
2026-02-23 17:12 ` Rob Herring (Arm)
2026-05-05 6:04 ` Biju Das
@ 2026-05-05 7:31 ` Geert Uytterhoeven
2026-05-05 8:06 ` Biju Das
2 siblings, 1 reply; 5+ messages in thread
From: Geert Uytterhoeven @ 2026-05-05 7:31 UTC (permalink / raw)
To: Biju
Cc: Wim Van Sebroeck, Guenter Roeck, Rob Herring,
Krzysztof Kozlowski, Conor Dooley, Geert Uytterhoeven,
Magnus Damm, Biju Das, linux-watchdog, devicetree, linux-kernel,
linux-renesas-soc, Prabhakar Mahadev Lad
Hi Biju,
On Fri, 6 Feb 2026 at 12:22, Biju <biju.das.au@gmail.com> wrote:
> From: Biju Das <biju.das.jz@bp.renesas.com>
>
> Document the support for the watchdog IP available on RZ/G3L SoC. The
> watchdog IP available on RZ/G3L SoC is identical to the one found on
> RZ/G2L SoC.
>
> Signed-off-by: Biju Das <biju.das.jz@bp.renesas.com>
Thanks for your patch!
> --- a/Documentation/devicetree/bindings/watchdog/renesas,rzg2l-wdt.yaml
> +++ b/Documentation/devicetree/bindings/watchdog/renesas,rzg2l-wdt.yaml
> @@ -18,6 +18,7 @@ properties:
> - renesas,r9a07g044-wdt # RZ/G2{L,LC}
> - renesas,r9a07g054-wdt # RZ/V2L
> - renesas,r9a08g045-wdt # RZ/G3S
> + - renesas,r9a08g046-wdt # RZ/G3L
> - const: renesas,rzg2l-wdt
>
> - items:
However, there seems to be a small difference in the formulas for the
timeout register value on the various SoCs:
RZ/G2L, RZ/G2UL, and RZ/V2L:
WDTTIME setting value = \frac{WDT cycle}{(WDTn_CLK (n = 0,1,2)
cycle × 1024 × 1024) − 1}
RZ/G3S and RZ/G3E:
WDTTIME setting value = \frac{WDT cycle}{(WDTn_CLK (n = 0,1,2)
cycle × 1024 × 1024} − 1
I.e. on the former, the "- 1" offset is inside the denominator, while
on the former, it is outside the fraction. I assume this is just a typo
in the documentation, and both formulas are supposed to be identical, so
Reviewed-by: Geert Uytterhoeven <geert+renesas@glider.be>
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] 5+ messages in thread
* RE: [PATCH] dt-bindings: watchdog: renesas,wdt: Document RZ/G3L support
2026-05-05 7:31 ` Geert Uytterhoeven
@ 2026-05-05 8:06 ` Biju Das
0 siblings, 0 replies; 5+ messages in thread
From: Biju Das @ 2026-05-05 8:06 UTC (permalink / raw)
To: geert, biju.das.au
Cc: Wim Van Sebroeck, Guenter Roeck, Rob Herring,
Krzysztof Kozlowski, Conor Dooley, Geert Uytterhoeven,
magnus.damm, linux-watchdog, devicetree, linux-kernel,
linux-renesas-soc, Prabhakar Mahadev Lad
Hi Geert,
Thanks for the feedback.
> -----Original Message-----
> From: Geert Uytterhoeven <geert@linux-m68k.org>
> Sent: 05 May 2026 08:32
> Subject: Re: [PATCH] dt-bindings: watchdog: renesas,wdt: Document RZ/G3L support
>
> Hi Biju,
>
> On Fri, 6 Feb 2026 at 12:22, Biju <biju.das.au@gmail.com> wrote:
> > From: Biju Das <biju.das.jz@bp.renesas.com>
> >
> > Document the support for the watchdog IP available on RZ/G3L SoC. The
> > watchdog IP available on RZ/G3L SoC is identical to the one found on
> > RZ/G2L SoC.
> >
> > Signed-off-by: Biju Das <biju.das.jz@bp.renesas.com>
>
> Thanks for your patch!
>
> > ---
> > a/Documentation/devicetree/bindings/watchdog/renesas,rzg2l-wdt.yaml
> > +++ b/Documentation/devicetree/bindings/watchdog/renesas,rzg2l-wdt.yam
> > +++ l
> > @@ -18,6 +18,7 @@ properties:
> > - renesas,r9a07g044-wdt # RZ/G2{L,LC}
> > - renesas,r9a07g054-wdt # RZ/V2L
> > - renesas,r9a08g045-wdt # RZ/G3S
> > + - renesas,r9a08g046-wdt # RZ/G3L
> > - const: renesas,rzg2l-wdt
> >
> > - items:
>
> However, there seems to be a small difference in the formulas for the timeout register value on the
> various SoCs:
>
> RZ/G2L, RZ/G2UL, and RZ/V2L:
>
> WDTTIME setting value = \frac{WDT cycle}{(WDTn_CLK (n = 0,1,2) cycle × 1024 × 1024) − 1}
>
> RZ/G3S and RZ/G3E:
>
> WDTTIME setting value = \frac{WDT cycle}{(WDTn_CLK (n = 0,1,2) cycle × 1024 × 1024} − 1
>
> I.e. on the former, the "- 1" offset is inside the denominator, while on the former, it is outside the
> fraction. I assume this is just a typo in the documentation, and both formulas are supposed to be
> identical, so
I agree it is a typo on RZ/G2L, RZ/G2UL, and RZ/V2L hardware manual, I will report this issue to
Documentation team.
as
Watchdog timer cycle = WDTn_CLK (n = 0, 1, 2) cycle × 1024 × 1024 × (WDTTIME setting value + 1)
To get min/max value 43.69/178956.97 msec with 24 MHz mentioned on all DoCs,
WDT min Cycle = 1024 * 1024 * (0 + 1) / (24 * 10^6) = 0.043690 = 43.69 msec
WDT max cycle = 1024 * 1024 * (0xfff + 1) / (24 * 10^6) = 178956.97 msec
Cheers,
Biju
> Reviewed-by: Geert Uytterhoeven <geert+renesas@glider.be>
>
> 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] 5+ messages in thread
end of thread, other threads:[~2026-05-05 8:06 UTC | newest]
Thread overview: 5+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2026-02-06 11:22 [PATCH] dt-bindings: watchdog: renesas,wdt: Document RZ/G3L support Biju
2026-02-23 17:12 ` Rob Herring (Arm)
2026-05-05 6:04 ` Biju Das
2026-05-05 7:31 ` Geert Uytterhoeven
2026-05-05 8:06 ` Biju Das
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox
all inboxes | Powered by JetHome®