Hi Wolfram, On Tue, Sep 28, 2021 at 10:08 PM Wolfram Sang <wsa+renesas@xxxxxxxxxxxxxxxxxxxx> wrote: > Signed-off-by: Wolfram Sang <wsa+renesas@xxxxxxxxxxxxxxxxxxxx> Thanks for your patch! > --- a/Documentation/devicetree/bindings/mmc/renesas,sdhi.yaml > +++ b/Documentation/devicetree/bindings/mmc/renesas,sdhi.yaml > @@ -132,12 +132,13 @@ allOf: > properties: > clocks: > minItems: 1 > - maxItems: 2 > + maxItems: 3 > clock-names: > minItems: 1 > items: > - const: core > - const: cd > + - const: clkh This doesn't work, as the second clock is an optional clock, even if clkh is present. > > - if: > properties: Gr{oetje,eeting}s, Geert -- Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@xxxxxxxxxxxxxx In personal conversations with technical people, I call myself a hacker. But when I'm talking to journalists I just say "programmer" or something like that. -- Linus Torvalds