| Message ID | 20240928083804.1073942-2-dario.binacchi@amarulasolutions.com (mailing list archive) |
|---|---|
| State | New |
| Headers | show |
| Series | Support spread spectrum clocking for i.MX8{M,N,P} PLLs | expand |
On 28/09/2024 10:37, Dario Binacchi wrote: > The patch adds the DT bindings for enabling and tuning spread spectrum > clocking generation. > > Signed-off-by: Dario Binacchi <dario.binacchi@amarulasolutions.com> > --- > > .../bindings/clock/fsl,imx8m-anatop.yaml | 41 +++++++++++++++++++ > 1 file changed, 41 insertions(+) > > diff --git a/Documentation/devicetree/bindings/clock/fsl,imx8m-anatop.yaml b/Documentation/devicetree/bindings/clock/fsl,imx8m-anatop.yaml > index bbd22e95b319..c91eb4229ed3 100644 > --- a/Documentation/devicetree/bindings/clock/fsl,imx8m-anatop.yaml > +++ b/Documentation/devicetree/bindings/clock/fsl,imx8m-anatop.yaml > @@ -32,6 +32,47 @@ properties: > > '#clock-cells': > const: 1 > +if: This should be allOf: and placed after required: block, like in example schema. > + properties: > + compatible: > + contains: > + enum: > + - fsl,imx8mm-anatop > + > +then: > + properties: > + fsl,ssc-clocks: Nope. Properties must be defined in top-level. > + $ref: /schemas/types.yaml#/definitions/phandle-array > + description: > + The phandles to the PLLs with spread spectrum clock generation > + hardware capability. These should be clocks. > + maxItems: 4 > + > + fsl,ssc-modfreq-hz: > + $ref: /schemas/types.yaml#/definitions/uint32-array This should fail. I don't think you tested this patch. https://github.com/devicetree-org/dt-schema/blob/main/dtschema/schemas/property-units.yaml > + description: > + The values of modulation frequency (Hz unit) of spread spectrum > + clocking for each PLL. > + maxItems: 4 > + > + fsl,ssc-modrate-percent: Same problems Best regards, Krzysztof
diff --git a/Documentation/devicetree/bindings/clock/fsl,imx8m-anatop.yaml b/Documentation/devicetree/bindings/clock/fsl,imx8m-anatop.yaml index bbd22e95b319..c91eb4229ed3 100644 --- a/Documentation/devicetree/bindings/clock/fsl,imx8m-anatop.yaml +++ b/Documentation/devicetree/bindings/clock/fsl,imx8m-anatop.yaml @@ -32,6 +32,47 @@ properties: '#clock-cells': const: 1 +if: + properties: + compatible: + contains: + enum: + - fsl,imx8mm-anatop + +then: + properties: + fsl,ssc-clocks: + $ref: /schemas/types.yaml#/definitions/phandle-array + description: + The phandles to the PLLs with spread spectrum clock generation + hardware capability. + maxItems: 4 + + fsl,ssc-modfreq-hz: + $ref: /schemas/types.yaml#/definitions/uint32-array + description: + The values of modulation frequency (Hz unit) of spread spectrum + clocking for each PLL. + maxItems: 4 + + fsl,ssc-modrate-percent: + $ref: /schemas/types.yaml#/definitions/uint32-array + description: + The percentage values of modulation rate of spread spectrum + clocking for each PLL. + maxItems: 4 + + fsl,ssc-modmethod: + $ref: /schemas/types.yaml#/definitions/string-array + description: + The modulation techniques of spread spectrum clocking for + each PLL. + oneOf: + - enum: + - down-spread + - up-spread + - center-spread + maxItems: 4 required: - compatible
The patch adds the DT bindings for enabling and tuning spread spectrum clocking generation. Signed-off-by: Dario Binacchi <dario.binacchi@amarulasolutions.com> --- .../bindings/clock/fsl,imx8m-anatop.yaml | 41 +++++++++++++++++++ 1 file changed, 41 insertions(+)