[PATCH v4 0/4] watchdog: rzn1: clean up and drop irq requirement

Devicetree
 help / color / mirror / Atom feed

* [PATCH v4 0/4] watchdog: rzn1: clean up and drop irq requirement
@ 2026-05-07 10:24 Wolfram Sang
  2026-05-07 10:24 ` [PATCH v4 4/4] dt-bindings: watchdog: renesas,rzn1-wdt: interrupts are not required Wolfram Sang
  0 siblings, 1 reply; 6+ messages in thread
From: Wolfram Sang @ 2026-05-07 10:24 UTC (permalink / raw)
  To: linux-renesas-soc
  Cc: Herve Codina (Schneider Electric), Wolfram Sang, Conor Dooley,
	devicetree, Geert Uytterhoeven, Guenter Roeck,
	Krzysztof Kozlowski, linux-watchdog, Magnus Damm, Rob Herring,
	Wim Van Sebroeck

Changes since v3:
* rebased to v7.1-rc2
* added patches 3+4 to remove irq handling
* added tags for patches 1+2 (Thanks, Guenter!)

For easier dependency handling, I picked up Herve's v3 series and added
my patches on top. Herve's changes to the N1D syscon are now upstream,
so the watchdog can reset the system directly. Leaving only his cleanup
patches to be applied, already reviewed by Guenter. Then, my patches
remove the now unneeded irq handling and make irqs optional in DTs. All
tested on the Renesas RZ/N1D demo board.

Please apply.


Herve Codina (Schneider Electric) (2):
  watchdog: rzn1: Fix reverse xmas tree declaration
  watchdog: rzn1: Use dev_err_probe()

Wolfram Sang (2):
  watchdog: rzn1: remove now obsolete interrupt support
  dt-bindings: watchdog: renesas,rzn1-wdt: interrupts are not required

 .../bindings/watchdog/renesas,rzn1-wdt.yaml   |  1 -
 drivers/watchdog/rzn1_wdt.c                   | 35 +++----------------
 2 files changed, 5 insertions(+), 31 deletions(-)

-- 
2.47.3


^ permalink raw reply	[flat|nested] 6+ messages in thread

* [PATCH v4 4/4] dt-bindings: watchdog: renesas,rzn1-wdt: interrupts are not required
  2026-05-07 10:24 [PATCH v4 0/4] watchdog: rzn1: clean up and drop irq requirement Wolfram Sang
@ 2026-05-07 10:24 ` Wolfram Sang
  2026-05-07 11:22   ` Herve Codina
                     ` (3 more replies)
  0 siblings, 4 replies; 6+ messages in thread
From: Wolfram Sang @ 2026-05-07 10:24 UTC (permalink / raw)
  To: linux-renesas-soc
  Cc: Herve Codina (Schneider Electric), Wolfram Sang, Wim Van Sebroeck,
	Guenter Roeck, Rob Herring, Krzysztof Kozlowski, Conor Dooley,
	Geert Uytterhoeven, Magnus Damm, linux-watchdog, devicetree

It is now understood how the watchdog can do its job without the need of
an interrupt. So, it is not required anymore but optional.

Signed-off-by: Wolfram Sang <wsa+renesas@sang-engineering.com>
---
 Documentation/devicetree/bindings/watchdog/renesas,rzn1-wdt.yaml | 1 -
 1 file changed, 1 deletion(-)

diff --git a/Documentation/devicetree/bindings/watchdog/renesas,rzn1-wdt.yaml b/Documentation/devicetree/bindings/watchdog/renesas,rzn1-wdt.yaml
index 7e3ee533cd56..0e4b5b529e9c 100644
--- a/Documentation/devicetree/bindings/watchdog/renesas,rzn1-wdt.yaml
+++ b/Documentation/devicetree/bindings/watchdog/renesas,rzn1-wdt.yaml
@@ -29,7 +29,6 @@ properties:
 required:
   - compatible
   - reg
