On Tue, Sep 10, 2019 at 1:21 AM Philippe Schenker <philippe.schenker@xxxxxxxxxxx> wrote: > > This adds the documentation to the compatible regulator-fixed-clock. > This binding is a special binding of regulator-fixed and adds the > ability to add a clock to regulator-fixed, so the regulator can be > enabled and disabled with that clock. If the special compatible > regulator-fixed-clock is used it is mandatory to supply a clock. > > Signed-off-by: Philippe Schenker <philippe.schenker@xxxxxxxxxxx> > > --- > > Changes in v2: > - Change select: to if: > - Change items: to enum: > - Defined how many clocks should be given > > .../bindings/regulator/fixed-regulator.yaml | 19 ++++++++++++++++++- > 1 file changed, 18 insertions(+), 1 deletion(-) > > diff --git a/Documentation/devicetree/bindings/regulator/fixed-regulator.yaml b/Documentation/devicetree/bindings/regulator/fixed-regulator.yaml > index a650b457085d..a78150c47aa2 100644 > --- a/Documentation/devicetree/bindings/regulator/fixed-regulator.yaml > +++ b/Documentation/devicetree/bindings/regulator/fixed-regulator.yaml > @@ -19,9 +19,19 @@ description: > allOf: > - $ref: "regulator.yaml#" > > +if: > + properties: > + compatible: > + contains: > + const: regulator-fixed-clock > + required: > + - clocks > + > properties: > compatible: > - const: regulator-fixed > + enum: > + - const: regulator-fixed > + - const: regulator-fixed-clock 'make dt_binding_check' is failing. You need to drop 'const: '. Please send a patch to fix this. Rob