> -----Original Message-----
> From: Horia Geanta
> Sent: Tuesday, July 23, 2019 2:38 PM
> To: Vakul Garg <vakul.g...@nxp.com>; linux-crypto@vger.kernel.org
> Cc: Aymen Sghaier <aymen.sgha...@nxp.com>;
> herb...@gondor.apana.org.au
> Subject: Re: [PATCH v4] crypto: caam/qi2 - Add printing dpseci fq stats using
> debugfs
> 
> On 7/23/2019 11:42 AM, Vakul Garg wrote:
> > diff --git a/drivers/crypto/caam/dpseci-debugfs.c
> > b/drivers/crypto/caam/dpseci-debugfs.c
> > new file mode 100644
> > index 000000000000..2e43ba9b7491
> > --- /dev/null
> > +++ b/drivers/crypto/caam/dpseci-debugfs.c
> > @@ -0,0 +1,80 @@
> > +/* SPDX-License-Identifier: (GPL-2.0+ OR BSD-3-Clause) */
> > +/* Copyright 2019 NXP
> > + */
> 
> .c and .h files have different syntax, see:
> https://www.kernel.org/doc/html/latest/process/license-rules.html#license-
> identifier-syntax
>

Thanks for pointing them out.
Earlier, I was not able to fix checkpatch warnings related to license headers.

> > diff --git a/drivers/crypto/caam/dpseci-debugfs.h
> > b/drivers/crypto/caam/dpseci-debugfs.h
> > new file mode 100644
> > index 000000000000..1dbdb2587758
> > --- /dev/null
> > +++ b/drivers/crypto/caam/dpseci-debugfs.h
> > @@ -0,0 +1,19 @@
> > +/* SPDX-License-Identifier: (GPL-2.0+ OR BSD-3-Clause) */
> > +/* Copyright 2019 NXP
> > + */
> 
> Yet another nitpick: incorrect commenting style (for the 2nd comment)
> https://www.kernel.org/doc/html/latest/process/coding-
> style.html#commenting
> 
> Why not make it a single-line comment?
> 
> Sorry for not catching these earlier.
> 


> Horia

Reply via email to