diff mbox series

[-next] usb: misc: use DEFINE_MUTEX (and mutex_init() had been too late)

Message ID 20201223141044.32235-1-zhengyongjun3@huawei.com (mailing list archive)
State New, archived
Headers show
Series [-next] usb: misc: use DEFINE_MUTEX (and mutex_init() had been too late) | expand

Commit Message

Zheng Yongjun Dec. 23, 2020, 2:10 p.m. UTC
Signed-off-by: Zheng Yongjun <zhengyongjun3@huawei.com>
---
 drivers/usb/misc/ftdi-elan.c | 3 +--
 1 file changed, 1 insertion(+), 2 deletions(-)

Comments

Greg Kroah-Hartman Dec. 23, 2020, 2:17 p.m. UTC | #1
On Wed, Dec 23, 2020 at 10:10:44PM +0800, Zheng Yongjun wrote:
> Signed-off-by: Zheng Yongjun <zhengyongjun3@huawei.com>
> ---
>  drivers/usb/misc/ftdi-elan.c | 3 +--
>  1 file changed, 1 insertion(+), 2 deletions(-)

I can not take patches without any changelog text, sorry.

And try including the driver name in the subject line please.

thanks,

greg k-h
diff mbox series

Patch

diff --git a/drivers/usb/misc/ftdi-elan.c b/drivers/usb/misc/ftdi-elan.c
index 8a3d9c0c8d8b..bfb538f2cac1 100644
--- a/drivers/usb/misc/ftdi-elan.c
+++ b/drivers/usb/misc/ftdi-elan.c
@@ -61,7 +61,7 @@  extern struct platform_driver u132_platform_driver;
  * ftdi_module_lock exists to protect access to global variables
  *
  */
-static struct mutex ftdi_module_lock;
+static DEFINE_MUTEX(ftdi_module_lock);
 static int ftdi_instances = 0;
 static struct list_head ftdi_static_list;
 /*
@@ -2761,7 +2761,6 @@  static int __init ftdi_elan_init(void)
 {
 	int result;
 	pr_info("driver %s\n", ftdi_elan_driver.name);
-	mutex_init(&ftdi_module_lock);
 	INIT_LIST_HEAD(&ftdi_static_list);
 	result = usb_register(&ftdi_elan_driver);
 	if (result) {