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

["Chawdhry, Manorit" <m-chawdhry@ti.com>]

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";
> +    };