Re: [PATCH] dt-bindings: watchdog: Convert marvell,orion-wdt to DT schema

From: Gregory CLEMENT

Date: Tue Oct 14 2025 - 08:40:02 EST


Rob Herring <robh@xxxxxxxxxx> writes:

> On Tue, Oct 14, 2025 at 4:45 AM Gregory CLEMENT
> <gregory.clement@xxxxxxxxxxx> wrote:
>>
>> Hello Rob,
>>
>> Thank you for your work. I have one question:
>>
>> > + reg:
>> > + minItems: 2
>>
>> Should we also include this constraint here?
>>
>> maxItems: 3
>
> No, that is implicit with the length of 'items'. (For json-schema,
> it's not though. The tools add it automatically.)

Thanks for the explanation!

then for this patch:

Acked-by: Gregory CLEMENT <gregory.clement@xxxxxxxxxxx>

Gregory

>
>>
>> This would further restrict the binding.
>>
>> Gregory
>>
>> > + items:
>> > + - description: Timer control register address.
>> > + - description: RSTOUT enable register address.
>> > + - description: Shared mask/unmask RSTOUT register address.
>> > +

--
Grégory CLEMENT, Bootlin
Embedded Linux and Kernel engineering
https://bootlin.com