Merge branches 'thermal-intel' and 'thermal-core' into next

7 files changed, 18 insertions, 3 deletions

diff --git a/drivers/thermal/clock_cooling.c b/drivers/thermal/clock_cooling.c
index 1b4ff0f4c716..ed5dd0e88657 100644
--- a/drivers/thermal/clock_cooling.c
+++ b/drivers/thermal/clock_cooling.c
@@ -426,6 +426,7 @@ clock_cooling_register(struct device *dev, const char *clock_name)
         if (!ccdev)
                 return ERR_PTR(-ENOMEM);
 
+        mutex_init(&ccdev->lock);
         ccdev->dev = dev;
         ccdev->clk = devm_clk_get(dev, clock_name);
         if (IS_ERR(ccdev->clk))
diff --git a/drivers/thermal/fair_share.c b/drivers/thermal/fair_share.c
index 34fe36504a55..68bd1b569118 100644
--- a/drivers/thermal/fair_share.c
+++ b/drivers/thermal/fair_share.c
@@ -116,7 +116,9 @@ static int fair_share_throttle(struct thermal_zone_device *tz, int trip)
                 instance->target = get_target_state(tz, cdev, percentage,
                                                     cur_trip_level);
 
+                mutex_lock(&instance->cdev->lock);
                 instance->cdev->updated = false;
+                mutex_unlock(&instance->cdev->lock);
                 thermal_cdev_update(cdev);
         }
         return 0;
diff --git a/drivers/thermal/gov_bang_bang.c b/drivers/thermal/gov_bang_bang.c
index fc52016d4e85..bb118a152cbb 100644
--- a/drivers/thermal/gov_bang_bang.c
+++ b/drivers/thermal/gov_bang_bang.c
@@ -71,7 +71,9 @@ static void thermal_zone_trip_update(struct thermal_zone_device *tz, int trip)
                 dev_dbg(&instance->cdev->device, "target=%d\n",
                                         (int)instance->target);
 
+                mutex_lock(&instance->cdev->lock);
                 instance->cdev->updated = false; /* cdev needs update */
+                mutex_unlock(&instance->cdev->lock);
         }
 
         mutex_unlock(&tz->lock);
diff --git a/drivers/thermal/power_allocator.c b/drivers/thermal/power_allocator.c
index 2f1a863a8e15..b4d3116cfdaf 100644
--- a/drivers/thermal/power_allocator.c
+++ b/drivers/thermal/power_allocator.c
@@ -529,7 +529,9 @@ static void allow_maximum_power(struct thermal_zone_device *tz)
                         continue;
 
                 instance->target = 0;
+                mutex_lock(&instance->cdev->lock);
                 instance->cdev->updated = false;
+                mutex_unlock(&instance->cdev->lock);
                 thermal_cdev_update(instance->cdev);
         }
 }
diff --git a/drivers/thermal/step_wise.c b/drivers/thermal/step_wise.c
index ea9366ad3e6b..bcef2e7c4ec9 100644
--- a/drivers/thermal/step_wise.c
+++ b/drivers/thermal/step_wise.c
@@ -175,7 +175,9 @@ static void thermal_zone_trip_update(struct thermal_zone_device *tz, int trip)
                         update_passive_instance(tz, trip_type, -1);
 
                 instance->initialized = true;
+                mutex_lock(&instance->cdev->lock);
                 instance->cdev->updated = false; /* cdev needs update */
+                mutex_unlock(&instance->cdev->lock);
         }
 
         mutex_unlock(&tz->lock);
diff --git a/drivers/thermal/thermal_core.c b/drivers/thermal/thermal_core.c
index 5133cd1e10b7..e2fc6161dded 100644
--- a/drivers/thermal/thermal_core.c
+++ b/drivers/thermal/thermal_core.c
@@ -1093,7 +1093,9 @@ int power_actor_set_power(struct thermal_cooling_device *cdev,
                 return ret;
 
         instance->target = state;
+        mutex_lock(&cdev->lock);
         cdev->updated = false;
+        mutex_unlock(&cdev->lock);
         thermal_cdev_update(cdev);
 
         return 0;
@@ -1623,11 +1625,13 @@ void thermal_cdev_update(struct thermal_cooling_device *cdev)
         struct thermal_instance *instance;
         unsigned long target = 0;
 
+        mutex_lock(&cdev->lock);
         /* cooling device is updated*/
-        if (cdev->updated)
+        if (cdev->updated) {
+                mutex_unlock(&cdev->lock);
                 return;
+        }
 
-        mutex_lock(&cdev->lock);
         /* Make sure cdev enters the deepest cooling state */
         list_for_each_entry(instance, &cdev->thermal_instances, cdev_node) {
                 dev_dbg(&cdev->device, "zone%d->target=%lu\n",
@@ -1637,9 +1641,9 @@ void thermal_cdev_update(struct thermal_cooling_device *cdev)
                 if (instance->target > target)
                         target = instance->target;
         }
-        mutex_unlock(&cdev->lock);
         cdev->ops->set_cur_state(cdev, target);
         cdev->updated = true;
+        mutex_unlock(&cdev->lock);
         trace_cdev_update(cdev, target);
         dev_dbg(&cdev->device, "set to state %lu\n", target);
 }
diff --git a/drivers/thermal/thermal_hwmon.c b/drivers/thermal/thermal_hwmon.c
index 06fd2ed9ef9d..c41c7742903a 100644
--- a/drivers/thermal/thermal_hwmon.c
+++ b/drivers/thermal/thermal_hwmon.c
@@ -232,6 +232,7 @@ int thermal_add_hwmon_sysfs(struct thermal_zone_device *tz)
 
         return result;
 }
+EXPORT_SYMBOL_GPL(thermal_add_hwmon_sysfs);
 
 void thermal_remove_hwmon_sysfs(struct thermal_zone_device *tz)
 {
@@ -270,3 +271,4 @@ void thermal_remove_hwmon_sysfs(struct thermal_zone_device *tz)
         hwmon_device_unregister(hwmon->device);
         kfree(hwmon);
 }
+EXPORT_SYMBOL_GPL(thermal_remove_hwmon_sysfs);