[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID:
<TYCPR01MB12093454F70F32ACC87A7A4D3C29EA@TYCPR01MB12093.jpnprd01.prod.outlook.com>
Date: Thu, 29 Jan 2026 17:41:24 +0000
From: Fabrizio Castro <fabrizio.castro.jz@...esas.com>
To: Conor Dooley <conor@...nel.org>
CC: Rob Herring <robh@...nel.org>, Guenter Roeck <linux@...ck-us.net>, Michael
Turquette <mturquette@...libre.com>, Stephen Boyd <sboyd@...nel.org>, Wim Van
Sebroeck <wim@...ux-watchdog.org>, Krzysztof Kozlowski <krzk+dt@...nel.org>,
Geert Uytterhoeven <geert+renesas@...der.be>, Conor Dooley
<conor+dt@...nel.org>, magnus.damm <magnus.damm@...il.com>, Prabhakar Mahadev
Lad <prabhakar.mahadev-lad.rj@...renesas.com>,
"linux-watchdog@...r.kernel.org" <linux-watchdog@...r.kernel.org>,
"linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>,
"linux-renesas-soc@...r.kernel.org" <linux-renesas-soc@...r.kernel.org>,
"devicetree@...r.kernel.org" <devicetree@...r.kernel.org>, wsa+renesas
<wsa+renesas@...g-engineering.com>, "linux-clk@...r.kernel.org"
<linux-clk@...r.kernel.org>, Biju Das <biju.das.jz@...renesas.com>
Subject: RE: [PATCH mainline-linux 1/3] dt-bindings: watchdog:
renesas,r9a09g057-wdt: Rework example
Hi Conor,
Thanks for your feedback.
> From: Conor Dooley <conor@...nel.org>
> Sent: 29 January 2026 17:39
> To: Fabrizio Castro <fabrizio.castro.jz@...esas.com>
> Cc: Rob Herring <robh@...nel.org>; Guenter Roeck <linux@...ck-us.net>; Michael Turquette
> <mturquette@...libre.com>; Stephen Boyd <sboyd@...nel.org>; Wim Van Sebroeck <wim@...ux-watchdog.org>;
> Krzysztof Kozlowski <krzk+dt@...nel.org>; Geert Uytterhoeven <geert+renesas@...der.be>; Conor Dooley
> <conor+dt@...nel.org>; magnus.damm <magnus.damm@...il.com>; Prabhakar Mahadev Lad <prabhakar.mahadev-
> lad.rj@...renesas.com>; linux-watchdog@...r.kernel.org; linux-kernel@...r.kernel.org; linux-renesas-
> soc@...r.kernel.org; devicetree@...r.kernel.org; wsa+renesas <wsa+renesas@...g-engineering.com>; linux-
> clk@...r.kernel.org; Biju Das <biju.das.jz@...renesas.com>
> Subject: Re: [PATCH mainline-linux 1/3] dt-bindings: watchdog: renesas,r9a09g057-wdt: Rework example
>
> On Thu, Jan 29, 2026 at 02:07:29PM +0000, Fabrizio Castro wrote:
> > When the bindings for the Renesas RZ/V2H(P) SoC were factored
> > out IP WDT0 was selected for the example, however the HW user
> > manual states that only IP WDT1 can be used by Linux.
> >
> > This commit is part of a series that removes WDT{0,2,3} support
> > from the kernel, therefore the example from the bindings has
> > lost its meaning.
> >
> > Update the example accordingly.
> >
> > Fixes: fcba2855251f ("dt-bindings: watchdog: factor out RZ/V2H(P) watchdog")
>
> Given this is an example, I don't think a fixes tag is appropriate here.
I'll take this out in v2.
Cheers,
Fab
>
> > Signed-off-by: Fabrizio Castro <fabrizio.castro.jz@...esas.com>
> > ---
> > .../bindings/watchdog/renesas,r9a09g057-wdt.yaml | 8 ++++----
> > 1 file changed, 4 insertions(+), 4 deletions(-)
> >
> > diff --git a/Documentation/devicetree/bindings/watchdog/renesas,r9a09g057-wdt.yaml
> b/Documentation/devicetree/bindings/watchdog/renesas,r9a09g057-wdt.yaml
> > index 099200c4f136..975c5aa4d747 100644
> > --- a/Documentation/devicetree/bindings/watchdog/renesas,r9a09g057-wdt.yaml
> > +++ b/Documentation/devicetree/bindings/watchdog/renesas,r9a09g057-wdt.yaml
> > @@ -89,11 +89,11 @@ examples:
> > - |
> > #include <dt-bindings/clock/renesas,r9a09g057-cpg.h>
> >
> > - watchdog@...00400 {
> > + watchdog@...00000 {
> > compatible = "renesas,r9a09g057-wdt";
> > - reg = <0x11c00400 0x400>;
> > - clocks = <&cpg CPG_MOD 0x4b>, <&cpg CPG_MOD 0x4c>;
> > + reg = <0x14400000 0x400>;
> > + clocks = <&cpg CPG_MOD 0x4d>, <&cpg CPG_MOD 0x4e>;
> > clock-names = "pclk", "oscclk";
> > - resets = <&cpg 0x75>;
> > + resets = <&cpg 0x76>;
> > power-domains = <&cpg>;
> > };
> > --
> > 2.34.1
> >
Powered by blists - more mailing lists