linux-kernel.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* [PATCH 0/4] Fixed comment coding style issues.
@ 2017-06-03 23:31 srishti sharma
  2017-06-03 23:32 ` [PATCH 1/4] Staging: ccree: cc_crypto_ctx.h: Added * on subsequent lines of a comment block srishti sharma
                   ` (3 more replies)
  0 siblings, 4 replies; 7+ messages in thread
From: srishti sharma @ 2017-06-03 23:31 UTC (permalink / raw)
  To: gregkh
  Cc: gilad, linux-crypto, driverdev-devel, devel, linux-kernel,
	srishti sharma

This patchset contains a series of comment coding style issues fixes.

srishti sharma (4):
  Staging: ccree: cc_crypto_ctx.h: Added * on subsequent lines of a
    comment block.
  Staging: ccree: cc_crypto_ctx.h: Fixed trailing */ issue in a comment
    block
  Staging: ccree: cc_crypto_ctx.h: Fixed * alignment issues in a comment
    block
  Staging: ccree: cc_crypto_ctx.h: Added * on subsequent lines of a
    comment block

 drivers/staging/ccree/cc_crypto_ctx.h | 19 ++++++++++---------
 1 file changed, 10 insertions(+), 9 deletions(-)

-- 
2.7.4

^ permalink raw reply	[flat|nested] 7+ messages in thread

* [PATCH 1/4] Staging: ccree: cc_crypto_ctx.h: Added * on subsequent lines of a comment block.
  2017-06-03 23:31 [PATCH 0/4] Fixed comment coding style issues srishti sharma
@ 2017-06-03 23:32 ` srishti sharma
  2017-06-04  8:19   ` Greg KH
  2017-06-03 23:32 ` [PATCH 2/4] Staging: ccree: cc_crypto_ctx.h: Fixed trailing */ issue in " srishti sharma
                   ` (2 subsequent siblings)
  3 siblings, 1 reply; 7+ messages in thread
From: srishti sharma @ 2017-06-03 23:32 UTC (permalink / raw)
  To: gregkh
  Cc: gilad, linux-crypto, driverdev-devel, devel, linux-kernel,
	srishti sharma

Added * on subsequent lines of a comment block.

Signed-off-by: srishti sharma <srishtishar@gmail.com>
---
 drivers/staging/ccree/cc_crypto_ctx.h | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/staging/ccree/cc_crypto_ctx.h b/drivers/staging/ccree/cc_crypto_ctx.h
index ac39d34..6ee51b8 100644
--- a/drivers/staging/ccree/cc_crypto_ctx.h
+++ b/drivers/staging/ccree/cc_crypto_ctx.h
@@ -219,7 +219,7 @@ struct drv_ctx_hash {
 };

 /* !!!! drv_ctx_hmac should have the same structure as drv_ctx_hash except
-   k0, k0_size fields */
+ * k0, k0_size fields */
 struct drv_ctx_hmac {
	enum drv_crypto_alg alg; /* DRV_CRYPTO_ALG_HMAC */
	enum drv_hash_mode mode;
--
2.7.4

^ permalink raw reply related	[flat|nested] 7+ messages in thread

* [PATCH 2/4] Staging: ccree: cc_crypto_ctx.h: Fixed trailing */ issue in a comment block
  2017-06-03 23:31 [PATCH 0/4] Fixed comment coding style issues srishti sharma
  2017-06-03 23:32 ` [PATCH 1/4] Staging: ccree: cc_crypto_ctx.h: Added * on subsequent lines of a comment block srishti sharma
@ 2017-06-03 23:32 ` srishti sharma
  2017-06-03 23:33 ` [PATCH 3/4] Staging: ccree: cc_crypto_ctx.h: Fixed * alignment issues " srishti sharma
  2017-06-03 23:33 ` [PATCH 4/4] Staging: ccree: cc_crypto_ctx.h: Added * on subsequent lines of " srishti sharma
  3 siblings, 0 replies; 7+ messages in thread
From: srishti sharma @ 2017-06-03 23:32 UTC (permalink / raw)
  To: gregkh
  Cc: gilad, linux-crypto, driverdev-devel, devel, linux-kernel,
	srishti sharma

Fixed trailing */ style issue in a block comment.

Signed-off-by: srishti sharma <srishtishar@gmail.com>
---
 drivers/staging/ccree/cc_crypto_ctx.h | 7 ++++---
 1 file changed, 4 insertions(+), 3 deletions(-)

diff --git a/drivers/staging/ccree/cc_crypto_ctx.h b/drivers/staging/ccree/cc_crypto_ctx.h
index 6ee51b8..27a5914 100644
--- a/drivers/staging/ccree/cc_crypto_ctx.h
+++ b/drivers/staging/ccree/cc_crypto_ctx.h
@@ -218,8 +218,10 @@ struct drv_ctx_hash {
			CC_DIGEST_SIZE_MAX];
 };

