[Linux-stm32] [PATCH] ARM: dts: stm32: Add timer interrupts

Uwe Kleine-König u.kleine-koenig at pengutronix.de
Thu Dec 16 13:07:24 UTC 2021


Hello Rob,

On Wed, Dec 15, 2021 at 06:49:33PM -0600, Rob Herring wrote:
> > diff --git a/Documentation/devicetree/bindings/mfd/st,stm32-timers.yaml b/Documentation/devicetree/bindings/mfd/st,stm32-timers.yaml
> > index 10b330d42901..5e4214d1613b 100644
> > --- a/Documentation/devicetree/bindings/mfd/st,stm32-timers.yaml
> > +++ b/Documentation/devicetree/bindings/mfd/st,stm32-timers.yaml
> > @@ -46,6 +46,19 @@ properties:
> >      minItems: 1
> >      maxItems: 7
> >
> > +  interrupts:
> > +    maxItems: 4
> 
> Please test this against your dts change. It will fail.

I thought I did. Probably I missed the error message in the noise ...

> You need a 'minItems: 1' otherwise 4 interrupts are always required.

Will address your feedback in a v3. (I sent a v2 already that still
suffers from the issues you pointed out.)

Best regards
Uwe

-- 
Pengutronix e.K.                           | Uwe Kleine-König            |
Industrial Linux Solutions                 | https://www.pengutronix.de/ |
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 484 bytes
Desc: not available
URL: <http://st-md-mailman.stormreply.com/pipermail/linux-stm32/attachments/20211216/f8482a86/attachment.sig>


More information about the Linux-stm32 mailing list