[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>

2 files changed, 6 insertions, 26 deletions

diff --git a/drivers/w1/slaves/w1_ds2433.c b/drivers/w1/slaves/w1_ds2433.c
index ddd01e6fc2c9..2ac238f1480e 100644
--- a/drivers/w1/slaves/w1_ds2433.c
+++ b/drivers/w1/slaves/w1_ds2433.c
@@ -105,11 +105,7 @@ static ssize_t w1_f23_read_bin(struct kobject *kobj, char *buf, loff_t off,
         if ((count = w1_f23_fix_count(off, count, W1_EEPROM_SIZE)) == 0)
                 return 0;
 
-        atomic_inc(&sl->refcnt);
-        if (down_interruptible(&sl->master->mutex)) {
-                count = 0;
-                goto out_dec;
-        }
+        mutex_lock(&sl->master->mutex);
 
 #ifdef CONFIG_W1_F23_CRC
 
@@ -140,9 +136,7 @@ static ssize_t w1_f23_read_bin(struct kobject *kobj, char *buf, loff_t off,
 #endif  /* CONFIG_W1_F23_CRC */
 
 out_up:
-        up(&sl->master->mutex);
-out_dec:
-        atomic_dec(&sl->refcnt);
+        mutex_unlock(&sl->master->mutex);
 
         return count;
 }
@@ -231,11 +225,7 @@ static ssize_t w1_f23_write_bin(struct kobject *kobj, char *buf, loff_t off,
         }
 #endif  /* CONFIG_W1_F23_CRC */
 
-        atomic_inc(&sl->refcnt);
-        if (down_interruptible(&sl->master->mutex)) {
-                count = 0;
-                goto out_dec;
-        }
+        mutex_lock(&sl->master->mutex);
 
         /* Can only write data to one page at a time */
         idx = 0;
@@ -253,9 +243,7 @@ static ssize_t w1_f23_write_bin(struct kobject *kobj, char *buf, loff_t off,
         }
 
 out_up:
-        up(&sl->master->mutex);
-out_dec:
-        atomic_dec(&sl->refcnt);
+        mutex_unlock(&sl->master->mutex);
 
         return count;
 }
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;
 }