-/* !!!! drv_ctx_hmac should have the same structure as drv_ctx_hash except
- * k0, k0_size fields */
+/*
+ * !!!! drv_ctx_hmac should have the same structure as drv_ctx_hash except
+ * k0, k0_size fields
+ */
 struct drv_ctx_hmac {
	enum drv_crypto_alg alg; /* DRV_CRYPTO_ALG_HMAC */
	enum drv_hash_mode mode;
@@ -285,4 +287,3 @@ struct drv_ctx_aead {
 #define GET_CTX_FIELD_ADDR(ctx, type, member) (ctx + offsetof(type, member))

 #endif /* _CC_CRYPTO_CTX_H_ */
-
--
2.7.4

^ permalink raw reply related	[flat|nested] 7+ messages in thread

* [PATCH 3/4] Staging: ccree: cc_crypto_ctx.h: Fixed * alignment issues in a comment block
  2017-06-03 23:31 [PATCH 0/4] Fixed comment coding style issues srishti sharma
  2017-06-03 23:32 ` [PATCH 1/4] Staging: ccree: cc_crypto_ctx.h: Added * on subsequent lines of a comment block srishti sharma
  2017-06-03 23:32 ` [PATCH 2/4] Staging: ccree: cc_crypto_ctx.h: Fixed trailing */ issue in " srishti sharma
@ 2017-06-03 23:33 ` srishti sharma
  2017-06-03 23:33 ` [PATCH 4/4] Staging: ccree: cc_crypto_ctx.h: Added * on subsequent lines of " srishti sharma
  3 siblings, 0 replies; 7+ messages in thread
From: srishti sharma @ 2017-06-03 23:33 UTC (permalink / raw)
  To: gregkh
  Cc: gilad, linux-crypto, driverdev-devel, devel, linux-kernel,
	srishti sharma

Fixed the alignment of * in a comment block.

Signed-off-by: srishti sharma <srishtishar@gmail.com>
---
 drivers/staging/ccree/cc_crypto_ctx.h | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/drivers/staging/ccree/cc_crypto_ctx.h b/drivers/staging/ccree/cc_crypto_ctx.h
index 27a5914..03164624 100644
--- a/drivers/staging/ccree/cc_crypto_ctx.h
+++ b/drivers/staging/ccree/cc_crypto_ctx.h
@@ -242,9 +242,9 @@ struct drv_ctx_cipher {
	u32 key_size; /* numeric value in bytes */
	u32 data_unit_size; /* required for XTS */
	/* block_state is the AES engine block state.
-	*  It is used by the host to pass IV or counter at initialization.
-	*  It is used by SeP for intermediate block chaining state and for
-	*  returning MAC algorithms results. */
+	 * It is used by the host to pass IV or counter at initialization.
+	 * It is used by SeP for intermediate block chaining state and for
+	 * returning MAC algorithms results. */
	u8 block_state[CC_AES_BLOCK_SIZE];
	u8 key[CC_AES_KEY_SIZE_MAX];
	u8 xex_key[CC_AES_KEY_SIZE_MAX];
--
2.7.4

^ permalink raw reply related	[flat|nested] 7+ messages in thread

* [PATCH 4/4] Staging: ccree: cc_crypto_ctx.h: Added * on subsequent lines of a comment block
  2017-06-03 23:31 [PATCH 0/4] Fixed comment coding style issues srishti sharma
                   ` (2 preceding siblings ...)
  2017-06-03 23:33 ` [PATCH 3/4] Staging: ccree: cc_crypto_ctx.h: Fixed * alignment issues " srishti sharma
@ 2017-06-03 23:33 ` srishti sharma
  3 siblings, 0 replies; 7+ messages in thread
From: srishti sharma @ 2017-06-03 23:33 UTC (permalink / raw)
  To: gregkh
  Cc: gilad, linux-crypto, driverdev-devel, devel, linux-kernel,
	srishti sharma

Added *'s on subsequent lines of a comment block to fix coding style issues.

Signed-off-by: srishti sharma <srishtishar@gmail.com>
---
 drivers/staging/ccree/cc_crypto_ctx.h | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/drivers/staging/ccree/cc_crypto_ctx.h b/drivers/staging/ccree/cc_crypto_ctx.h
index 03164624..a8747d5 100644
--- a/drivers/staging/ccree/cc_crypto_ctx.h
+++ b/drivers/staging/ccree/cc_crypto_ctx.h
@@ -281,9 +281,9 @@ struct drv_ctx_aead {


 /* Get the address of a @member within a given @ctx address
-   @ctx: The context address
-   @type: Type of context structure
-   @member: Associated context field */
+ * @ctx: The context address
+ * @type: Type of context structure
+ * @member: Associated context field */
 #define GET_CTX_FIELD_ADDR(ctx, type, member) (ctx + offsetof(type, member))

 #endif /* _CC_CRYPTO_CTX_H_ */
--
2.7.4

^ permalink raw reply related	[flat|nested] 7+ messages in thread

* Re: [PATCH 1/4] Staging: ccree: cc_crypto_ctx.h: Added * on subsequent lines of a comment block.
  2017-06-03 23:32 ` [PATCH 1/4] Staging: ccree: cc_crypto_ctx.h: Added * on subsequent lines of a comment block srishti sharma
@ 2017-06-04  8:19   ` Greg KH
  2017-06-06 13:40     ` srishti sharma
  0 siblings, 1 reply; 7+ messages in thread
From: Greg KH @ 2017-06-04  8:19 UTC (permalink / raw)
  To: srishti sharma; +Cc: devel, driverdev-devel, linux-kernel, linux-crypto

On Sun, Jun 04, 2017 at 05:02:08AM +0530, srishti sharma wrote:
> Added * on subsequent lines of a comment block.
> 
> Signed-off-by: srishti sharma <srishtishar@gmail.com>
> ---
>  drivers/staging/ccree/cc_crypto_ctx.h | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)

This whole series also does not apply.  Note, this driver is under
active development, you are probably running into the problem that lots
of people are working on it at the same time.

sorry,

greg k-h

^ permalink raw reply	[flat|nested] 7+ messages in thread

* Re: [PATCH 1/4] Staging: ccree: cc_crypto_ctx.h: Added * on subsequent lines of a comment block.
  2017-06-04  8:19   ` Greg KH
@ 2017-06-06 13:40     ` srishti sharma
  0 siblings, 0 replies; 7+ messages in thread
From: srishti sharma @ 2017-06-06 13:40 UTC (permalink / raw)
  To: Greg KH; +Cc: devel, driverdev-devel, linux-kernel, linux-crypto

On Sun, Jun 4, 2017 at 1:49 PM, Greg KH <gregkh@linuxfoundation.org> wrote:
> On Sun, Jun 04, 2017 at 05:02:08AM +0530, srishti sharma wrote:
>> Added * on subsequent lines of a comment block.
>>
>> Signed-off-by: srishti sharma <srishtishar@gmail.com>
>> ---
>>  drivers/staging/ccree/cc_crypto_ctx.h | 2 +-
>>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> This whole series also does not apply.  Note, this driver is under
> active development, you are probably running into the problem that lots
> of people are working on it at the same time.
>
> sorry,
>
> greg k-h



Okay , Thanks a lot !

Regards,
Srishti

^ permalink raw reply	[flat|nested] 7+ messages in thread

end of thread, other threads:[~2017-06-06 13:40 UTC | newest]

Thread overview: 7+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2017-06-03 23:31 [PATCH 0/4] Fixed comment coding style issues srishti sharma
2017-06-03 23:32 ` [PATCH 1/4] Staging: ccree: cc_crypto_ctx.h: Added * on subsequent lines of a comment block srishti sharma
2017-06-04  8:19   ` Greg KH
2017-06-06 13:40     ` srishti sharma
2017-06-03 23:32 ` [PATCH 2/4] Staging: ccree: cc_crypto_ctx.h: Fixed trailing */ issue in " srishti sharma
2017-06-03 23:33 ` [PATCH 3/4] Staging: ccree: cc_crypto_ctx.h: Fixed * alignment issues " srishti sharma
2017-06-03 23:33 ` [PATCH 4/4] Staging: ccree: cc_crypto_ctx.h: Added * on subsequent lines of " srishti sharma

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).