linux-kernel - Re: [RFC PATCH 04/10] dt-bindings: arm: apple: Add t6001/t6002 Mac Studio compatibles

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 for Android: free password hash cracker in your pocket

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

Message-ID: <20220910113714.GC4024@jannau.net>
Date:   Sat, 10 Sep 2022 13:37:14 +0200
From:   Janne Grunau <j@...nau.net>
To:     Krzysztof Kozlowski <krzysztof.kozlowski@...aro.org>
Cc:     asahi@...ts.linux.dev, Mark Kettenis <kettenis@...nbsd.org>,
        Alyssa Rosenzweig <alyssa@...enzweig.io>,
        Hector Martin <marcan@...can.st>,
        Krzysztof Kozlowski <krzysztof.kozlowski+dt@...aro.org>,
        Rob Herring <robh+dt@...nel.org>,
        Sven Peter <sven@...npeter.dev>, devicetree@...r.kernel.org,
        linux-arm-kernel@...ts.infradead.org, linux-kernel@...r.kernel.org
Subject: Re: [RFC PATCH 04/10] dt-bindings: arm: apple: Add t6001/t6002 Mac
 Studio compatibles

On 2022-09-10 11:57:34 +0200, Krzysztof Kozlowski wrote:
> On 09/09/2022 15:50, Janne Grunau wrote:
> > This adds the following apple,t6001 platform:
> > 
> > - apple,j375c - Mac Studio (M1 Max, 2022)
> > 
> > And the initial apple,t6002 platform:
> > 
> > - apple,j375d - Mac Studio (M1 Ultra, 2022)
> > 
> > Signed-off-by: Janne Grunau <j@...nau.net>
> > ---
> > 
> >  Documentation/devicetree/bindings/arm/apple.yaml | 11 ++++++++++-
> >  1 file changed, 10 insertions(+), 1 deletion(-)
> > 
> > diff --git a/Documentation/devicetree/bindings/arm/apple.yaml b/Documentation/devicetree/bindings/arm/apple.yaml
> > index 8d93e8a6cc18..b34e56b9dc59 100644
> > --- a/Documentation/devicetree/bindings/arm/apple.yaml
> > +++ b/Documentation/devicetree/bindings/arm/apple.yaml
> > @@ -19,12 +19,14 @@ description: |
> >    - MacBook Air (M1, 2020)
> >    - iMac (24-inch, M1, 2021)
> >  
> > -  And devices based on the "M1 Pro" and "M1 Max" SoCs:
> > +  And devices based on the "M1 Pro", "M1 Max" and "M1 Ultra" SoCs:
> >  
> >    - MacBook Pro (14-inch, M1 Pro, 2021)
> >    - MacBook Pro (14-inch, M1 Max, 2021)
> >    - MacBook Pro (16-inch, M1 Pro, 2021)
> >    - MacBook Pro (16-inch, M1 Max, 2021)
> > +  - Mac Studio (M1 Max, 2022)
> > +  - Mac Studio (M1 Ultra, 2022)
> >  
> >    The compatible property should follow this format:
> >  
> > @@ -79,8 +81,15 @@ properties:
> >            - enum:
> >                - apple,j314c # MacBook Pro (14-inch, M1 Max, 2021)
> >                - apple,j316c # MacBook Pro (16-inch, M1 Max, 2021)
> > +              - apple,j375c # Mac Studio (M1 Max, 2022)
> >            - const: apple,t6001
> >            - const: apple,arm-platform
> 
> Blank line.

Added. The other entries in this list are not separated by blank lines.  
What would preferred to fix this? A whitespace only patch or can just 
add those additional blank lines in this change?

> > +      - description: Apple M1 Ultra SoC based platforms
> 
> With this:
> 
> 
> Acked-by: Krzysztof Kozlowski <krzysztof.kozlowski@...aro.org>

Thanks,
Janne