On Wed, Sep 18, 2019 at 9:02 AM Pragnesh Patel <pragnesh.patel@xxxxxxxxxx> wrote: > The subject could be more specific rather than being one that applies to any dt_binding_check breakage in regulators. > Compatible property is not of type 'string' so replace enum > with items. > > Signed-off-by: Pragnesh Patel <pragnesh.patel@xxxxxxxxxx> > --- > Documentation/devicetree/bindings/regulator/fixed-regulator.yaml | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/Documentation/devicetree/bindings/regulator/fixed-regulator.yaml b/Documentation/devicetree/bindings/regulator/fixed-regulator.yaml > index a78150c..8d4a7b2 100644 > --- a/Documentation/devicetree/bindings/regulator/fixed-regulator.yaml > +++ b/Documentation/devicetree/bindings/regulator/fixed-regulator.yaml > @@ -29,7 +29,7 @@ if: > > properties: > compatible: > - enum: > + items: > - const: regulator-fixed > - const: regulator-fixed-clock This means you expect: compatible = "regulator-fixed", "regulator-fixed-clock"; Did you actually run 'dt_binding_check' because it should fail on the example? I gave you exact change to make. Just remove 'const: ' on each entry. Rob
