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

[Conor Dooley <conor@kernel.org>]

[-- Attachment #1: Type: text/plain, Size: 3320 bytes --]

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
> 

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 228 bytes --]