Hi Wolfram, Thanks for the patch. > Subject: [PATCH v3] dt-bindings: mmc: renesas,sdhi: add optional SDnH > clock > > This only applies to R-Car Gen2 and later generations, so we need to > distinguish. > > Signed-off-by: Wolfram Sang <wsa+renesas@xxxxxxxxxxxxxxxxxxxx> > Acked-by: Ulf Hansson <ulf.hansson@xxxxxxxxxx> > --- > > v1 and v2 were part of a 21-patch-series which was accepted now except for > this patch. Updated according to Geert's comments and finally also sent to > Rob and the DT mailing list. > > Tested with: > m dtbs_check > DT_SCHEMA_FILES=Documentation/devicetree/bindings/mmc/renesas,sdhi.yaml > > I hope it really does what I intended to check. > > If so, the patch can be applied individually. I think, however, it is most > convenient if Geert picks it up together with the 20 other patches. > > .../devicetree/bindings/mmc/renesas,sdhi.yaml | 40 ++++++++++++++----- > 1 file changed, 31 insertions(+), 9 deletions(-) > > diff --git a/Documentation/devicetree/bindings/mmc/renesas,sdhi.yaml > b/Documentation/devicetree/bindings/mmc/renesas,sdhi.yaml > index 9f1e7092cf44..43fc6ac56038 100644 > --- a/Documentation/devicetree/bindings/mmc/renesas,sdhi.yaml > +++ b/Documentation/devicetree/bindings/mmc/renesas,sdhi.yaml > @@ -129,15 +129,37 @@ allOf: > - clock-names > - resets > else: > - properties: > - clocks: > - minItems: 1 > - maxItems: 2 > - clock-names: > - minItems: 1 > - items: > - - const: core > - - const: cd > + if: > + properties: > + compatible: > + contains: > + enum: > + - renesas,rcar-gen2-sdhi > + - renesas,rcar-gen3-sdhi What about RZ/G2L, as it has 4 clocks? Regards, Biju > + then: > + properties: > + clocks: > + minItems: 1 > + maxItems: 3 > + clock-names: > + minItems: 1 > + maxItems: 3 > + uniqueItems: true > + items: > + - const: core > + - enum: [ clkh, cd ] > + - const: cd > + else: > + properties: > + clocks: > + minItems: 1 > + maxItems: 2 > + clock-names: > + minItems: 1 > + maxItems: 2 > + items: > + - const: core > + - const: cd > > - if: > properties: > -- > 2.30.2
