* [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; 4+ 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] 4+ 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
2026-05-08 16:32 ` Guenter Roeck
0 siblings, 2 replies; 4+ 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] 4+ 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
1 sibling, 0 replies; 4+ 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] 4+ 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
1 sibling, 0 replies; 4+ 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] 4+ messages in thread
end of thread, other threads:[~2026-05-08 16:32 UTC | newest]
Thread overview: 4+ 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
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox