Devicetree
 help / color / mirror / Atom feed
* [PATCH] dt-bindings: rtc: rzn1: add resets property
@ 2026-10-08 20:16 Wolfram Sang
  2026-10-08 20:23 ` sashiko-bot
  2026-10-09 15:36 ` Conor Dooley
  0 siblings, 2 replies; 3+ messages in thread
From: Wolfram Sang @ 2026-10-08 20:16 UTC (permalink / raw)
  To: linux-renesas-soc
  Cc: Wolfram Sang, Geert Uytterhoeven, Miquel Raynal,
	Alexandre Belloni, Rob Herring, Krzysztof Kozlowski, Conor Dooley,
	Magnus Damm, linux-rtc, devicetree

The RTC on R-Car X5H does have a reset unlike the other versions.

Suggested-by: Geert Uytterhoeven <geert+renesas@glider.be>
Signed-off-by: Wolfram Sang <wsa+renesas@sang-engineering.com>
---

Passes both DT checks. Kind of fixes 3b3fdc9fd40d ("dt-bindings: rtc:
rzn1: add R-Car X5H support") in -next, but we don't use Fixes: for
bindings, or?

 .../devicetree/bindings/rtc/renesas,rzn1-rtc.yaml          | 7 +++++++
 1 file changed, 7 insertions(+)

diff --git a/Documentation/devicetree/bindings/rtc/renesas,rzn1-rtc.yaml b/Documentation/devicetree/bindings/rtc/renesas,rzn1-rtc.yaml
index b448725671ec..5a869e6c0ae7 100644
--- a/Documentation/devicetree/bindings/rtc/renesas,rzn1-rtc.yaml
+++ b/Documentation/devicetree/bindings/rtc/renesas,rzn1-rtc.yaml
@@ -51,6 +51,9 @@ properties:
   power-domains:
     maxItems: 1
 
+  resets:
+    maxItems: 1
+
 required:
   - compatible
   - reg
@@ -85,7 +88,11 @@ allOf:
       properties:
         interrupts: false
         interrupt-names: false
+      required:
+        - resets
     else:
+      properties:
+        resets: false
       required:
         - interrupts
         - interrupt-names
-- 
2.53.0


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

* Re: [PATCH] dt-bindings: rtc: rzn1: add resets property
  2026-10-08 20:16 [PATCH] dt-bindings: rtc: rzn1: add resets property Wolfram Sang
@ 2026-10-08 20:23 ` sashiko-bot
  2026-10-09 15:36 ` Conor Dooley
  1 sibling, 0 replies; 3+ messages in thread
From: sashiko-bot @ 2026-10-08 20:23 UTC (permalink / raw)
  To: Wolfram Sang
  Cc: Alexandre Belloni, robh, conor+dt, wsa+renesas, linux-rtc,
	devicetree

> The RTC on R-Car X5H does have a reset unlike the other versions.
> 
> Suggested-by: Geert Uytterhoeven <geert+renesas@glider.be>
> Signed-off-by: Wolfram Sang <wsa+renesas@sang-engineering.com>

Sashiko has reviewed this patch and found no issues. It looks great!

-- 
Sashiko AI review · https://sashiko.dev/#/patchset/20261008201953.34307-2-wsa+renesas@sang-engineering.com?part=1


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

* Re: [PATCH] dt-bindings: rtc: rzn1: add resets property
  2026-10-08 20:16 [PATCH] dt-bindings: rtc: rzn1: add resets property Wolfram Sang
  2026-10-08 20:23 ` sashiko-bot
@ 2026-10-09 15:36 ` Conor Dooley
  1 sibling, 0 replies; 3+ messages in thread
From: Conor Dooley @ 2026-10-09 15:36 UTC (permalink / raw)
  To: Wolfram Sang
  Cc: linux-renesas-soc, Geert Uytterhoeven, Miquel Raynal,
	Alexandre Belloni, Rob Herring, Krzysztof Kozlowski, Conor Dooley,
	Magnus Damm, linux-rtc, devicetree

[-- Attachment #1: Type: text/plain, Size: 1501 bytes --]

On Thu, Oct 08, 2026 at 10:16:53PM +0200, Wolfram Sang wrote:
> The RTC on R-Car X5H does have a reset unlike the other versions.
> 
> Suggested-by: Geert Uytterhoeven <geert+renesas@glider.be>
> Signed-off-by: Wolfram Sang <wsa+renesas@sang-engineering.com>
> ---
> 
> Passes both DT checks. Kind of fixes 3b3fdc9fd40d ("dt-bindings: rtc:
> rzn1: add R-Car X5H support") in -next, but we don't use Fixes: for
> bindings, or?

If you want, make it more obvious IMO that this is a correction.

Acked-by: Conor Dooley <conor.dooley@microchip.com>
pw-bot: not-applicable

> 
>  .../devicetree/bindings/rtc/renesas,rzn1-rtc.yaml          | 7 +++++++
>  1 file changed, 7 insertions(+)
> 
> diff --git a/Documentation/devicetree/bindings/rtc/renesas,rzn1-rtc.yaml b/Documentation/devicetree/bindings/rtc/renesas,rzn1-rtc.yaml
> index b448725671ec..5a869e6c0ae7 100644
> --- a/Documentation/devicetree/bindings/rtc/renesas,rzn1-rtc.yaml
> +++ b/Documentation/devicetree/bindings/rtc/renesas,rzn1-rtc.yaml
> @@ -51,6 +51,9 @@ properties:
>    power-domains:
>      maxItems: 1
>  
> +  resets:
> +    maxItems: 1
> +
>  required:
>    - compatible
>    - reg
> @@ -85,7 +88,11 @@ allOf:
>        properties:
>          interrupts: false
>          interrupt-names: false
> +      required:
> +        - resets
>      else:
> +      properties:
> +        resets: false
>        required:
>          - interrupts
>          - interrupt-names
> -- 
> 2.53.0
> 

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 228 bytes --]

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

end of thread, other threads:[~2026-10-09 15:36 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2026-10-08 20:16 [PATCH] dt-bindings: rtc: rzn1: add resets property Wolfram Sang
2026-10-08 20:23 ` sashiko-bot
2026-10-09 15:36 ` Conor Dooley

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