Re: [PATCH 3/6] dt-bindings: rtc: rzn1: interrupts are not required

[Conor Dooley <conor@kernel.org>]

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

On Tue, Sep 15, 2026 at 10:55:20AM +0100, Lad, Prabhakar wrote:
> Hi Wolfram,
> 
> Thank you for the patch.
> 
> On Mon, Sep 14, 2026 at 10:24 PM Wolfram Sang
> <wsa+renesas@sang-engineering.com> wrote:
> >
> > Interrupts are only needed for alarms to work. Reading/Setting the RTC
> > works fine without interrupts. Drop the requirement for interrupts
> > because an upcoming SoC does not have interrupts wired.
> >
> > Signed-off-by: Wolfram Sang <wsa+renesas@sang-engineering.com>
> > ---
> >  Documentation/devicetree/bindings/rtc/renesas,rzn1-rtc.yaml | 2 --
> >  1 file changed, 2 deletions(-)
> >
> > diff --git a/Documentation/devicetree/bindings/rtc/renesas,rzn1-rtc.yaml b/Documentation/devicetree/bindings/rtc/renesas,rzn1-rtc.yaml
> > index 7ea77cd0bee1..b426a34958d6 100644
> > --- a/Documentation/devicetree/bindings/rtc/renesas,rzn1-rtc.yaml
> > +++ b/Documentation/devicetree/bindings/rtc/renesas,rzn1-rtc.yaml
> > @@ -52,8 +52,6 @@ properties:
> >  required:
> >    - compatible
> >    - reg
> > -  - interrupts
> > -  - interrupt-names
> Maybe we can add a conditional schema check for SoCs where
> interrupts{-names} are required.

Yeah, I'd prefer this. If all current devices have the interrupt wired,
there's little value in permitting them to not be populated and thereby
lose the support for alarms.

> 
> Cheers,
> Prabhakar
> 
> >    - clocks
> >    - clock-names
> >    - power-domains
> > --
> > 2.53.0
> >
> >

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