Re: [PATCH v10 3/6] media: dt-bindings: Add Amlogic V4L2 video decoder
From: Zhentao Guo
Date: Tue Sep 29 2026 - 05:52:23 EST
On 29/09/2026 10:21, Zhentao Guo wrote:
Hi Krzysztof,So please kindly provide a proof that one entry was before allowed, how
On 29/09/2026 08:23, Zhentao Guo wrote:I re-ran the dt_binding_check against the new schema and a full
In the previous version of this patch,I do not think it is true, please test it.
allowing one entry was simply a mistake, not an option we ever supported.
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.
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