diff options
author | Lukasz Majewski <l.majewski@samsung.com> | 2015-01-28 16:25:22 +0100 |
---|---|---|
committer | Eduardo Valentin <edubezval@gmail.com> | 2015-01-28 01:29:02 -0400 |
commit | eccb60145415dd8e171687cd1694f50141f50d6d (patch) | |
tree | 24c062368548f869e47f9b28197a5f2934ff65fa | |
parent | d3a5247e5332eb56d8f4833570b6d88f26709b18 (diff) | |
download | op-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.c | 4 |
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; } |