On Mon, 2024-12-23 at 16:39 +0100, Krzysztof Kozlowski wrote: > On 23/12/2024 16:31, André Draszik wrote: > > On Mon, 2024-12-23 at 15:14 +0100, Krzysztof Kozlowski wrote: > > > > > > > > You now say that these are valid boards: > > > > > > compatible = "google,gs101", "google,gs101"; > > > > Sorry, I don't see how (apart from the fact that dtbs_check flags > > non-unique elements anyway). The result of the patch is: > > > > minItems: 2 > > maxItems: 3 > > items: > > enum: > > - google,gs101-oriole > > - google,gs101-raven > > - google,gs101-pixel > > - google,gs101 > > The problem is this line. Although entire concept of flexible list is > neither need nor ever recommended. All of this was inspired by Documentation/devicetree/bindings/soc/xilinx/xilinx.yaml I guess not a good example in this case... > > > allOf: > > - contains: > > const: google,gs101-pixel > > - contains: > > const: google,gs101 > > > > So one can not have 'google,gs101' twice. And if I only add > > Still can be, but indeed not with my example but: > > "google,gs101", "google,gs101", "google,gs101-pixel"; This example doesn't pass irrespective of binding, because dtbs_check will complain about non-unique elements. Anyway, will use separate entries. Thanks Krzysztof, Cheers, Andre' >
