thermal: ti-soc-thermal: remove usage of IS_ERR_OR_NULL
authorEduardo Valentin <eduardo.valentin@ti.com>
Wed, 29 May 2013 15:07:43 +0000 (15:07 +0000)
committerChanho Park <chanho61.park@samsung.com>
Tue, 18 Nov 2014 02:45:10 +0000 (11:45 +0900)
commitbc143ad74f4c7f220d6913a6e5835660a1e0b318
tree4580532c7170adb3f3cdd1a4f7dac65bfeea8773
parent27167d0af7a2cc769763e735747b0c9833c8e947
thermal: ti-soc-thermal: remove usage of IS_ERR_OR_NULL

This patch changes the driver to avoid the usage of IS_ERR_OR_NULL()
macro. This macro can lead to dangerous results, like returning
success (0) during a failure scenario (NULL pointer handling).

For this reason this patch is changing the driver after
revisiting the code. These are the cases:
i. For cases in which IS_ERR_OR_NULL() is used for checking
return values of functions that returns either PTR_ERR()
or a valid pointer, it has been translated to IS_ERR() check only.
ii. For cases that a NULL check is still needed, it has been
translated to if (!ptr || IS_ERR(ptr)).

Cc: Zhang Rui <rui.zhang@intel.com>
Cc: linux-pm@vger.kernel.org
Cc: linux-kernel@vger.kernel.org
Signed-off-by: Eduardo Valentin <eduardo.valentin@ti.com>
Signed-off-by: Zhang Rui <rui.zhang@intel.com>
drivers/thermal/ti-soc-thermal/ti-bandgap.c
drivers/thermal/ti-soc-thermal/ti-thermal-common.c