Re: [PATCH v10 3/6] media: dt-bindings: Add Amlogic V4L2 video decoder

[Zhentao Guo <zhentao.guo@amlogic.com>]

> On 29/09/2026 10:21, Zhentao Guo wrote:
>> Hi Krzysztof,
>>
>>> On 29/09/2026 08:23, Zhentao Guo wrote:
>>>> In the previous version of this patch,
>>>> allowing one entry was simply a mistake, not an option we ever supported.
>>> I do not think it is true, please test it.
>> I re-ran the dt_binding_check against the new schema and a full
>> dtbs_check, the binding validates, its example compiles and passes, no
>> failures found.
>>
>> If there is any additional testing you would recommend, I would be glad
>> to run it.Or do you have any remaining concerns about the change? I'm
>> happy to clarify anything that is still unclear.
> So please kindly provide a proof that one entry was before allowed, how
> you claim. Remember, we speak ABOUT ONLY one thing - the quote I started
> with.

Sorry for missing your point eariler.I tested it:

With the previous version (|reg: maxItems: 2 |plus the 2-entry 
|reg-names |items list), a DTS with a single |reg |entry is rejected by 
dt_binding_check ("reg: ... is too short").

Let me clarify what I originally meant by "maxItems: 2 implies that a 
single item is also allowed" — my point was about readability, not about 
dtschema's actual validation behavior. I should have phrased it better.

Just to make sure if my understanding is right: the two items already 
listed in each |"*-names"|array imply an effective minimum of 2 for the 
properties (including regs, interrupts, power-domains), so I still need 
to specify the maxItems for them? I would greatly appreciate your 
correction if I have misunderstood this.

>> maxItems: 2 implies that a single items is also allowed,
>
> Best regards,
> Krzysztof

Best regards,

Zhentao


_______________________________________________
linux-amlogic mailing list
linux-amlogic@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-amlogic