diff options
| author | Evgeniy Polyakov | 2006-04-03 10:04:27 +0200 |
|---|---|---|
| committer | Greg Kroah-Hartman | 2006-06-22 20:22:50 +0200 |
| commit | abd52a13206e02537ca1dc08fc5438c7d27bdbf1 (patch) | |
| tree | efe0ff89898aad10600d392ac727dcea9e7af322 /drivers/w1/slaves/w1_therm.c | |
| parent | [PATCH] w1: Make w1 connector notifications depend on connector. (diff) | |
| download | kernel-qcow2-linux-abd52a13206e02537ca1dc08fc5438c7d27bdbf1.tar.gz kernel-qcow2-linux-abd52a13206e02537ca1dc08fc5438c7d27bdbf1.tar.xz kernel-qcow2-linux-abd52a13206e02537ca1dc08fc5438c7d27bdbf1.zip | |
[PATCH] w1: Use mutexes instead of semaphores.
Use mutexes instead of semaphores.
Patch tested on x86_64 and i386 with test bus master driver.
Signed-off-by: Evgeniy Polyakov <johnpol@2ka.mipt.ru>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
Diffstat (limited to 'drivers/w1/slaves/w1_therm.c')
| -rw-r--r-- | drivers/w1/slaves/w1_therm.c | 12 |
1 files changed, 2 insertions, 10 deletions
diff --git a/drivers/w1/slaves/w1_therm.c b/drivers/w1/slaves/w1_therm.c index 44afdffe3c8b..5372cfcbd054 100644 --- a/drivers/w1/slaves/w1_therm.c +++ b/drivers/w1/slaves/w1_therm.c @@ -165,12 +165,7 @@ static ssize_t w1_therm_read_bin(struct kobject *kobj, char *buf, loff_t off, si u8 rom[9], crc, verdict; int i, max_trying = 10; - atomic_inc(&sl->refcnt); - smp_mb__after_atomic_inc(); - if (down_interruptible(&sl->master->mutex)) { - count = 0; - goto out_dec; - } + mutex_lock(&sl->master->mutex); if (off > W1_SLAVE_DATA_SIZE) { count = 0; @@ -233,10 +228,7 @@ static ssize_t w1_therm_read_bin(struct kobject *kobj, char *buf, loff_t off, si count += sprintf(buf + count, "t=%d\n", w1_convert_temp(rom, sl->family->fid)); out: - up(&dev->mutex); -out_dec: - smp_mb__before_atomic_inc(); - atomic_dec(&sl->refcnt); + mutex_unlock(&dev->mutex); return count; } |