* [PATCH] crypto: ccp: remove duplicate module version and author entry
@ 2017-07-20  3:29 Brijesh Singh
  2017-08-03  6:23 ` Herbert Xu
  0 siblings, 1 reply; 2+ messages in thread
From: Brijesh Singh @ 2017-07-20  3:29 UTC (permalink / raw)
  To: linux-crypto
  Cc: Brijesh Singh, Tom Lendacky, Gary R Hook, Herbert Xu,
	David S . Miller
commit 720419f01832 ("crypto: ccp - Introduce the AMD Secure Processor device")
moved the module registeration from ccp-dev.c to sp-dev.c but patch missed
removing the module version and author entry from ccp-dev.c.
It causes the below warning during boot when CONFIG_CRYPTO_DEV_SP_CCP=y
and CONFIG_CRYPTO_DEV_CCP_CRYPTO=y is set.
[ 0.187825] sysfs: cannot create duplicate filename '/module/ccp/version'
[ 0.187825] sysfs: cannot create duplicate filename '/module/ccp/version'
Cc: Tom Lendacky <thomas.lendacky@amd.com>
Cc: Gary R Hook <gary.hook@amd.com>
Cc: Herbert Xu <herbert@gondor.apana.org.au>
Cc: David S. Miller <davem@davemloft.net>
Signed-off-by: Brijesh Singh <brijesh.singh@amd.com>
---
 drivers/crypto/ccp/ccp-dev.c | 7 -------
 1 file changed, 7 deletions(-)
diff --git a/drivers/crypto/ccp/ccp-dev.c b/drivers/crypto/ccp/ccp-dev.c
index b2d1761..b8695e7 100644
--- a/drivers/crypto/ccp/ccp-dev.c
+++ b/drivers/crypto/ccp/ccp-dev.c
@@ -11,7 +11,6 @@
  * published by the Free Software Foundation.
  */
 
-#include <linux/module.h>
 #include <linux/kernel.h>
 #include <linux/kthread.h>
 #include <linux/sched.h>
@@ -30,12 +29,6 @@
 
 #include "ccp-dev.h"
 
-MODULE_AUTHOR("Tom Lendacky <thomas.lendacky@amd.com>");
-MODULE_AUTHOR("Gary R Hook <gary.hook@amd.com>");
-MODULE_LICENSE("GPL");
-MODULE_VERSION("1.1.0");
-MODULE_DESCRIPTION("AMD Cryptographic Coprocessor driver");
-
 struct ccp_tasklet_data {
 	struct completion completion;
 	struct ccp_cmd *cmd;
-- 
2.9.4
^ permalink raw reply related	[flat|nested] 2+ messages in thread
* Re: [PATCH] crypto: ccp: remove duplicate module version and author entry
  2017-07-20  3:29 [PATCH] crypto: ccp: remove duplicate module version and author entry Brijesh Singh
@ 2017-08-03  6:23 ` Herbert Xu
  0 siblings, 0 replies; 2+ messages in thread
From: Herbert Xu @ 2017-08-03  6:23 UTC (permalink / raw)
  To: Brijesh Singh; +Cc: linux-crypto, Tom Lendacky, Gary R Hook, David S . Miller
On Wed, Jul 19, 2017 at 10:29:08PM -0500, Brijesh Singh wrote:
> commit 720419f01832 ("crypto: ccp - Introduce the AMD Secure Processor device")
> moved the module registeration from ccp-dev.c to sp-dev.c but patch missed
> removing the module version and author entry from ccp-dev.c.
> 
> It causes the below warning during boot when CONFIG_CRYPTO_DEV_SP_CCP=y
> and CONFIG_CRYPTO_DEV_CCP_CRYPTO=y is set.
> 
> [ 0.187825] sysfs: cannot create duplicate filename '/module/ccp/version'
> [ 0.187825] sysfs: cannot create duplicate filename '/module/ccp/version'
> 
> Cc: Tom Lendacky <thomas.lendacky@amd.com>
> Cc: Gary R Hook <gary.hook@amd.com>
> Cc: Herbert Xu <herbert@gondor.apana.org.au>
> Cc: David S. Miller <davem@davemloft.net>
> Signed-off-by: Brijesh Singh <brijesh.singh@amd.com>
Patch applied.  Thanks.
-- 
Email: Herbert Xu <herbert@gondor.apana.org.au>
Home Page: http://gondor.apana.org.au/~herbert/
PGP Key: http://gondor.apana.org.au/~herbert/pubkey.txt
^ permalink raw reply	[flat|nested] 2+ messages in thread
end of thread, other threads:[~2017-08-03  6:23 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2017-07-20  3:29 [PATCH] crypto: ccp: remove duplicate module version and author entry Brijesh Singh
2017-08-03  6:23 ` Herbert Xu
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).