On 16/03/2023 10:53, AngeloGioacchino Del Regno wrote: > Hello Krzysztof, Nancy, > > Since this series has reached v29, can we please reach an agreement on the bindings > to use here, so that we can get this finally upstreamed? > > I will put some examples to try to get this issue resolved. > > ### Example 1: Constrain the number of GCE entries to *seven* array elements (7x4!) > > mediatek,gce-client-reg: > $ref: /schemas/types.yaml#/definitions/phandle-array > maxItems: 1 > description: The register of display function block to be set by gce. > There are 4 arguments in this property, gce node, subsys id, offset and > register size. The subsys id is defined in the gce header of each chips > include/dt-bindings/gce/<chip>-gce.h, mapping to the register of display > function block. > items: > minItems: 28 > maxItems: 28 > items: <----- this block doesn't seem to get checked :\ > - description: phandle of GCE > - description: GCE subsys id > - description: register offset > - description: register size This is what we would like to have but it requires exception in dtschema. Thus: > > > ### Example 2: Don't care about constraining the number of arguments > > mediatek,gce-client-reg: > $ref: /schemas/types.yaml#/definitions/phandle-array > maxItems: 1 > description: The register of display function block to be set by gce. > There are 4 arguments in this property, gce node, subsys id, offset and > register size. The subsys id is defined in the gce header of each chips > include/dt-bindings/gce/<chip>-gce.h, mapping to the register of display > function block. use this. Best regards, Krzysztof
