Re: [PATCH v9 7/7] dt-bindings: crypto: Document support for SPAcc

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi Krzysztof,
   Thanks for the quick review and I do really appreciate everybody's time here.
   If something got missed, it's just because of the exhaustive
hardware and the SystemC Model testing.
   We make minimal/incremental changes and run things in debug mode
which takes a lot of time,
   since this is a large code base. Never ignored anything till date.
   Every single comment has been and will be addressed. We will work
on code quality as per your inputs.

Warm regards,
PK


On Mon, Sep 30, 2024 at 6:50 PM Krzysztof Kozlowski <krzk@xxxxxxxxxx> wrote:
>
> On 30/09/2024 11:30, Pavitrakumar M wrote:
> > Add DT bindings related to the SPAcc driver for Documentation.
> > DWC Synopsys Security Protocol Accelerator(SPAcc) Hardware Crypto
> > Engine is a crypto IP designed by Synopsys.
> >
> > Co-developed-by: Bhoomika Kadabi <bhoomikak@xxxxxxxxxxxxxxx>
> > Signed-off-by: Bhoomika Kadabi <bhoomikak@xxxxxxxxxxxxxxx>
> > Co-developed-by: Pavitrakumar Managutte <pavitrakumarm@xxxxxxxxxxxxxxx>
> > Signed-off-by: Pavitrakumar Managutte <pavitrakumarm@xxxxxxxxxxxxxxx>
> > Acked-by: Ruud Derwig <Ruud.Derwig@xxxxxxxxxxxx>
> > ---
> >  .../bindings/crypto/snps,dwc-spacc.yaml       | 71 +++++++++++++++++++
> >  1 file changed, 71 insertions(+)
> >  create mode 100644 Documentation/devicetree/bindings/crypto/snps,dwc-spacc.yaml
>
> Bindings come before users, so please re-order your patches.

PK: Will re-order
>
>
> >
> > diff --git a/Documentation/devicetree/bindings/crypto/snps,dwc-spacc.yaml b/Documentation/devicetree/bindings/crypto/snps,dwc-spacc.yaml
> > new file mode 100644
> > index 000000000000..6b94d0aa7280
> > --- /dev/null
> > +++ b/Documentation/devicetree/bindings/crypto/snps,dwc-spacc.yaml
> > @@ -0,0 +1,71 @@
> > +# SPDX-License-Identifier: (GPL-2.0-only OR BSD-2-Clause)
> > +%YAML 1.2
> > +---
> > +$id: http://devicetree.org/schemas/crypto/snps,dwc-spacc.yaml#
> > +$schema: http://devicetree.org/meta-schemas/core.yaml#
> > +
> > +title: Synopsys DesignWare Security Protocol Accelerator(SPAcc) Crypto Engine
> > +
> > +maintainers:
> > +  - Ruud Derwig <Ruud.Derwig@xxxxxxxxxxxx>
> > +
> > +description:
> > +  DWC Synopsys Security Protocol Accelerator(SPAcc) Hardware Crypto Engine is
> > +  a crypto IP designed by Synopsys, that can accelerate cryptographic
> > +  operations.
> > +
> > +properties:
> > +  compatible:
> > +    contains:
>
> Nope, you cannot have contains. From where did you get it? Please use
> existing, recent bindings as starting point or just use exampl-eschema.

PK: Will fix that.
>
>
> Eh, you already got this comment and just ignored it.

PK: It got missed, never ignored. Too valuable to ignore comments from demigods.
>
>
> You ignored all other comments as well. This is quite disappointing to
> ask us to do the same review over and over.

PK: That never was the intent nor the impression I wanted to make.
Appreciate everybody's time here.
>
>
> Best regards,
> Krzysztof
>





[Index of Archives]     [Kernel]     [Gnu Classpath]     [Gnu Crypto]     [DM Crypt]     [Netfilter]     [Bugtraq]
  Powered by Linux