lists.openwall.net   lists  /  announce  owl-users  owl-dev  john-users  john-dev  passwdqc-users  yescrypt  popa3d-users  /  oss-security  kernel-hardening  musl  sabotage  tlsify  passwords  /  crypt-dev  xvendor  /  Bugtraq  Full-Disclosure  linux-kernel  linux-netdev  linux-ext4  linux-hardening  linux-cve-announce  PHC 
Open Source and information security mailing list archives
 
Hash Suite: Windows password security audit tool. GUI, reports in PDF.
[<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

Powered by Openwall GNU/*/Linux Powered by OpenVZ