Re: [PATCH V3 1/2] dt-bindings: clk: versaclock5: Add optional load capacitance property
Quoting Adam Ford (2021-02-07 10:51:38) > There are two registers which can set the load capacitance for > XTAL1 and XTAL2. These are optional registers when using an > external crystal. Since XTAL1 and XTAL2 will set to the same value, > update the binding to support a single property called > xtal-load-femtofarads. > > Signed-off-by: Adam Ford > --- Applied to clk-next
Re: [PATCH V3 1/2] dt-bindings: clk: versaclock5: Add optional load capacitance property
On Wed, Feb 10, 2021 at 2:40 PM Adam Ford wrote: > > On Wed, Feb 10, 2021 at 2:18 PM Rob Herring wrote: > > > > On Sun, Feb 07, 2021 at 12:51:38PM -0600, Adam Ford wrote: > > > There are two registers which can set the load capacitance for > > > XTAL1 and XTAL2. These are optional registers when using an > > > external crystal. Since XTAL1 and XTAL2 will set to the same value, > > > update the binding to support a single property called > > > xtal-load-femtofarads. > > > > > > Signed-off-by: Adam Ford > > > --- > > > V3: No Change > > > V2: No Change > > > > > > A couple people suggested that I not use the $ref, but without it, > > > the bindings check failed with errors. > > > > > > diff --git a/Documentation/devicetree/bindings/clock/idt,versaclock5.yaml > > > b/Documentation/devicetree/bindings/clock/idt,versaclock5.yaml > > > index 2ac1131fd922..c268debe5b8d 100644 > > > --- a/Documentation/devicetree/bindings/clock/idt,versaclock5.yaml > > > +++ b/Documentation/devicetree/bindings/clock/idt,versaclock5.yaml > > > @@ -59,6 +59,12 @@ properties: > > > minItems: 1 > > > maxItems: 2 > > > > > > + idt,xtal-load-femtofarads: > > > +$ref: /schemas/types.yaml#/definitions/uint32 > > > > Don't need a type with standard unit suffix. > > If I remove that line, the binding check fails. Ah, looks like femtofarads got added to property-units.txt but not the schemas. I'll add it, but fine to leave this as-is for now. Reviewed-by: Rob Herring Rob
Re: [PATCH V3 1/2] dt-bindings: clk: versaclock5: Add optional load capacitance property
Quoting Adam Ford (2021-02-10 12:40:38) > On Wed, Feb 10, 2021 at 2:18 PM Rob Herring wrote: > > > > On Sun, Feb 07, 2021 at 12:51:38PM -0600, Adam Ford wrote: > > > There are two registers which can set the load capacitance for > > > XTAL1 and XTAL2. These are optional registers when using an > > > external crystal. Since XTAL1 and XTAL2 will set to the same value, > > > update the binding to support a single property called > > > xtal-load-femtofarads. > > > > > > Signed-off-by: Adam Ford > > > --- > > > V3: No Change > > > V2: No Change > > > > > > A couple people suggested that I not use the $ref, but without it, > > > the bindings check failed with errors. > > > > > > diff --git a/Documentation/devicetree/bindings/clock/idt,versaclock5.yaml > > > b/Documentation/devicetree/bindings/clock/idt,versaclock5.yaml > > > index 2ac1131fd922..c268debe5b8d 100644 > > > --- a/Documentation/devicetree/bindings/clock/idt,versaclock5.yaml > > > +++ b/Documentation/devicetree/bindings/clock/idt,versaclock5.yaml > > > @@ -59,6 +59,12 @@ properties: > > > minItems: 1 > > > maxItems: 2 > > > > > > + idt,xtal-load-femtofarads: > > > +$ref: /schemas/types.yaml#/definitions/uint32 > > > > Don't need a type with standard unit suffix. > > If I remove that line, the binding check fails. > Is your dt-schema up to date?
Re: [PATCH V3 1/2] dt-bindings: clk: versaclock5: Add optional load capacitance property
On Wed, Feb 10, 2021 at 2:18 PM Rob Herring wrote: > > On Sun, Feb 07, 2021 at 12:51:38PM -0600, Adam Ford wrote: > > There are two registers which can set the load capacitance for > > XTAL1 and XTAL2. These are optional registers when using an > > external crystal. Since XTAL1 and XTAL2 will set to the same value, > > update the binding to support a single property called > > xtal-load-femtofarads. > > > > Signed-off-by: Adam Ford > > --- > > V3: No Change > > V2: No Change > > > > A couple people suggested that I not use the $ref, but without it, > > the bindings check failed with errors. > > > > diff --git a/Documentation/devicetree/bindings/clock/idt,versaclock5.yaml > > b/Documentation/devicetree/bindings/clock/idt,versaclock5.yaml > > index 2ac1131fd922..c268debe5b8d 100644 > > --- a/Documentation/devicetree/bindings/clock/idt,versaclock5.yaml > > +++ b/Documentation/devicetree/bindings/clock/idt,versaclock5.yaml > > @@ -59,6 +59,12 @@ properties: > > minItems: 1 > > maxItems: 2 > > > > + idt,xtal-load-femtofarads: > > +$ref: /schemas/types.yaml#/definitions/uint32 > > Don't need a type with standard unit suffix. If I remove that line, the binding check fails. adam > > > +minimum: 9000 > > +maximum: 22760 > > +description: Optional load capacitor for XTAL1 and XTAL2 > > + > > patternProperties: > >"^OUT[1-4]$": > > type: object > > -- > > 2.25.1 > >
Re: [PATCH V3 1/2] dt-bindings: clk: versaclock5: Add optional load capacitance property
On Sun, Feb 07, 2021 at 12:51:38PM -0600, Adam Ford wrote: > There are two registers which can set the load capacitance for > XTAL1 and XTAL2. These are optional registers when using an > external crystal. Since XTAL1 and XTAL2 will set to the same value, > update the binding to support a single property called > xtal-load-femtofarads. > > Signed-off-by: Adam Ford > --- > V3: No Change > V2: No Change > > A couple people suggested that I not use the $ref, but without it, > the bindings check failed with errors. > > diff --git a/Documentation/devicetree/bindings/clock/idt,versaclock5.yaml > b/Documentation/devicetree/bindings/clock/idt,versaclock5.yaml > index 2ac1131fd922..c268debe5b8d 100644 > --- a/Documentation/devicetree/bindings/clock/idt,versaclock5.yaml > +++ b/Documentation/devicetree/bindings/clock/idt,versaclock5.yaml > @@ -59,6 +59,12 @@ properties: > minItems: 1 > maxItems: 2 > > + idt,xtal-load-femtofarads: > +$ref: /schemas/types.yaml#/definitions/uint32 Don't need a type with standard unit suffix. > +minimum: 9000 > +maximum: 22760 > +description: Optional load capacitor for XTAL1 and XTAL2 > + > patternProperties: >"^OUT[1-4]$": > type: object > -- > 2.25.1 >
[PATCH V3 1/2] dt-bindings: clk: versaclock5: Add optional load capacitance property
There are two registers which can set the load capacitance for XTAL1 and XTAL2. These are optional registers when using an external crystal. Since XTAL1 and XTAL2 will set to the same value, update the binding to support a single property called xtal-load-femtofarads. Signed-off-by: Adam Ford --- V3: No Change V2: No Change A couple people suggested that I not use the $ref, but without it, the bindings check failed with errors. diff --git a/Documentation/devicetree/bindings/clock/idt,versaclock5.yaml b/Documentation/devicetree/bindings/clock/idt,versaclock5.yaml index 2ac1131fd922..c268debe5b8d 100644 --- a/Documentation/devicetree/bindings/clock/idt,versaclock5.yaml +++ b/Documentation/devicetree/bindings/clock/idt,versaclock5.yaml @@ -59,6 +59,12 @@ properties: minItems: 1 maxItems: 2 + idt,xtal-load-femtofarads: +$ref: /schemas/types.yaml#/definitions/uint32 +minimum: 9000 +maximum: 22760 +description: Optional load capacitor for XTAL1 and XTAL2 + patternProperties: "^OUT[1-4]$": type: object -- 2.25.1
