[v1] dt-bindings: arm: omap: Convert Counter-32K to DT schema

[PATCH] dt-bindings: arm: omap: Convert Counter-32K to DT schema

Posted by Eduard Bostina 1 week, 3 days ago

Convert the Texas Instruments OMAP Counter-32K bindings to DT schema.

During the conversion, two updates were made to reflect actual hardware
usage and resolve dtbs_check warnings:
- 'ti,hwmods' has been made optional. While a search confirms it is no
  longer used, it is kept to maintain backwards compatibility.
- Added the 'ti,am4372-counter32k' compatible string, as an AM437x
  device tree uses this instead of the generic 'ti,omap-counter32k'.

Signed-off-by: Eduard Bostina <egbostina@gmail.com>
---
 .../devicetree/bindings/arm/omap/counter.txt  | 15 -------
 .../bindings/arm/omap/ti,omap-counter32k.yaml | 41 +++++++++++++++++++
 2 files changed, 41 insertions(+), 15 deletions(-)
 delete mode 100644 Documentation/devicetree/bindings/arm/omap/counter.txt
 create mode 100644 Documentation/devicetree/bindings/arm/omap/ti,omap-counter32k.yaml

diff --git a/Documentation/devicetree/bindings/arm/omap/counter.txt b/Documentation/devicetree/bindings/arm/omap/counter.txt
deleted file mode 100644
index 5bd8aa091..000000000
--- a/Documentation/devicetree/bindings/arm/omap/counter.txt
+++ /dev/null
@@ -1,15 +0,0 @@
-OMAP Counter-32K bindings
-
-Required properties:
-- compatible:	Must be "ti,omap-counter32k" for OMAP controllers
-- reg:		Contains timer register address range (base address and length)
-- ti,hwmods:	Name of the hwmod associated to the counter, which is typically
-		"counter_32k"
-
-Example:
-
-counter32k: counter@4a304000 {
-	compatible = "ti,omap-counter32k";
-	reg = <0x4a304000 0x20>;
-	ti,hwmods = "counter_32k";
-};
diff --git a/Documentation/devicetree/bindings/arm/omap/ti,omap-counter32k.yaml b/Documentation/devicetree/bindings/arm/omap/ti,omap-counter32k.yaml
new file mode 100644
index 000000000..aadb5ea1b
--- /dev/null
+++ b/Documentation/devicetree/bindings/arm/omap/ti,omap-counter32k.yaml
@@ -0,0 +1,41 @@
+# SPDX-License-Identifier: (GPL-2.0-only OR BSD-2-Clause)
+%YAML 1.2
+---
+$id: http://devicetree.org/schemas/arm/omap/ti,omap-counter32k.yaml#
+$schema: http://devicetree.org/meta-schemas/core.yaml#
+
+title: Texas Instruments OMAP Counter-32K
+
+maintainers:
+  - Eduard Bostina <egbostina@gmail.com>
+
+properties:
+  compatible:
+    oneOf:
+      - const: ti,omap-counter32k
+      - items:
+          - const: ti,am4372-counter32k
+          - const: ti,omap-counter32k
+
+  reg:
+    maxItems: 1
+
+  ti,hwmods:
+    description: Name of the hwmod associated to the counter
+    $ref: /schemas/types.yaml#/definitions/string-array
+    items:
+      - const: counter_32k
+
+required:
+  - compatible
+  - reg
+
+additionalProperties: false
+
+examples:
+  - |
+    counter32k: counter@4a304000 {
+        compatible = "ti,omap-counter32k";
+        reg = <0x4a304000 0x20>;
+        ti,hwmods = "counter_32k";
+    };
-- 
2.54.0

Re: [PATCH] dt-bindings: arm: omap: Convert Counter-32K to DT schema

Posted by Conor Dooley 5 days, 20 hours ago

