On 31/12/2024 8:20 pm, Paul Handrigan wrote:
Add device tree support for the Cirrus Logic CS2600 clock device. Signed-off-by: Paul Handrigan <paulha@xxxxxxxxxxxxxxxxxxxxx> --- .../bindings/clock/cirrus,cs2600.yaml | 100 ++++++++++++++++++ 1 file changed, 100 insertions(+) create mode 100644 Documentation/devicetree/bindings/clock/cirrus,cs2600.yaml diff --git a/Documentation/devicetree/bindings/clock/cirrus,cs2600.yaml b/Documentation/devicetree/bindings/clock/cirrus,cs2600.yaml new file mode 100644 index 000000000000..1ef4d5ddfc51 --- /dev/null +++ b/Documentation/devicetree/bindings/clock/cirrus,cs2600.yaml @@ -0,0 +1,100 @@ +# SPDX-License-Identifier: (GPL-2.0-only OR BSD-2-Clause) +%YAML 1.2 +--- +$id: http://devicetree.org/schemas/clock/cirrus,cs2600.yaml# +$schema: http://devicetree.org/meta-schemas/core.yaml# + +title: Cirrus Logic Fractional-N Clock Synthesizer & Clock Multiplier + +maintainers: + - Paul Handrigan <paulha@xxxxxxxxxxxxxxxxxxxxx>
Paul no longer works at Cirrus so shouldn't be listed as a maintainer.
+ - patches@xxxxxxxxxxxxxxxxxxxxx> + +description: + The CS2600 is a system-clocking device that enables frequency synthesis and + clock generation from a stable timing reference clock. The device can + generate low-jitter clocks from a noisy clock reference at frequencies + as low as 50 Hz. The device has two potental clock inputs (xti and clk_in)
The pin isn't called "xti" in the datasheet. Also typo potental -> potential.
+ where xti can either be a crystal or a constant refclk and the clk_in. The + device can have either xti, clk_in, or both as a clock input depending on + the usecase. It also has three possible outputs CLK_OUT, BCLK_OUT, and + FSYNC_OUT. + +properties: + compatible: + enum: + - cirrus,cs2600 + + reg: + maxItems: 1 + + clocks: + minItems: 1 + maxItems: 2 + + clock-names: + items: + enum: + - xti
This doesn't match the datasheet pin name
+ - clk_in + minItems: 1 + maxItems: 2 + + '#clock-cells': + const: 1 + + vdd-supply: + description: Power Supply + + clock-output-names: + maxItems: 3 + description: Names for CLK_OUT, BCLK_OUT and FSYNC_OUT clocks. + + cirrus,aux-output-source: + description: + Specifies the function of the auxiliary output pin with "phase_unlock" + to indicate the input and output clocks are not in phase, "freq_unlock" + to indicate the PLL is unlocked, and "no_clkin" to indicate the clock on + the clk_in pin is not present. + $ref: /schemas/types.yaml#/definitions/string + enum: + - phase_unock + - freq_unlock + - no_clkin + + cirrus,clock-mode: + description: + Sets the device into smart mode whith "smart_mode" and sets the device + into smart mode that only outputs a clock when clk_in starts with + "smart_clkin_only_mode". + $ref: /schemas/types.yaml#/definitions/string + enum: + - smart_mode + - smart_clkin_only_mode
This doesn't need to be an enum. It can be two booleans. The two modes are only flags to indicate whether a feature is enabled.
