[PATCH] dt-bindings: clock: ti: Convert ti-clkctrl.txt to json-schema

[PATCH] dt-bindings: clock: ti: Convert ti-clkctrl.txt to json-schema

[Andreas Kemnade]

Convert the TI clkctrl clock device tree binding to json-schema.
Specify the creator of the original binding as a maintainer.

reg property is used mostly with one item, in am3xxx also with
an arbitrary number of items, so divert from the original binding
specifying two (probably meaning one address and one size).
The consumer part of the example is left out because the full consumer
node would be needed.

Signed-off-by: Andreas Kemnade <andreas@kemnade.info>
---
@Tony: you seem to be the only contributor to the txt binding,
so we could go with dual-licensing if you agree.

 .../devicetree/bindings/clock/ti,clkctrl.yaml | 64 +++++++++++++++++++
 1 file changed, 64 insertions(+)
 create mode 100644 Documentation/devicetree/bindings/clock/ti,clkctrl.yaml

diff --git a/Documentation/devicetree/bindings/clock/ti,clkctrl.yaml b/Documentation/devicetree/bindings/clock/ti,clkctrl.yaml
new file mode 100644
index 0000000000000..bf4119c9c61fe
--- /dev/null
+++ b/Documentation/devicetree/bindings/clock/ti,clkctrl.yaml
@@ -0,0 +1,64 @@
+# SPDX-License-Identifier: GPL-2.0-only
+%YAML 1.2
+---
+$id: http://devicetree.org/schemas/clock/ti,clkctrl.yaml#
+$schema: http://devicetree.org/meta-schemas/core.yaml#
+
+title: Texas Instruments clkctrl clock
+
+maintainers:
+  - Tony Lindgren <tony@atomide.com>
+
+description: |
+  Texas Instruments SoCs can have a clkctrl clock controller for each
+  interconnect target module. The clkctrl clock controller manages functional
+  and interface clocks for each module. Each clkctrl controller can also
+  gate one or more optional functional clocks for a module, and can have one
+  or more clock muxes. There is a clkctrl clock controller typically for each
+  interconnect target module on omap4 and later variants.
+
+  The clock consumers can specify the index of the clkctrl clock using
+  the hardware offset from the clkctrl instance register space. The optional
+  clocks can be specified by clkctrl hardware offset and the index of the
+  optional clock.
+
+properties:
+  compatible:
+    enum:
+      - ti,clkctrl
+      - ti,clkctrl-l4-cfg
+      - ti,clkctrl-l4-per
+      - ti,clkctrl-l4-secure
+      - ti,clkctrl-l4-wkup
+
+  "#clock-cells":
+    const: 2
+
+  clock-output-names:
+    maxItems: 1
+
+  reg:
+    minItems: 1
+    maxItems: 8 # arbitrary, should be enough
+
+required:
+  - compatible
+  - "#clock-cells"
+  - clock-output-names
+  - reg
+
+additionalProperties: false
+
+examples:
+  - |
+    bus {
+      #address-cells = <1>;
+      #size-cells = <1>;
+
+      clock@20 {
+        compatible = "ti,clkctrl";
+        clock-output-names = "l4_per";
+        reg = <0x20 0x1b0>;
+        #clock-cells = <2>;
+      };
+    };
-- 
2.39.5

Re: [PATCH] dt-bindings: clock: ti: Convert ti-clkctrl.txt to json-schema

[Krzysztof Kozlowski]

On Wed, Mar 05, 2025 at 11:47:22PM +0100, Andreas Kemnade wrote:
> Convert the TI clkctrl clock device tree binding to json-schema.

I do not see conversion here - old TXT?

Best regards,
Krzysztof

Re: [PATCH] dt-bindings: clock: ti: Convert ti-clkctrl.txt to json-schema

[Tony Lindgren]

* Andreas Kemnade <andreas@kemnade.info> [250305 22:47]:
> Convert the TI clkctrl clock device tree binding to json-schema.
> Specify the creator of the original binding as a maintainer.

Good to see this happening :)

> @Tony: you seem to be the only contributor to the txt binding,
> so we could go with dual-licensing if you agree.

Yes I agree dual-licensing makes sense for the binding.

> --- /dev/null
> +++ b/Documentation/devicetree/bindings/clock/ti,clkctrl.yaml
> @@ -0,0 +1,64 @@
> +# SPDX-License-Identifier: GPL-2.0-only
> +%YAML 1.2
> +---
> +$id: http://devicetree.org/schemas/clock/ti,clkctrl.yaml#
> +$schema: http://devicetree.org/meta-schemas/core.yaml#
> +
> +title: Texas Instruments clkctrl clock
> +
> +maintainers:
> +  - Tony Lindgren <tony@atomide.com>

Please add yourself as a maintainer too for the binding.

Regards,

Tony