linux-kernel - Re: [PATCH v2 2/2] dt-bindings: cisco: document the CrayAR 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: Windows password security audit tool. GUI, reports in PDF.

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

Message-ID: <20230406151512.GL40472@zorba>
Date:   Thu, 6 Apr 2023 15:15:29 +0000
From:   "Daniel Walker (danielwa)" <danielwa@...co.com>
To:     Krzysztof Kozlowski <krzysztof.kozlowski@...aro.org>
CC:     "xe-linux-external(mailer list)" <xe-linux-external@...co.com>,
        "Marcin Wierzbicki -X (mawierzb - GLOBALLOGIC INC at Cisco)" 
        <mawierzb@...co.com>, Rob Herring <robh+dt@...nel.org>,
        Daniel Walker <dwalker@...o99.com>,
        Krzysztof Kozlowski <krzysztof.kozlowski+dt@...aro.org>,
        "devicetree@...r.kernel.org" <devicetree@...r.kernel.org>,
        "linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>
Subject: Re: [PATCH v2 2/2] dt-bindings: cisco: document the CrayAR
 compatibles

On Thu, Apr 06, 2023 at 09:12:34AM +0200, Krzysztof Kozlowski wrote:
> On 06/04/2023 00:30, Daniel Walker wrote:
> > Describe the compatible properties for the Cisco CrayAR SoC.
> > 
> > Cc: xe-linux-external@...co.com
> > Cc: Marcin Wierzbicki <mawierzb@...co.com>
> > Cc: Krzysztof Kozlowski <krzysztof.kozlowski@...aro.org>
> > Cc: Rob Herring <robh+dt@...nel.org>
> 
> Please drop the autogenerated scripts/get_maintainer.pl CC-entries from
> commit msg. There is no single need to store automated output of
> get_maintainers.pl in the git log. It can be easily re-created at any
> given time, thus its presence in the git history is redundant and
> obfuscates the log.
> 
> If you need it for your own patch management purposes, keep it under the
> --- separator.
 
I added these, so it's not script output. I can move them under the separator.
The reason for that was to make sure people who commented get Cc'd.

> > Signed-off-by: Daniel Walker <dwalker@...o99.com>
> > ---
> >  .../devicetree/bindings/arm/cisco/crayar.yaml | 27 +++++++++++++++++++
> >  1 file changed, 27 insertions(+)
> >  create mode 100644 Documentation/devicetree/bindings/arm/cisco/crayar.yaml
> > 
> > diff --git a/Documentation/devicetree/bindings/arm/cisco/crayar.yaml b/Documentation/devicetree/bindings/arm/cisco/crayar.yaml
> > new file mode 100644
> > index 000000000000..0ee4e6313ab0
> > --- /dev/null
> > +++ b/Documentation/devicetree/bindings/arm/cisco/crayar.yaml
> 
> How many (or not many) platforms do you expect from Cisco? We mostly
> have one file per SoC manufacturer.

We have two SoC's one called CrayAR and another called Craw. I've submitted the
Craw dts file before , but I've not resubmitted it. Under Craw I think we have
at least two, but more likely five variations. CrayAR we have one variation with
at least one more planned.

So we would plan over time to add two dtsi files and three to four dts files to
this directory. Then more over time.

Daniel