[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-ID: <CA+V-a8s_3hv9z0HFKiUQ76d7FVJ+-vrLahhFVEafZbS3oO8iJw@mail.gmail.com>
Date: Tue, 29 Jul 2025 18:10:52 +0100
From: "Lad, Prabhakar" <prabhakar.csengg@...il.com>
To: Wolfram Sang <wsa+renesas@...g-engineering.com>
Cc: Wim Van Sebroeck <wim@...ux-watchdog.org>, Guenter Roeck <linux@...ck-us.net>,
Rob Herring <robh@...nel.org>, Krzysztof Kozlowski <krzk+dt@...nel.org>, Conor Dooley <conor+dt@...nel.org>,
Philipp Zabel <p.zabel@...gutronix.de>, Geert Uytterhoeven <geert+renesas@...der.be>,
Magnus Damm <magnus.damm@...il.com>, linux-watchdog@...r.kernel.org,
devicetree@...r.kernel.org, linux-kernel@...r.kernel.org,
linux-renesas-soc@...r.kernel.org, Biju Das <biju.das.jz@...renesas.com>,
Fabrizio Castro <fabrizio.castro.jz@...esas.com>,
Lad Prabhakar <prabhakar.mahadev-lad.rj@...renesas.com>
Subject: Re: [PATCH v2 0/9] Add watchdog driver support for RZ/T2H and RZ/N2H SoCs
Hi Wolfram,
On Tue, Jul 29, 2025 at 6:08 PM Wolfram Sang
<wsa+renesas@...g-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
Powered by blists - more mailing lists