On Thu, May 28, 2026 at 07:58:50PM +0300, Eduard Bostina wrote:
> Convert the Texas Instruments OMAP Counter-32K bindings to DT schema.
> 
> During the conversion, two updates were made to reflect actual hardware
> usage and resolve dtbs_check warnings:
> - 'ti,hwmods' has been made optional. While a search confirms it is no
>   longer used, it is kept to maintain backwards compatibility.
> - Added the 'ti,am4372-counter32k' compatible string, as an AM437x
>   device tree uses this instead of the generic 'ti,omap-counter32k'.
> 
> Signed-off-by: Eduard Bostina <egbostina@gmail.com>
> ---
>  .../devicetree/bindings/arm/omap/counter.txt  | 15 -------
>  .../bindings/arm/omap/ti,omap-counter32k.yaml | 41 +++++++++++++++++++
>  2 files changed, 41 insertions(+), 15 deletions(-)
>  delete mode 100644 Documentation/devicetree/bindings/arm/omap/counter.txt
>  create mode 100644 Documentation/devicetree/bindings/arm/omap/ti,omap-counter32k.yaml
> 
> diff --git a/Documentation/devicetree/bindings/arm/omap/counter.txt b/Documentation/devicetree/bindings/arm/omap/counter.txt
> deleted file mode 100644
> index 5bd8aa091..000000000
> --- a/Documentation/devicetree/bindings/arm/omap/counter.txt
> +++ /dev/null
> @@ -1,15 +0,0 @@
> -OMAP Counter-32K bindings
> -
> -Required properties:
> -- compatible:	Must be "ti,omap-counter32k" for OMAP controllers
> -- reg:		Contains timer register address range (base address and length)
> -- ti,hwmods:	Name of the hwmod associated to the counter, which is typically
> -		"counter_32k"
> -
> -Example:
> -
> -counter32k: counter@4a304000 {
> -	compatible = "ti,omap-counter32k";
> -	reg = <0x4a304000 0x20>;
> -	ti,hwmods = "counter_32k";
> -};
> diff --git a/Documentation/devicetree/bindings/arm/omap/ti,omap-counter32k.yaml b/Documentation/devicetree/bindings/arm/omap/ti,omap-counter32k.yaml
> new file mode 100644
> index 000000000..aadb5ea1b
> --- /dev/null
> +++ b/Documentation/devicetree/bindings/arm/omap/ti,omap-counter32k.yaml
> @@ -0,0 +1,41 @@
> +# SPDX-License-Identifier: (GPL-2.0-only OR BSD-2-Clause)
> +%YAML 1.2
> +---
> +$id: http://devicetree.org/schemas/arm/omap/ti,omap-counter32k.yaml#
> +$schema: http://devicetree.org/meta-schemas/core.yaml#
> +
> +title: Texas Instruments OMAP Counter-32K
> +
> +maintainers:
> +  - Eduard Bostina <egbostina@gmail.com>
> +
> +properties:
> +  compatible:
> +    oneOf:
> +      - const: ti,omap-counter32k
> +      - items:
> +          - const: ti,am4372-counter32k
> +          - const: ti,omap-counter32k
> +
> +  reg:
> +    maxItems: 1
> +
> +  ti,hwmods:
> +    description: Name of the hwmod associated to the counter
> +    $ref: /schemas/types.yaml#/definitions/string-array
> +    items:
> +      - const: counter_32k
> +
> +required:
> +  - compatible
> +  - reg
> +
> +additionalProperties: false
> +
> +examples:
> +  - |
> +    counter32k: counter@4a304000 {

Drop the label here if you respin. In examples, there's no reason to
have labels with no users.

Acked-by: Conor Dooley <conor.dooley@microchip.com>
pw-bot: not-applicable

Cheers,
Conor.

> +        compatible = "ti,omap-counter32k";
> +        reg = <0x4a304000 0x20>;
> +        ti,hwmods = "counter_32k";
> +    };
> -- 
> 2.54.0
>

Re: [PATCH] dt-bindings: arm: omap: Convert Counter-32K to DT schema

Posted by Chawdhry, Manorit 1 week ago

Hi Eduard,

