[RFC,1/1] thermal: introduce thermal_zone_lookup_temperature helper function
diff mbox

Message ID 1357142024-8324-1-git-send-email-eduardo.valentin@ti.com
State Rejected
Delegated to: Zhang Rui
Headers show

Commit Message

Eduardo Valentin Jan. 2, 2013, 3:53 p.m. UTC
This patch adds a helper function to get temperature of
a thermal zone, based on the zone type name.

It will perform a zone name lookup and return the last
sensor temperature reading. In case the zone is not found
or if the required parameters are invalid, it will return
the corresponding error code.


With this, we could write platform code to fetch several temperature
from several sensors and derive "summary" temperature for a virtual
thermal zone.

Signed-off-by: Eduardo Valentin <eduardo.valentin@ti.com>
---
 drivers/thermal/thermal_sys.c |   30 ++++++++++++++++++++++++++++++
 include/linux/thermal.h       |    1 +
 2 files changed, 31 insertions(+), 0 deletions(-)

Patch
diff mbox

diff --git a/drivers/thermal/thermal_sys.c b/drivers/thermal/thermal_sys.c
index a24b010..4269571 100644
--- a/drivers/thermal/thermal_sys.c
+++ b/drivers/thermal/thermal_sys.c
@@ -1703,6 +1703,36 @@  void thermal_zone_device_unregister(struct thermal_zone_device *tz)
 }
 EXPORT_SYMBOL(thermal_zone_device_unregister);
 
+/**
+ * thermal_lookup_temperature - search for a zone and returns its temperature
+ * @name: thermal zone name to fetch the temperature
+ * @temperature: pointer to store the zone temperature, in case it is found
+ *
+ * When the zone is found, updates @temperature and returns 0. Returns
+ * -EINVAL in case of wrong parameters and -ENODEV in case the zone is not found
+ */
+int thermal_zone_lookup_temperature(const char *name, int *temperature)
+{
+	struct thermal_zone_device *pos = NULL;
+	bool found = false;
+
+	if (!name || !temperature)
+		return -EINVAL;
+
+	mutex_lock(&thermal_list_lock);
+	list_for_each_entry(pos, &thermal_tz_list, node)
+		if (!strcmp(pos->type, name)) {
+			found = true;
+			break;
+		}
+	if (found)
+		*temperature = pos->last_temperature;
+	mutex_unlock(&thermal_list_lock);
+
+	return found ? 0 : -ENODEV;
+}
+EXPORT_SYMBOL(thermal_zone_lookup_temperature);
+
 #ifdef CONFIG_NET
 static struct genl_family thermal_event_genl_family = {
 	.id = GENL_ID_GENERATE,
diff --git a/include/linux/thermal.h b/include/linux/thermal.h
index e85ac70..e0cb335c 100644
--- a/include/linux/thermal.h
+++ b/include/linux/thermal.h
@@ -233,6 +233,7 @@  void thermal_zone_device_update(struct thermal_zone_device *);
 struct thermal_cooling_device *thermal_cooling_device_register(char *, void *,
 		const struct thermal_cooling_device_ops *);
 void thermal_cooling_device_unregister(struct thermal_cooling_device *);
+int thermal_zone_lookup_temperature(const char *name, int *temperature);
 
 int get_tz_trend(struct thermal_zone_device *, int);
 struct thermal_instance *get_thermal_instance(struct thermal_zone_device *,