summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorLukasz Majewski <l.majewski@samsung.com>2013-10-09 08:29:50 +0200
committerEduardo Valentin <eduardo.valentin@ti.com>2013-10-15 10:10:42 -0400
commitacecdb3cf42576431ba17d82fcc7b221e63a8d8c (patch)
treef8d96f602edd0eb02d8079fa3d098d2858ae3523
parent22356f447ceb8d97a4885792e7d9e4607f712e1b (diff)
downloadop-kernel-dev-acecdb3cf42576431ba17d82fcc7b221e63a8d8c.zip
op-kernel-dev-acecdb3cf42576431ba17d82fcc7b221e63a8d8c.tar.gz
thermal: exynos: Remove check for thermal device pointer at exynos_report_trigger()
The commit 4de0bdaa9677d11406c9becb70c60887c957e1f0 ("thermal: exynos: Add support for instance based register/unregister") broke check for presence of therm_dev at global thermal zone in exynos_report_trigger(). The resulting wrong test prevents thermal_zone_device_update() call, which calls handlers for situation when trip points are passed. Such behavior prevents thermal driver from proper reaction (when TMU interrupt is raised) in a situation when overheating is detected at TMU hardware. It turns out, that after exynos thermal subsystem redesign (at v3.12) this check is not needed, since it is not possible to register thermal zone without valid thermal device. Signed-off-by: Lukasz Majewski <l.majewski@samsung.com> Reviewed-by: Bartlomiej Zolnierkiewicz <b.zolnierkie@samsung.com> Reviewed-by: Tomasz Figa <t.figa@samsung.com> Signed-off-by: Eduardo Valentin <eduardo.valentin@ti.com>
-rw-r--r--drivers/thermal/samsung/exynos_thermal_common.c2
1 files changed, 0 insertions, 2 deletions
diff --git a/drivers/thermal/samsung/exynos_thermal_common.c b/drivers/thermal/samsung/exynos_thermal_common.c
index f10a6ad..c2301da 100644
--- a/drivers/thermal/samsung/exynos_thermal_common.c
+++ b/drivers/thermal/samsung/exynos_thermal_common.c
@@ -310,8 +310,6 @@ void exynos_report_trigger(struct thermal_sensor_conf *conf)
}
th_zone = conf->pzone_data;
- if (th_zone->therm_dev)
- return;
if (th_zone->bind == false) {
for (i = 0; i < th_zone->cool_dev_size; i++) {
OpenPOWER on IntegriCloud