From: Lad Prabhakar <prabhakar.mahadev-lad.rj@bp.renesas.com>

Hi All,

This patch series adds watchdog driver support for the Renesas RZ/T2H
(R9A09G077) and RZ/N2H (R9A09G087) SoCs. The necessary device tree
bindings and driver modifications are included.

v1->v2:
- Dropped items from clock-names and instead added maxItems: 1.
- Added reviewed-by from Rob.

Cheers,
Prabhakar

Lad Prabhakar (9):
  dt-bindings: watchdog: renesas,wdt: Add support for RZ/T2H and RZ/N2H
  watchdog: rzv2h_wdt: Obtain clock-divider ranges from OF match data
  watchdog: rzv2h_wdt: Obtain CKS divider via OF data
  watchdog: rzv2h_wdt: Make "oscclk" an optional clock
  watchdog: rzv2h_wdt: Add support for configurable count clock source
  watchdog: rzv2h_wdt: Make reset controller optional
  watchdog: rzv2h: Set min_timeout based on max_hw_heartbeat_ms
  watchdog: rzv2h: Add support for RZ/T2H
  watchdog: rzv2h_wdt: Improve error strings and add newlines

 .../bindings/watchdog/renesas,wdt.yaml        |  36 ++++-
 drivers/watchdog/rzv2h_wdt.c                  | 137 ++++++++++++++++--
 2 files changed, 154 insertions(+), 19 deletions(-)

-- 
2.50.1

Hi,

>   dt-bindings: watchdog: renesas,wdt: Add support for RZ/T2H and RZ/N2H
>   watchdog: rzv2h_wdt: Obtain clock-divider ranges from OF match data
>   watchdog: rzv2h_wdt: Obtain CKS divider via OF data
>   watchdog: rzv2h_wdt: Make "oscclk" an optional clock
>   watchdog: rzv2h_wdt: Add support for configurable count clock source
>   watchdog: rzv2h_wdt: Make reset controller optional
>   watchdog: rzv2h: Set min_timeout based on max_hw_heartbeat_ms
>   watchdog: rzv2h: Add support for RZ/T2H
>   watchdog: rzv2h_wdt: Improve error strings and add newlines

Minor nit, but still: inconsistent prefix

I'd suggest to use "rzv2h" instead of "rzv2h_wdt" but it is ultimately
the WDT maintainers call...

Hi Wolfram,

On Tue, Jul 29, 2025 at 6:08 PM Wolfram Sang
<wsa+renesas@sang-engineering.com> wrote:
>
> Hi,
>
> >   dt-bindings: watchdog: renesas,wdt: Add support for RZ/T2H and RZ/N2H
> >   watchdog: rzv2h_wdt: Obtain clock-divider ranges from OF match data
> >   watchdog: rzv2h_wdt: Obtain CKS divider via OF data
> >   watchdog: rzv2h_wdt: Make "oscclk" an optional clock
> >   watchdog: rzv2h_wdt: Add support for configurable count clock source
> >   watchdog: rzv2h_wdt: Make reset controller optional
> >   watchdog: rzv2h: Set min_timeout based on max_hw_heartbeat_ms
> >   watchdog: rzv2h: Add support for RZ/T2H
> >   watchdog: rzv2h_wdt: Improve error strings and add newlines
>
> Minor nit, but still: inconsistent prefix
>
> I'd suggest to use "rzv2h" instead of "rzv2h_wdt" but it is ultimately
> the WDT maintainers call...
>
I agree, I will switch to using "rzv2h".

Cheers,
Prabhakar