regulator: lock supply in regulator enable
authorMattias Wallin <mattias.wallin@stericsson.com>
Thu, 4 Nov 2010 10:01:31 +0000 (11:01 +0100)
committerLiam Girdwood <lrg@slimlogic.co.uk>
Wed, 12 Jan 2011 14:32:59 +0000 (14:32 +0000)
This patch add locks around regulator supply enable.

Signed-off-by: Mattias Wallin <mattias.wallin@stericsson.com>
Acked-by: Mark Brown <broonie@opensource.wolfsonmicro.com>
Signed-off-by: Liam Girdwood <lrg@slimlogic.co.uk>
drivers/regulator/core.c

index ba521f0..81336e2 100644 (file)
@@ -1267,17 +1267,15 @@ static int _regulator_enable(struct regulator_dev *rdev)
 {
        int ret, delay;
 
-       if (rdev->use_count == 0) {
-               /* do we need to enable the supply regulator first */
-               if (rdev->supply) {
-                       mutex_lock(&rdev->supply->mutex);
-                       ret = _regulator_enable(rdev->supply);
-                       mutex_unlock(&rdev->supply->mutex);
-                       if (ret < 0) {
-                               printk(KERN_ERR "%s: failed to enable %s: %d\n",
-                                      __func__, rdev_get_name(rdev), ret);
-                               return ret;
-                       }
+       /* do we need to enable the supply regulator first */
+       if (rdev->supply) {
+               mutex_lock(&rdev->supply->mutex);
+               ret = _regulator_enable(rdev->supply);
+               mutex_unlock(&rdev->supply->mutex);
+               if (ret < 0) {
+                       printk(KERN_ERR "%s: failed to enable %s: %d\n",
+                              __func__, rdev_get_name(rdev), ret);
+                       return ret;
                }
        }