Re: [PATCH v6 3/5] dt-bindings: arm: ti: Add bindings for J742S2 SoCs and Boards

[Siddharth Vadapalli]

On Mon, Sep 02, 2024 at 05:56:51PM +0530, Manorit Chawdhry wrote:
> Add devicetree bindings for J742S2 family of devices.
> 
> Acked-by: Krzysztof Kozlowski <krzysztof.kozlowski@xxxxxxxxxx>
> Reviewed-by: Beleswar Padhi <b-padhi@xxxxxx>
> Signed-off-by: Manorit Chawdhry <m-chawdhry@xxxxxx>
> ---
> 
> Notes:
>     v6: No change
> 
>  Documentation/devicetree/bindings/arm/ti/k3.yaml | 6 ++++++
>  1 file changed, 6 insertions(+)
> 
> diff --git a/Documentation/devicetree/bindings/arm/ti/k3.yaml b/Documentation/devicetree/bindings/arm/ti/k3.yaml
> index 5df99e361c21..b0be02f9d125 100644
> --- a/Documentation/devicetree/bindings/arm/ti/k3.yaml
> +++ b/Documentation/devicetree/bindings/arm/ti/k3.yaml
> @@ -144,6 +144,12 @@ properties:
>                - ti,j722s-evm
>            - const: ti,j722s
>  
> +      - description: K3 J742S2 SoC

Is it J742S2 or J742s2? The naming seems to be inconsistent considering
that it is a lowercase 's' for J784s4 below.

> +        items:
> +          - enum:
> +              - ti,j742s2-evm
> +          - const: ti,j742s2
> +
>        - description: K3 J784s4 SoC
>          items:
>            - enum:
> 
> -- 
> 2.46.0
>