From: Amit Daniel Kachhap <amit.daniel@samsung.com>
To: linux-pm@vger.kernel.org
Cc: Zhang Rui <rui.zhang@intel.com>,
linux-samsung-soc@vger.kernel.org, linux-kernel@vger.kernel.org,
amit.kachhap@gmail.com, Kukjin Kim <kgene.kim@samsung.com>,
Eduardo Valentin <eduardo.valentin@ti.com>
Subject: [PATCH V2 14/20] thermal: exynos: Make the zone handling dependent on trip count
Date: Fri, 26 Apr 2013 16:07:45 +0530 [thread overview]
Message-ID: <1366972671-9227-15-git-send-email-amit.daniel@samsung.com> (raw)
In-Reply-To: <1366972671-9227-1-git-send-email-amit.daniel@samsung.com>
This code simplifies the zone handling to use the trip count passed
by the TMU driver. This also helps in adding more zone support.
Acked-by: Kukjin Kim <kgene.kim@samsung.com>
Signed-off-by: Amit Daniel Kachhap <amit.daniel@samsung.com>
---
drivers/thermal/samsung/exynos_thermal_common.c | 55 ++++++++++++-----------
drivers/thermal/samsung/exynos_thermal_common.h | 2 -
2 files changed, 29 insertions(+), 28 deletions(-)
diff --git a/drivers/thermal/samsung/exynos_thermal_common.c b/drivers/thermal/samsung/exynos_thermal_common.c
index b7ca8a4..b0dc63e 100644
--- a/drivers/thermal/samsung/exynos_thermal_common.c
+++ b/drivers/thermal/samsung/exynos_thermal_common.c
@@ -83,17 +83,16 @@ static int exynos_set_mode(struct thermal_zone_device *thermal,
static int exynos_get_trip_type(struct thermal_zone_device *thermal, int trip,
enum thermal_trip_type *type)
{
- switch (GET_ZONE(trip)) {
- case MONITOR_ZONE:
- case WARN_ZONE:
- *type = THERMAL_TRIP_ACTIVE;
- break;
- case PANIC_ZONE:
- *type = THERMAL_TRIP_CRITICAL;
- break;
- default:
+ struct exynos_thermal_zone *th_zone = thermal->devdata;
+ int max_trip = th_zone->sensor_conf->trip_data.trip_count;
+
+ if (trip < 0 || trip >= max_trip)
return -EINVAL;
- }
+ else if (trip == (max_trip - 1))
+ *type = THERMAL_TRIP_CRITICAL;
+ else
+ *type = THERMAL_TRIP_ACTIVE;
+
return 0;
}
@@ -102,8 +101,9 @@ static int exynos_get_trip_temp(struct thermal_zone_device *thermal, int trip,
unsigned long *temp)
{
struct exynos_thermal_zone *th_zone = thermal->devdata;
+ int max_trip = th_zone->sensor_conf->trip_data.trip_count;
- if (trip < GET_TRIP(MONITOR_ZONE) || trip > GET_TRIP(PANIC_ZONE))
+ if (trip < 0 || trip >= max_trip)
return -EINVAL;
*temp = th_zone->sensor_conf->trip_data.trip_val[trip];
@@ -117,10 +117,10 @@ static int exynos_get_trip_temp(struct thermal_zone_device *thermal, int trip,
static int exynos_get_crit_temp(struct thermal_zone_device *thermal,
unsigned long *temp)
{
- int ret;
- /* Panic zone */
- ret = exynos_get_trip_temp(thermal, GET_TRIP(PANIC_ZONE), temp);
- return ret;
+ struct exynos_thermal_zone *th_zone = thermal->devdata;
+ int max_trip = th_zone->sensor_conf->trip_data.trip_count;
+ /* Get the temp of highest trip*/
+ return exynos_get_trip_temp(thermal, max_trip - 1, temp);
}
/* Bind callback functions for thermal zone */
@@ -345,19 +345,22 @@ int exynos_register_thermal(struct thermal_sensor_conf *sensor_conf)
return -ENOMEM;
th_zone->sensor_conf = sensor_conf;
- cpumask_set_cpu(0, &mask_val);
- th_zone->cool_dev[0] = cpufreq_cooling_register(&mask_val);
- if (IS_ERR(th_zone->cool_dev[0])) {
- pr_err("Failed to register cpufreq cooling device\n");
- ret = -EINVAL;
- goto err_unregister;
+ if (sensor_conf->cooling_data.freq_clip_count > 0) {
+ cpumask_set_cpu(0, &mask_val);
+ th_zone->cool_dev[0] = cpufreq_cooling_register(&mask_val);
+ if (IS_ERR(th_zone->cool_dev[0])) {
+ pr_err("Failed to register cpufreq cooling device\n");
+ ret = -EINVAL;
+ goto err_unregister;
+ }
+ th_zone->cool_dev_size++;
}
- th_zone->cool_dev_size++;
- th_zone->therm_dev = thermal_zone_device_register(sensor_conf->name,
- EXYNOS_ZONE_COUNT, 0, th_zone, &exynos_dev_ops, NULL, 0,
- sensor_conf->trip_data.trigger_falling ?
- 0 : IDLE_INTERVAL);
+ th_zone->therm_dev = thermal_zone_device_register(
+ sensor_conf->name, sensor_conf->trip_data.trip_count,
+ 0, th_zone, &exynos_dev_ops, NULL, 0,
+ sensor_conf->trip_data.trigger_falling ? 0 :
+ IDLE_INTERVAL);
if (IS_ERR(th_zone->therm_dev)) {
pr_err("Failed to register thermal zone device\n");
diff --git a/drivers/thermal/samsung/exynos_thermal_common.h b/drivers/thermal/samsung/exynos_thermal_common.h
index 878120b..58c3a88 100644
--- a/drivers/thermal/samsung/exynos_thermal_common.h
+++ b/drivers/thermal/samsung/exynos_thermal_common.h
@@ -42,8 +42,6 @@
#define GET_ZONE(trip) (trip + 2)
#define GET_TRIP(zone) (zone - 2)
-#define EXYNOS_ZONE_COUNT 3
-
/**
* struct freq_clip_table
* @freq_clip_max: maximum frequency allowed for this cooling state.
--
1.7.1
next prev parent reply other threads:[~2013-04-26 10:37 UTC|newest]
Thread overview: 31+ messages / expand[flat|nested] mbox.gz Atom feed top
2013-04-26 10:37 [PATCH V2 0/20] thermal: exynos: Add thermal driver for exynos5440 Amit Daniel Kachhap
2013-04-26 10:37 ` [PATCH V2 01/20] thermal: exynos: Moving exynos thermal files into samsung directory Amit Daniel Kachhap
2013-05-06 15:05 ` Zhang Rui
2013-05-07 10:21 ` Tomasz Figa
2013-05-07 13:18 ` amit daniel kachhap
2013-05-07 13:36 ` Tomasz Figa
2013-05-07 13:12 ` amit daniel kachhap
2013-04-26 10:37 ` [PATCH V2 02/20] thermal: exynos: Bifurcate exynos thermal common and tmu controller code Amit Daniel Kachhap
2013-05-06 15:16 ` Zhang Rui
2013-05-07 13:32 ` amit daniel kachhap
2013-04-26 10:37 ` [PATCH V2 03/20] thermal: exynos: Rename exynos_thermal.c to exynos_tmu.c Amit Daniel Kachhap
2013-05-06 15:18 ` Zhang Rui
2013-04-26 10:37 ` [PATCH V2 04/20] thermal: exynos: Move exynos_thermal.h from include/* to driver/* folder Amit Daniel Kachhap
2013-05-06 15:19 ` Zhang Rui
2013-04-26 10:37 ` [PATCH V2 05/20] thermal: exynos: Bifurcate exynos tmu driver and configuration data Amit Daniel Kachhap
2013-05-06 15:28 ` Zhang Rui
2013-04-26 10:37 ` [PATCH V2 06/20] thermal: exynos: Add missing definations and code cleanup Amit Daniel Kachhap
2013-04-26 10:37 ` [PATCH V2 07/20] thermal: exynos: Add extra entries in the tmu platform data Amit Daniel Kachhap
2013-04-26 10:37 ` [PATCH V2 08/20] thermal: exynos: Support thermal tripping Amit Daniel Kachhap
2013-04-26 10:37 ` [PATCH V2 09/20] thermal: exynos: Move register definitions from driver file to data file Amit Daniel Kachhap
2013-04-26 10:37 ` [PATCH V2 10/20] thermal: exynos: Fix to clear only the generated interrupts Amit Daniel Kachhap
2013-04-26 10:37 ` [PATCH V2 11/20] thermal: exynos: Add support for instance based register/unregister Amit Daniel Kachhap
2013-04-26 10:37 ` [PATCH V2 12/20] thermal: exynos: Modify private_data to appropriate name driver_data Amit Daniel Kachhap
2013-04-26 10:37 ` [PATCH V2 13/20] thermal: exynos: Return success even if no cooling data supplied Amit Daniel Kachhap
2013-04-26 10:37 ` Amit Daniel Kachhap [this message]
2013-04-26 10:37 ` [PATCH V2 15/20] thermal: exynos: Add support to handle many instances of TMU Amit Daniel Kachhap
2013-04-26 10:37 ` [PATCH V2 16/20] thermal: exynos: Add features to check instead of SOC type Amit Daniel Kachhap
2013-04-26 10:37 ` [PATCH V2 17/20] ARM: dts: thermal: exynos4: Add documentation for Exynos SoC thermal bindings Amit Daniel Kachhap
2013-04-26 10:37 ` [PATCH V2 18/20] thermal: exynos: Add support for exynos5440 TMU sensor Amit Daniel Kachhap
2013-04-26 10:37 ` [PATCH V2 19/20] Documentation: thermal: Explain the exynos thermal driver model Amit Daniel Kachhap
2013-04-26 10:37 ` [PATCH V2 20/20] ARM: dts: Add device tree node for exynos5440 TMU controller Amit Daniel Kachhap
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=1366972671-9227-15-git-send-email-amit.daniel@samsung.com \
--to=amit.daniel@samsung.com \
--cc=amit.kachhap@gmail.com \
--cc=eduardo.valentin@ti.com \
--cc=kgene.kim@samsung.com \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-pm@vger.kernel.org \
--cc=linux-samsung-soc@vger.kernel.org \
--cc=rui.zhang@intel.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).