diff options
| author | Arjan van de Ven <arjan@infradead.org> | 2006-03-21 01:32:53 -0500 |
|---|---|---|
| committer | David S. Miller <davem@davemloft.net> | 2006-03-21 01:32:53 -0500 |
| commit | d4ccd08cdfa8d34f4d25b62041343c52fc79385f (patch) | |
| tree | aa966641f2ce74cc0206d6be654ce5680391e164 | |
| parent | 153330618691694af64f39fb56c9de051862380e (diff) | |
[IRDA] sem2mutex: drivers/net/irda
Semaphore to mutex conversion.
The conversion was generated via scripts, and the result was validated
automatically via a script as well.
Signed-off-by: Arjan van de Ven <arjan@infradead.org>
Signed-off-by: Ingo Molnar <mingo@elte.hu>
Signed-off-by: Andrew Morton <akpm@osdl.org>
Signed-off-by: David S. Miller <davem@davemloft.net>
| -rw-r--r-- | drivers/net/irda/irtty-sir.c | 19 | ||||
| -rw-r--r-- | drivers/net/irda/sir_dongle.c | 19 |
2 files changed, 20 insertions, 18 deletions
diff --git a/drivers/net/irda/irtty-sir.c b/drivers/net/irda/irtty-sir.c index 101750bf21..6a98b7ae49 100644 --- a/drivers/net/irda/irtty-sir.c +++ b/drivers/net/irda/irtty-sir.c | |||
| @@ -33,6 +33,7 @@ | |||
| 33 | #include <asm/uaccess.h> | 33 | #include <asm/uaccess.h> |
| 34 | #include <linux/smp_lock.h> | 34 | #include <linux/smp_lock.h> |
| 35 | #include <linux/delay.h> | 35 | #include <linux/delay.h> |
| 36 | #include <linux/mutex.h> | ||
| 36 | 37 | ||
| 37 | #include <net/irda/irda.h> | 38 | #include <net/irda/irda.h> |
| 38 | #include <net/irda/irda_device.h> | 39 | #include <net/irda/irda_device.h> |
| @@ -338,7 +339,7 @@ static inline void irtty_stop_receiver(struct tty_struct *tty, int stop) | |||
| 338 | /*****************************************************************/ | 339 | /*****************************************************************/ |
| 339 | 340 | ||
| 340 | /* serialize ldisc open/close with sir_dev */ | 341 | /* serialize ldisc open/close with sir_dev */ |
| 341 | static DECLARE_MUTEX(irtty_sem); | 342 | static DEFINE_MUTEX(irtty_mutex); |
| 342 | 343 | ||
| 343 | /* notifier from sir_dev when irda% device gets opened (ifup) */ | 344 | /* notifier from sir_dev when irda% device gets opened (ifup) */ |
| 344 | 345 | ||
| @@ -348,11 +349,11 @@ static int irtty_start_dev(struct sir_dev *dev) | |||
| 348 | struct tty_struct *tty; | 349 | struct tty_struct *tty; |
| 349 | 350 | ||
| 350 | /* serialize with ldisc open/close */ | 351 | /* serialize with ldisc open/close */ |
| 351 | down(&irtty_sem); | 352 | mutex_lock(&irtty_mutex); |
| 352 | 353 | ||
| 353 | priv = dev->priv; | 354 | priv = dev->priv; |
| 354 | if (unlikely(!priv || priv->magic!=IRTTY_MAGIC)) { | 355 | if (unlikely(!priv || priv->magic!=IRTTY_MAGIC)) { |
| 355 | up(&irtty_sem); | 356 | mutex_unlock(&irtty_mutex); |
| 356 | return -ESTALE; | 357 | return -ESTALE; |
| 357 | } | 358 | } |
| 358 | 359 | ||
| @@ -363,7 +364,7 @@ static int irtty_start_dev(struct sir_dev *dev) | |||
| 363 | /* Make sure we can receive more data */ | 364 | /* Make sure we can receive more data */ |
| 364 | irtty_stop_receiver(tty, FALSE); | 365 | irtty_stop_receiver(tty, FALSE); |
| 365 | 366 | ||
| 366 | up(&irtty_sem); | 367 | mutex_unlock(&irtty_mutex); |
| 367 | return 0; | 368 | return 0; |
| 368 | } | 369 | } |
| 369 | 370 | ||
| @@ -375,11 +376,11 @@ static int irtty_stop_dev(struct sir_dev *dev) | |||
| 375 | struct tty_struct *tty; | 376 | struct tty_struct *tty; |
| 376 | 377 | ||
| 377 | /* serialize with ldisc open/close */ | 378 | /* serialize with ldisc open/close */ |
| 378 | down(&irtty_sem); | 379 | mutex_lock(&irtty_mutex); |
| 379 | 380 | ||
| 380 | priv = dev->priv; | 381 | priv = dev->priv; |
| 381 | if (unlikely(!priv || priv->magic!=IRTTY_MAGIC)) { | 382 | if (unlikely(!priv || priv->magic!=IRTTY_MAGIC)) { |
| 382 | up(&irtty_sem); | 383 | mutex_unlock(&irtty_mutex); |
| 383 | return -ESTALE; | 384 | return -ESTALE; |
| 384 | } | 385 | } |
| 385 | 386 | ||
| @@ -390,7 +391,7 @@ static int irtty_stop_dev(struct sir_dev *dev) | |||
| 390 | if (tty->driver->stop) | 391 | if (tty->driver->stop) |
| 391 | tty->driver->stop(tty); | 392 | tty->driver->stop(tty); |
| 392 | 393 | ||
| 393 | up(&irtty_sem); | 394 | mutex_unlock(&irtty_mutex); |
| 394 | 395 | ||
| 395 | return 0; | 396 | return 0; |
| 396 | } | 397 | } |
| @@ -514,13 +515,13 @@ static int irtty_open(struct tty_struct *tty) | |||
| 514 | priv->dev = dev; | 515 | priv->dev = dev; |
| 515 | 516 | ||
| 516 | /* serialize with start_dev - in case we were racing with ifup */ | 517 | /* serialize with start_dev - in case we were racing with ifup */ |
| 517 | down(&irtty_sem); | 518 | mutex_lock(&irtty_mutex); |
| 518 | 519 | ||
| 519 | dev->priv = priv; | 520 | dev->priv = priv; |
| 520 | tty->disc_data = priv; | 521 | tty->disc_data = priv; |
| 521 | tty->receive_room = 65536; | 522 | tty->receive_room = 65536; |
| 522 | 523 | ||
| 523 | up(&irtty_sem); | 524 | mutex_unlock(&irtty_mutex); |
| 524 | 525 | ||
| 525 | IRDA_DEBUG(0, "%s - %s: irda line discipline opened\n", __FUNCTION__, tty->name); | 526 | IRDA_DEBUG(0, "%s - %s: irda line discipline opened\n", __FUNCTION__, tty->name); |
| 526 | 527 | ||
diff --git a/drivers/net/irda/sir_dongle.c b/drivers/net/irda/sir_dongle.c index 8d225921ae..d7e32d9554 100644 --- a/drivers/net/irda/sir_dongle.c +++ b/drivers/net/irda/sir_dongle.c | |||
| @@ -16,6 +16,7 @@ | |||
| 16 | #include <linux/init.h> | 16 | #include <linux/init.h> |
| 17 | #include <linux/smp_lock.h> | 17 | #include <linux/smp_lock.h> |
| 18 | #include <linux/kmod.h> | 18 | #include <linux/kmod.h> |
| 19 | #include <linux/mutex.h> | ||
| 19 | 20 | ||
| 20 | #include <net/irda/irda.h> | 21 | #include <net/irda/irda.h> |
| 21 | 22 | ||
| @@ -28,7 +29,7 @@ | |||
| 28 | */ | 29 | */ |
| 29 | 30 | ||
| 30 | static LIST_HEAD(dongle_list); /* list of registered dongle drivers */ | 31 | static LIST_HEAD(dongle_list); /* list of registered dongle drivers */ |
| 31 | static DECLARE_MUTEX(dongle_list_lock); /* protects the list */ | 32 | static DEFINE_MUTEX(dongle_list_lock); /* protects the list */ |
| 32 | 33 | ||
| 33 | int irda_register_dongle(struct dongle_driver *new) | 34 | int irda_register_dongle(struct dongle_driver *new) |
| 34 | { | 35 | { |
| @@ -38,25 +39,25 @@ int irda_register_dongle(struct dongle_driver *new) | |||
| 38 | IRDA_DEBUG(0, "%s : registering dongle \"%s\" (%d).\n", | 39 | IRDA_DEBUG(0, "%s : registering dongle \"%s\" (%d).\n", |
| 39 | __FUNCTION__, new->driver_name, new->type); | 40 | __FUNCTION__, new->driver_name, new->type); |
| 40 | 41 | ||
| 41 | down(&dongle_list_lock); | 42 | mutex_lock(&dongle_list_lock); |
| 42 | list_for_each(entry, &dongle_list) { | 43 | list_for_each(entry, &dongle_list) { |
| 43 | drv = list_entry(entry, struct dongle_driver, dongle_list); | 44 | drv = list_entry(entry, struct dongle_driver, dongle_list); |
| 44 | if (new->type == drv->type) { | 45 | if (new->type == drv->type) { |
| 45 | up(&dongle_list_lock); | 46 | mutex_unlock(&dongle_list_lock); |
| 46 | return -EEXIST; | 47 | return -EEXIST; |
| 47 | } | 48 | } |
| 48 | } | 49 | } |
| 49 | list_add(&new->dongle_list, &dongle_list); | 50 | list_add(&new->dongle_list, &dongle_list); |
| 50 | up(&dongle_list_lock); | 51 | mutex_unlock(&dongle_list_lock); |
| 51 | return 0; | 52 | return 0; |
| 52 | } | 53 | } |
| 53 | EXPORT_SYMBOL(irda_register_dongle); | 54 | EXPORT_SYMBOL(irda_register_dongle); |
| 54 | 55 | ||
| 55 | int irda_unregister_dongle(struct dongle_driver *drv) | 56 | int irda_unregister_dongle(struct dongle_driver *drv) |
| 56 | { | 57 | { |
| 57 | down(&dongle_list_lock); | 58 | mutex_lock(&dongle_list_lock); |
| 58 | list_del(&drv->dongle_list); | 59 | list_del(&drv->dongle_list); |
| 59 | up(&dongle_list_lock); | 60 | mutex_unlock(&dongle_list_lock); |
| 60 | return 0; | 61 | return 0; |
| 61 | } | 62 | } |
| 62 | EXPORT_SYMBOL(irda_unregister_dongle); | 63 | EXPORT_SYMBOL(irda_unregister_dongle); |
| @@ -75,7 +76,7 @@ int sirdev_get_dongle(struct sir_dev *dev, IRDA_DONGLE type) | |||
| 75 | return -EBUSY; | 76 | return -EBUSY; |
| 76 | 77 | ||
| 77 | /* serialize access to the list of registered dongles */ | 78 | /* serialize access to the list of registered dongles */ |
| 78 | down(&dongle_list_lock); | 79 | mutex_lock(&dongle_list_lock); |
| 79 | 80 | ||
| 80 | list_for_each(entry, &dongle_list) { | 81 | list_for_each(entry, &dongle_list) { |
| 81 | drv = list_entry(entry, struct dongle_driver, dongle_list); | 82 | drv = list_entry(entry, struct dongle_driver, dongle_list); |
| @@ -109,14 +110,14 @@ int sirdev_get_dongle(struct sir_dev *dev, IRDA_DONGLE type) | |||
| 109 | if (!drv->open || (err=drv->open(dev))!=0) | 110 | if (!drv->open || (err=drv->open(dev))!=0) |
| 110 | goto out_reject; /* failed to open driver */ | 111 | goto out_reject; /* failed to open driver */ |
| 111 | 112 | ||
| 112 | up(&dongle_list_lock); | 113 | mutex_unlock(&dongle_list_lock); |
| 113 | return 0; | 114 | return 0; |
| 114 | 115 | ||
| 115 | out_reject: | 116 | out_reject: |
| 116 | dev->dongle_drv = NULL; | 117 | dev->dongle_drv = NULL; |
| 117 | module_put(drv->owner); | 118 | module_put(drv->owner); |
| 118 | out_unlock: | 119 | out_unlock: |
| 119 | up(&dongle_list_lock); | 120 | mutex_unlock(&dongle_list_lock); |
| 120 | return err; | 121 | return err; |
| 121 | } | 122 | } |
| 122 | 123 | ||
