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

zhang-rui · zhang-rui · commit 1577ddfac75f · 2016-08-08T10:59:35.000+08:00
diff --git 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
@@ -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
@@ -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
@@ -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
@@ -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
@@ -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
@@ -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);

Original file line number	Diff line number	Diff line change
`@@ -116,7 +116,9 @@ static int fair_share_throttle(struct thermal_zone_device *tz, int trip)`
`116`	`116`	`instance->target = get_target_state(tz, cdev, percentage,`
`117`	`117`	`cur_trip_level);`
`118`	`118`
	`119`	`+ mutex_lock(&instance->cdev->lock);`
`119`	`120`	`instance->cdev->updated = false;`
	`121`	`+ mutex_unlock(&instance->cdev->lock);`
`120`	`122`	`thermal_cdev_update(cdev);`
`121`	`123`	`}`
`122`	`124`	`return 0;`
Original file line number	Diff line number	Diff line change
`@@ -71,7 +71,9 @@ static void thermal_zone_trip_update(struct thermal_zone_device *tz, int trip)`
`71`	`71`	`dev_dbg(&instance->cdev->device, "target=%d\n",`
`72`	`72`	`(int)instance->target);`
`73`	`73`
	`74`	`+ mutex_lock(&instance->cdev->lock);`
`74`	`75`	`instance->cdev->updated = false; /* cdev needs update */`
	`76`	`+ mutex_unlock(&instance->cdev->lock);`
`75`	`77`	`}`
`76`	`78`
`77`	`79`	`mutex_unlock(&tz->lock);`
Original file line number	Diff line number	Diff line change
`@@ -529,7 +529,9 @@ static void allow_maximum_power(struct thermal_zone_device *tz)`
`529`	`529`	`continue;`
`530`	`530`
`531`	`531`	`instance->target = 0;`
	`532`	`+ mutex_lock(&instance->cdev->lock);`
`532`	`533`	`instance->cdev->updated = false;`
	`534`	`+ mutex_unlock(&instance->cdev->lock);`
`533`	`535`	`thermal_cdev_update(instance->cdev);`
`534`	`536`	`}`
`535`	`537`	`}`
Original file line number	Diff line number	Diff line change
`@@ -175,7 +175,9 @@ static void thermal_zone_trip_update(struct thermal_zone_device *tz, int trip)`
`175`	`175`	`update_passive_instance(tz, trip_type, -1);`
`176`	`176`
`177`	`177`	`instance->initialized = true;`
	`178`	`+ mutex_lock(&instance->cdev->lock);`
`178`	`179`	`instance->cdev->updated = false; /* cdev needs update */`
	`180`	`+ mutex_unlock(&instance->cdev->lock);`
`179`	`181`	`}`
`180`	`182`
`181`	`183`	`mutex_unlock(&tz->lock);`
Original file line number	Diff line number	Diff line change
`@@ -232,6 +232,7 @@ int thermal_add_hwmon_sysfs(struct thermal_zone_device *tz)`
`232`	`232`
`233`	`233`	`return result;`
`234`	`234`	`}`
	`235`	`+EXPORT_SYMBOL_GPL(thermal_add_hwmon_sysfs);`
`235`	`236`
`236`	`237`	`void thermal_remove_hwmon_sysfs(struct thermal_zone_device *tz)`
`237`	`238`	`{`
`@@ -270,3 +271,4 @@ void thermal_remove_hwmon_sysfs(struct thermal_zone_device *tz)`
`270`	`271`	`hwmon_device_unregister(hwmon->device);`
`271`	`272`	`kfree(hwmon);`
`272`	`273`	`}`
	`274`	`+EXPORT_SYMBOL_GPL(thermal_remove_hwmon_sysfs);`