linux-kernel - Re: [PATCH 1/3] dt-bindings: arm: ti: Add bindings for AM62P5 SoCs

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <20230810184308.q67ldarld6qwo5uq@erased>
Date:   Thu, 10 Aug 2023 13:43:08 -0500
From:   Nishanth Menon <nm@...com>
To:     Conor Dooley <conor@...nel.org>
CC:     Vignesh Raghavendra <vigneshr@...com>,
        Tero Kristo <kristo@...nel.org>,
        Rob Herring <robh+dt@...nel.org>,
        Krzysztof Kozlowski <krzysztof.kozlowski+dt@...aro.org>,
        Conor Dooley <conor+dt@...nel.org>,
        <linux-arm-kernel@...ts.infradead.org>,
        <devicetree@...r.kernel.org>, <linux-kernel@...r.kernel.org>
Subject: Re: [PATCH 1/3] dt-bindings: arm: ti: Add bindings for AM62P5 SoCs

On 17:03-20230810, Conor Dooley wrote:
> On Thu, Aug 10, 2023 at 10:23:12AM +0530, Vignesh Raghavendra wrote:
> > From: Bryan Brattlof <bb@...com>
> > 
> > Add bindings for TI's AM62P5 family of devices.
> > 
> > Signed-off-by: Bryan Brattlof <bb@...com>
> > Signed-off-by: Vignesh Raghavendra <vigneshr@...com>
> > ---
> >  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 5ca6af492507..93b2774cc0a9 100644
> > --- a/Documentation/devicetree/bindings/arm/ti/k3.yaml
> > +++ b/Documentation/devicetree/bindings/arm/ti/k3.yaml
> > @@ -25,6 +25,12 @@ properties:
> >                - ti,am62a7-sk
> >            - const: ti,am62a7
> >  
> > +      - description: K3 AM62P5 SoC
> 
> SoC seems a bit off here, since the sk is actually a board using the
> SoC, but it seems to be the convention for TI stuff,

Thanks for providing that perspective - there will be new boards
getting added to the enum list, they should be one line additions at
this point.

But, your point taken. K3 AM625P SoC based boards is more appropriate.

I can do the local change as I apply and keep your ack unless you
object.

> Acked-by: Conor Dooley <conor.dooley@...rochip.com>
> 
> 
> 
> > +        items:
> > +          - enum:
> > +              - ti,am62p5-sk
> > +          - const: ti,am62p5
> > +
> >        - description: K3 AM625 SoC PHYTEC phyBOARD-Lyra
> >          items:
> >            - const: phytec,am625-phyboard-lyra-rdk
> > -- 
> > 2.41.0
> > 



-- 
Regards,
Nishanth Menon
Key (0xDDB5849D1736249D) / Fingerprint: F8A2 8693 54EB 8232 17A3  1A34 DDB5 849D 1736 249D