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@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. 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