On 5/28/2026 10:28 PM, Eduard Bostina wrote:
> Convert the Texas Instruments OMAP Counter-32K bindings to DT schema.
>
> During the conversion, two updates were made to reflect actual hardware
> usage and resolve dtbs_check warnings:
> - 'ti,hwmods' has been made optional. While a search confirms it is no
>    longer used, it is kept to maintain backwards compatibility.
> - Added the 'ti,am4372-counter32k' compatible string, as an AM437x
>    device tree uses this instead of the generic 'ti,omap-counter32k'.

It's not instead, it's more of an alongside, the reason to keep this 
would be more of a backward
compatibility reason instead as we don't know if other software stacks 
relying on DTs are using
that compatible somehow.

Would be good to fix the commit message in v2 along with any other 
comments Krzysztof might have.

Regards,
Manorit

>
> Signed-off-by: Eduard Bostina <egbostina@gmail.com>
> ---
>   .../devicetree/bindings/arm/omap/counter.txt  | 15 -------
>   .../bindings/arm/omap/ti,omap-counter32k.yaml | 41 +++++++++++++++++++
>   2 files changed, 41 insertions(+), 15 deletions(-)
>   delete mode 100644 Documentation/devicetree/bindings/arm/omap/counter.txt
>   create mode 100644 Documentation/devicetree/bindings/arm/omap/ti,omap-counter32k.yaml
>
> diff --git a/Documentation/devicetree/bindings/arm/omap/counter.txt b/Documentation/devicetree/bindings/arm/omap/counter.txt
> deleted file mode 100644
> index 5bd8aa091..000000000
> --- a/Documentation/devicetree/bindings/arm/omap/counter.txt
> +++ /dev/null
> @@ -1,15 +0,0 @@
> -OMAP Counter-32K bindings
> -
> -Required properties:
> -- compatible:	Must be "ti,omap-counter32k" for OMAP controllers
> -- reg:		Contains timer register address range (base address and length)
> -- ti,hwmods:	Name of the hwmod associated to the counter, which is typically
> -		"counter_32k"
> -
> -Example:
> -
> -counter32k: counter@4a304000 {
> -	compatible = "ti,omap-counter32k";
> -	reg = <0x4a304000 0x20>;
> -	ti,hwmods = "counter_32k";
> -};
> diff --git a/Documentation/devicetree/bindings/arm/omap/ti,omap-counter32k.yaml b/Documentation/devicetree/bindings/arm/omap/ti,omap-counter32k.yaml
> new file mode 100644
> index 000000000..aadb5ea1b
> --- /dev/null
> +++ b/Documentation/devicetree/bindings/arm/omap/ti,omap-counter32k.yaml
> @@ -0,0 +1,41 @@
> +# SPDX-License-Identifier: (GPL-2.0-only OR BSD-2-Clause)
> +%YAML 1.2
> +---
> +$id: http://devicetree.org/schemas/arm/omap/ti,omap-counter32k.yaml#
> +$schema: http://devicetree.org/meta-schemas/core.yaml#
> +
> +title: Texas Instruments OMAP Counter-32K
> +
> +maintainers:
> +  - Eduard Bostina <egbostina@gmail.com>
> +
> +properties:
> +  compatible:
> +    oneOf:
> +      - const: ti,omap-counter32k
> +      - items:
> +          - const: ti,am4372-counter32k
> +          - const: ti,omap-counter32k
> +
> +  reg:
> +    maxItems: 1
> +
> +  ti,hwmods:
> +    description: Name of the hwmod associated to the counter
> +    $ref: /schemas/types.yaml#/definitions/string-array
> +    items:
> +      - const: counter_32k
> +
> +required:
> +  - compatible
> +  - reg
> +
> +additionalProperties: false
> +
> +examples:
> +  - |
> +    counter32k: counter@4a304000 {
> +        compatible = "ti,omap-counter32k";
> +        reg = <0x4a304000 0x20>;
> +        ti,hwmods = "counter_32k";
> +    };