On Wed, Nov 20, 2019 at 12:49 AM Manish Narani <manish.narani@xxxxxxxxxx> wrote: > > Add optional properties for mmc hosts which are used to set clk delays > for different speed modes in the controller. > > Signed-off-by: Manish Narani <manish.narani@xxxxxxxxxx> > --- > .../devicetree/bindings/mmc/mmc-controller.yaml | 13 +++++++++++++ > 1 file changed, 13 insertions(+) > > diff --git a/Documentation/devicetree/bindings/mmc/mmc-controller.yaml b/Documentation/devicetree/bindings/mmc/mmc-controller.yaml > index 080754e0ef35..305b2016bc17 100644 > --- a/Documentation/devicetree/bindings/mmc/mmc-controller.yaml > +++ b/Documentation/devicetree/bindings/mmc/mmc-controller.yaml > @@ -333,6 +333,18 @@ patternProperties: > required: > - reg > > + "^clk-phase-(legacy|sd-hs|mmc-(hs|hs[24]00|ddr52)|uhs-(sdr(12|25|50|104)|ddr50))$": > + minItems: 2 > + maxItems: 2 > + allOf: > + - $ref: /schemas/types.yaml#/definitions/uint32 > + - minimum: 0 > + maximum: 359 This is wrong. It can't be both minItems of 2 and a single uint32. What's needed is: allOf: - $ref: /schemas/types.yaml#/definitions/uint32-array minItems: 2 maxItems: 2 items: minimum: 0 maximum: 359