-  - interrupts
   - clocks
 
 allOf:
-- 
2.47.3


^ permalink raw reply related	[flat|nested] 6+ messages in thread

* Re: [PATCH v4 4/4] dt-bindings: watchdog: renesas,rzn1-wdt: interrupts are not required
  2026-05-07 10:24 ` [PATCH v4 4/4] dt-bindings: watchdog: renesas,rzn1-wdt: interrupts are not required Wolfram Sang
@ 2026-05-07 11:22   ` Herve Codina
  2026-05-08 16:32   ` Guenter Roeck
                     ` (2 subsequent siblings)
  3 siblings, 0 replies; 6+ messages in thread
From: Herve Codina @ 2026-05-07 11:22 UTC (permalink / raw)
  To: Wolfram Sang
  Cc: linux-renesas-soc, Wim Van Sebroeck, Guenter Roeck, Rob Herring,
	Krzysztof Kozlowski, Conor Dooley, Geert Uytterhoeven,
	Magnus Damm, linux-watchdog, devicetree

Hi Wolfram,

On Thu,  7 May 2026 12:24:09 +0200
Wolfram Sang <wsa+renesas@sang-engineering.com> wrote:

> It is now understood how the watchdog can do its job without the need of
> an interrupt. So, it is not required anymore but optional.
> 
> Signed-off-by: Wolfram Sang <wsa+renesas@sang-engineering.com>
> ---
>  Documentation/devicetree/bindings/watchdog/renesas,rzn1-wdt.yaml | 1 -
>  1 file changed, 1 deletion(-)
> 

Reviewed-by: Herve Codina <herve.codina@bootlin.com>

Best regards,
Hervé

^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: [PATCH v4 4/4] dt-bindings: watchdog: renesas,rzn1-wdt: interrupts are not required
  2026-05-07 10:24 ` [PATCH v4 4/4] dt-bindings: watchdog: renesas,rzn1-wdt: interrupts are not required Wolfram Sang
  2026-05-07 11:22   ` Herve Codina
@ 2026-05-08 16:32   ` Guenter Roeck
  2026-05-13 22:28   ` Rob Herring (Arm)
  2026-05-13 22:39   ` Guenter Roeck
  3 siblings, 0 replies; 6+ messages in thread
From: Guenter Roeck @ 2026-05-08 16:32 UTC (permalink / raw)
  To: Wolfram Sang
  Cc: linux-renesas-soc, Herve Codina (Schneider Electric),
	Wim Van Sebroeck, Rob Herring, Krzysztof Kozlowski, Conor Dooley,
	Geert Uytterhoeven, Magnus Damm, linux-watchdog, devicetree

On Thu, May 07, 2026 at 12:24:09PM +0200, Wolfram Sang wrote:
> It is now understood how the watchdog can do its job without the need of
> an interrupt. So, it is not required anymore but optional.
> 
> Signed-off-by: Wolfram Sang <wsa+renesas@sang-engineering.com>
> Reviewed-by: Herve Codina <herve.codina@bootlin.com>

Waiting for DT maintainer approval.

For my reference:

Reviewed-by: Guenter Roeck <linux@roeck-us.net>

Thanks,
Guenter

> ---
>  Documentation/devicetree/bindings/watchdog/renesas,rzn1-wdt.yaml | 1 -
>  1 file changed, 1 deletion(-)
> 
> diff --git a/Documentation/devicetree/bindings/watchdog/renesas,rzn1-wdt.yaml b/Documentation/devicetree/bindings/watchdog/renesas,rzn1-wdt.yaml
> index 7e3ee533cd56..0e4b5b529e9c 100644
> --- a/Documentation/devicetree/bindings/watchdog/renesas,rzn1-wdt.yaml
> +++ b/Documentation/devicetree/bindings/watchdog/renesas,rzn1-wdt.yaml
> @@ -29,7 +29,6 @@ properties:
>  required:
>    - compatible
>    - reg
> -  - interrupts
>    - clocks
>  
>  allOf:

^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: [PATCH v4 4/4] dt-bindings: watchdog: renesas,rzn1-wdt: interrupts are not required
  2026-05-07 10:24 ` [PATCH v4 4/4] dt-bindings: watchdog: renesas,rzn1-wdt: interrupts are not required Wolfram Sang
  2026-05-07 11:22   ` Herve Codina
  2026-05-08 16:32   ` Guenter Roeck
