On 30/01/2026 07:04, Swamil Jain wrote: >>>>> + properties: >>>>> + reg-names: >>>>> + items: >>>>> + - const: common >>>>> + - const: vidl1 >>>>> + - const: vid >>>>> + - const: ovr1 >>>>> + - const: ovr2 >>>>> + - const: vp1 >>>>> + - const: vp2 >>>>> + - const: common1 >>>> >>>> Why are you defining the names twice? >>>> >>> >>> For AM62L we don't have "vid", "ovr2" and "vp2", the dtbs_check will fail. >>> Could you please suggest a better way? >> >> You already defined them at the top level. Here in the if/then schema, >> all you need is 'maxItems: 5' and 'minItems: 8'. But then you already >> have that as well. >> >> Rob > > Posted a v2[1], could you please review if the patch looks good? > > [1]: https://lore.kernel.org/all/[email protected]/
It is not good. Rob asked you what to do. Read his second sentence. You posted something completely different ignoring what he asked you. Also ignoring rules stated in the docs. Best regards, Krzysztof
