Re: [PATCH] dt-bindings: arm: Convert ti,keystone to DT schema

[Andrew Davis <afd@ti.com>]

On 8/6/25 4:28 PM, Rob Herring (Arm) wrote:
> Signed-off-by: Rob Herring (Arm) <robh@kernel.org>
> ---

Reviewed-by: Andrew Davis <afd@ti.com>

>   .../bindings/arm/keystone/keystone.txt        | 42 -------------------
>   .../bindings/arm/ti/ti,keystone.yaml          | 42 +++++++++++++++++++
>   2 files changed, 42 insertions(+), 42 deletions(-)
>   delete mode 100644 Documentation/devicetree/bindings/arm/keystone/keystone.txt
>   create mode 100644 Documentation/devicetree/bindings/arm/ti/ti,keystone.yaml
> 
> diff --git a/Documentation/devicetree/bindings/arm/keystone/keystone.txt b/Documentation/devicetree/bindings/arm/keystone/keystone.txt
> deleted file mode 100644
> index f310bad04483..000000000000
> --- a/Documentation/devicetree/bindings/arm/keystone/keystone.txt
> +++ /dev/null
> @@ -1,42 +0,0 @@
> -TI Keystone Platforms Device Tree Bindings
> ------------------------------------------------
> -
> -Boards with Keystone2 based devices (TCI66xxK2H) SOC shall have the
> -following properties.
> -
> -Required properties:
> - - compatible: All TI specific devices present in Keystone SOC should be in
> -   the form "ti,keystone-*". Generic devices like gic, arch_timers, ns16550
> -   type UART should use the specified compatible for those devices.
> -
> -SoC families:
> -
> -- Keystone 2 generic SoC:
> -   compatible = "ti,keystone"
> -
> -SoCs:
> -
> -- Keystone 2 Hawking/Kepler
> -   compatible = "ti,k2hk", "ti,keystone"
> -- Keystone 2 Lamarr
> -   compatible = "ti,k2l", "ti,keystone"
> -- Keystone 2 Edison
> -   compatible = "ti,k2e", "ti,keystone"
> -- K2G
> -   compatible = "ti,k2g", "ti,keystone"
> -
> -Boards:
> --  Keystone 2 Hawking/Kepler EVM
> -   compatible = "ti,k2hk-evm", "ti,k2hk", "ti,keystone"
> -
> --  Keystone 2 Lamarr EVM
> -   compatible = "ti,k2l-evm", "ti, k2l", "ti,keystone"
> -
> --  Keystone 2 Edison EVM
> -   compatible = "ti,k2e-evm", "ti,k2e", "ti,keystone"
> -
> --  K2G EVM
> -   compatible = "ti,k2g-evm", "ti,k2g", "ti-keystone"
> -
> --  K2G Industrial Communication Engine EVM
> -   compatible = "ti,k2g-ice", "ti,k2g", "ti-keystone"
> diff --git a/Documentation/devicetree/bindings/arm/ti/ti,keystone.yaml b/Documentation/devicetree/bindings/arm/ti/ti,keystone.yaml
> new file mode 100644
> index 000000000000..20d4084f4506
> --- /dev/null
> +++ b/Documentation/devicetree/bindings/arm/ti/ti,keystone.yaml
> @@ -0,0 +1,42 @@
> +# SPDX-License-Identifier: (GPL-2.0-only OR BSD-2-Clause)
> +%YAML 1.2
> +---
> +$id: http://devicetree.org/schemas/arm/ti/ti,keystone.yaml#
> +$schema: http://devicetree.org/meta-schemas/core.yaml#
> +
> +title: TI Keystone Platforms
> +
> +maintainers:
> +  - Nishanth Menon <nm@ti.com>
> +  - Santosh Shilimkar <ssantosh@kernel.org>
> +
> +properties:
> +  compatible:
> +    oneOf:
> +      - description: K2G
> +        items:
> +          - enum:
> +              - ti,k2g-evm
> +              - ti,k2g-ice
> +          - const: ti,k2g
> +          - const: ti,keystone
> +      - description: Keystone 2 Edison
> +        items:
> +          - enum:
> +              - ti,k2e-evm
> +          - const: ti,k2e
> +          - const: ti,keystone
> +      - description: Keystone 2 Lamarr
> +        items:
> +          - enum:
> +              - ti,k2l-evm
> +          - const: ti,k2l
> +          - const: ti,keystone
> +      - description: Keystone 2 Hawking/Kepler
> +        items:
> +          - enum:
> +              - ti,k2hk-evm
> +          - const: ti,k2hk
> +          - const: ti,keystone
> +
> +additionalProperties: true