summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorLukasz Majewski <l.majewski@samsung.com>2015-01-28 16:25:22 +0100
committerEduardo Valentin <edubezval@gmail.com>2015-01-28 01:29:02 -0400
commiteccb60145415dd8e171687cd1694f50141f50d6d (patch)
tree24c062368548f869e47f9b28197a5f2934ff65fa
parentd3a5247e5332eb56d8f4833570b6d88f26709b18 (diff)
downloadop-kernel-dev-eccb60145415dd8e171687cd1694f50141f50d6d.zip
op-kernel-dev-eccb60145415dd8e171687cd1694f50141f50d6d.tar.gz
thermal: exynos: Correct sanity check at exynos_report_trigger() function
Up till now, by mistake, wrong variable was tested against being NULL. Since exynos_report_trigger() is always called with valid p pointer, it is only necessary to check if a valid thermal zone device is passed. Reported-by: Dan Carpenter <dan.carpenter@oracle.com> Signed-off-by: Lukasz Majewski <l.majewski@samsung.com> Signed-off-by: Eduardo Valentin <edubezval@gmail.com>
-rw-r--r--drivers/thermal/samsung/exynos_tmu.c4
1 files changed, 2 insertions, 2 deletions
diff --git a/drivers/thermal/samsung/exynos_tmu.c b/drivers/thermal/samsung/exynos_tmu.c
index 864eec8..b6a6e90 100644
--- a/drivers/thermal/samsung/exynos_tmu.c
+++ b/drivers/thermal/samsung/exynos_tmu.c
@@ -172,8 +172,8 @@ static void exynos_report_trigger(struct exynos_tmu_data *p)
unsigned long temp;
unsigned int i;
- if (!p) {
- pr_err("Wrong temperature configuration data\n");
+ if (!tz) {
+ pr_err("No thermal zone device defined\n");
return;
}
OpenPOWER on IntegriCloud