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

Manorit Chawdhry posted 5 patches 1 year, 5 months ago
There is a newer version of this series
[PATCH v6 3/5] dt-bindings: arm: ti: Add bindings for J742S2 SoCs and Boards
Posted by Manorit Chawdhry 1 year, 5 months ago
Add devicetree bindings for J742S2 family of devices.

Acked-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
Reviewed-by: Beleswar Padhi <b-padhi@ti.com>
Signed-off-by: Manorit Chawdhry <m-chawdhry@ti.com>
---

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
+        items:
+          - enum:
+              - ti,j742s2-evm
+          - const: ti,j742s2
+
       - description: K3 J784s4 SoC
         items:
           - enum:

-- 
2.46.0
Re: [PATCH v6 3/5] dt-bindings: arm: ti: Add bindings for J742S2 SoCs and Boards
Posted by Siddharth Vadapalli 1 year, 5 months ago
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@linaro.org>
> Reviewed-by: Beleswar Padhi <b-padhi@ti.com>
> Signed-off-by: Manorit Chawdhry <m-chawdhry@ti.com>
> ---
> 
> 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
>
Re: [PATCH v6 3/5] dt-bindings: arm: ti: Add bindings for J742S2 SoCs and Boards
Posted by Manorit Chawdhry 1 year, 5 months ago
Hi Siddharth,

On 12:28-20240903, Siddharth Vadapalli wrote:
> 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@linaro.org>
> > Reviewed-by: Beleswar Padhi <b-padhi@ti.com>
> > Signed-off-by: Manorit Chawdhry <m-chawdhry@ti.com>
> > ---
> > 
> > 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.

Even am not sure... I see the above one using J722S and there are
examples like AM62A7 as well, can see the majority being capitalized.
The delta's are AM62x, J784s4 and J721s2. I believe they should be
capitalized correctly if we need to go for any change.

Regards,
Manorit