Re: [PATCH v2] dt-bindings: iio: dac: mcp47feb02: Fix maxItems value for reg property
From: David Lechner
Date: Tue Apr 21 2026 - 10:24:05 EST
On 4/21/26 9:11 AM, Ariana.Lazar@xxxxxxxxxxxxx wrote:
> Hello Rob,
>
>>> @@ -161,8 +161,7 @@ patternProperties:
>>> properties:
>>> reg:
>>> description: The channel number.
>>> - minItems: 1
>>> - maxItems: 8
>>> + maxItems: 1
>>
>> Perhaps there are 8 channels and 'maximum: 8' is what was intended?
>
>
>
> Thank you for the review.
>
> Indeed, I mistakenly set maxItems to the maximum number of channels,
> rather than to a single entry for the reg property (reg value is
> between 0 and max 7 based on the part number).
>
> If preferred, I can also add minimum: 0 and maximum: 7 to specify the
> valid range for the channel number.
>
Yes, that sounds like the right idea.