@ 2026-05-13 22:28   ` Rob Herring (Arm)
  2026-05-13 22:39   ` Guenter Roeck
  3 siblings, 0 replies; 6+ messages in thread
From: Rob Herring (Arm) @ 2026-05-13 22:28 UTC (permalink / raw)
  To: Wolfram Sang
  Cc: Geert Uytterhoeven, Guenter Roeck, devicetree, Magnus Damm,
	Wim Van Sebroeck, linux-renesas-soc, Conor Dooley,
	Herve Codina (Schneider Electric), linux-watchdog,
	Krzysztof Kozlowski


On Thu, 07 May 2026 12:24:09 +0200, Wolfram Sang wrote:
> It is now understood how the watchdog can do its job without the need of
> an interrupt. So, it is not required anymore but optional.
> 
> Signed-off-by: Wolfram Sang <wsa+renesas@sang-engineering.com>
> ---
>  Documentation/devicetree/bindings/watchdog/renesas,rzn1-wdt.yaml | 1 -
>  1 file changed, 1 deletion(-)
> 

Acked-by: Rob Herring (Arm) <robh@kernel.org>


^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: [PATCH v4 4/4] dt-bindings: watchdog: renesas,rzn1-wdt: interrupts are not required
  2026-05-07 10:24 ` [PATCH v4 4/4] dt-bindings: watchdog: renesas,rzn1-wdt: interrupts are not required Wolfram Sang
                     ` (2 preceding siblings ...)
  2026-05-13 22:28   ` Rob Herring (Arm)
@ 2026-05-13 22:39   ` Guenter Roeck
  3 siblings, 0 replies; 6+ messages in thread
From: Guenter Roeck @ 2026-05-13 22:39 UTC (permalink / raw)
  To: Wolfram Sang
  Cc: linux-renesas-soc, Herve Codina (Schneider Electric),
	Wim Van Sebroeck, Rob Herring, Krzysztof Kozlowski, Conor Dooley,
	Geert Uytterhoeven, Magnus Damm, linux-watchdog, devicetree

On Thu, May 07, 2026 at 12:24:09PM +0200, Wolfram Sang wrote:
> It is now understood how the watchdog can do its job without the need of
> an interrupt. So, it is not required anymore but optional.
> 
> Signed-off-by: Wolfram Sang <wsa+renesas@sang-engineering.com>
> Reviewed-by: Herve Codina <herve.codina@bootlin.com>
> Reviewed-by: Guenter Roeck <linux@roeck-us.net>
> Acked-by: Rob Herring (Arm) <robh@kernel.org>

Applied to my watchdog-next branch.

Thanks,
Guenter

^ permalink raw reply	[flat|nested] 6+ messages in thread

end of thread, other threads:[~2026-05-13 22:39 UTC | newest]

Thread overview: 6+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2026-05-07 10:24 [PATCH v4 0/4] watchdog: rzn1: clean up and drop irq requirement Wolfram Sang
2026-05-07 10:24 ` [PATCH v4 4/4] dt-bindings: watchdog: renesas,rzn1-wdt: interrupts are not required Wolfram Sang
2026-05-07 11:22   ` Herve Codina
2026-05-08 16:32   ` Guenter Roeck
2026-05-13 22:28   ` Rob Herring (Arm)
2026-05-13 22:39   ` Guenter Roeck

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox