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 06:29:02 +0100 |
commit | eccb60145415dd8e171687cd1694f50141f50d6d (patch) | |
tree | 24c062368548f869e47f9b28197a5f2934ff65fa /drivers/thermal | |
parent | thermal: Kconfig: Remove config for not used EXYNOS_THERMAL_CORE (diff) | |
download | linux-eccb60145415dd8e171687cd1694f50141f50d6d.tar.xz linux-eccb60145415dd8e171687cd1694f50141f50d6d.zip |
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>
Diffstat (limited to 'drivers/thermal')
-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 864eec8f5164..b6a6e90ba340 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; } |