diff mbox series

[-next] crypto: ixp4xx -: use DEFINE_SPINLOCK() for spinlock

Message ID 1617710577-48660-1-git-send-email-huangguobin4@huawei.com
State Accepted
Commit 7dad7d007ab73b36a4a2438f063dfabbdc2df288
Headers show
Series [-next] crypto: ixp4xx -: use DEFINE_SPINLOCK() for spinlock | expand

Commit Message

Huang Guobin April 6, 2021, 12:02 p.m. UTC
From: Guobin Huang <huangguobin4@huawei.com>

spinlock can be initialized automatically with DEFINE_SPINLOCK()
rather than explicitly calling spin_lock_init().

Reported-by: Hulk Robot <hulkci@huawei.com>
Signed-off-by: Guobin Huang <huangguobin4@huawei.com>
---
 drivers/crypto/ixp4xx_crypto.c | 7 ++-----
 1 file changed, 2 insertions(+), 5 deletions(-)

Comments

Herbert Xu April 16, 2021, 11:29 a.m. UTC | #1
On Tue, Apr 06, 2021 at 08:02:57PM +0800, Huang Guobin wrote:
> From: Guobin Huang <huangguobin4@huawei.com>

> 

> spinlock can be initialized automatically with DEFINE_SPINLOCK()

> rather than explicitly calling spin_lock_init().

> 

> Reported-by: Hulk Robot <hulkci@huawei.com>

> Signed-off-by: Guobin Huang <huangguobin4@huawei.com>

> ---

>  drivers/crypto/ixp4xx_crypto.c | 7 ++-----

>  1 file changed, 2 insertions(+), 5 deletions(-)


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
diff mbox series

Patch

diff --git a/drivers/crypto/ixp4xx_crypto.c b/drivers/crypto/ixp4xx_crypto.c
index 8b0f17fc09fb..0616e369522e 100644
--- a/drivers/crypto/ixp4xx_crypto.c
+++ b/drivers/crypto/ixp4xx_crypto.c
@@ -265,7 +265,7 @@  static int setup_crypt_desc(void)
 	return 0;
 }
 
-static spinlock_t desc_lock;
+static DEFINE_SPINLOCK(desc_lock);
 static struct crypt_ctl *get_crypt_desc(void)
 {
 	int i;
@@ -293,7 +293,7 @@  static struct crypt_ctl *get_crypt_desc(void)
 	}
 }
 
-static spinlock_t emerg_lock;
+static DEFINE_SPINLOCK(emerg_lock);
 static struct crypt_ctl *get_crypt_desc_emerg(void)
 {
 	int i;
@@ -1379,9 +1379,6 @@  static int __init ixp_module_init(void)
 	if (IS_ERR(pdev))
 		return PTR_ERR(pdev);
 
-	spin_lock_init(&desc_lock);
-	spin_lock_init(&emerg_lock);
-
 	err = init_ixp_crypto(&pdev->dev);
 	if (err) {
 		platform_device_unregister(pdev);