diff mbox series

[-next] thermal: ti-soc-thermal: Use helper function IS_ERR_OR_NULL()

Message ID 20230817014900.3094512-1-lizetao1@huawei.com (mailing list archive)
State New, archived
Headers show
Series [-next] thermal: ti-soc-thermal: Use helper function IS_ERR_OR_NULL() | expand

Commit Message

Li Zetao Aug. 17, 2023, 1:49 a.m. UTC
Use IS_ERR_OR_NULL() to detect an error pointer or a null pointer
open-coding to simplify the code.

Signed-off-by: Li Zetao <lizetao1@huawei.com>
---
 drivers/thermal/ti-soc-thermal/ti-bandgap.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comments

Daniel Lezcano Aug. 17, 2023, 7:32 a.m. UTC | #1
On 17/08/2023 03:49, Li Zetao wrote:
> Use IS_ERR_OR_NULL() to detect an error pointer or a null pointer
> open-coding to simplify the code.
> 
> Signed-off-by: Li Zetao <lizetao1@huawei.com>
> ---

Applied, thanks
diff mbox series

Patch

diff --git a/drivers/thermal/ti-soc-thermal/ti-bandgap.c b/drivers/thermal/ti-soc-thermal/ti-bandgap.c
index a1c9a1530183..0c2eb9c6e58b 100644
--- a/drivers/thermal/ti-soc-thermal/ti-bandgap.c
+++ b/drivers/thermal/ti-soc-thermal/ti-bandgap.c
@@ -314,7 +314,7 @@  int ti_bandgap_adc_to_mcelsius(struct ti_bandgap *bgp, int adc_val, int *t)
  */
 static inline int ti_bandgap_validate(struct ti_bandgap *bgp, int id)
 {
-	if (!bgp || IS_ERR(bgp)) {
+	if (IS_ERR_OR_NULL(bgp)) {
 		pr_err("%s: invalid bandgap pointer\n", __func__);
 		return -EINVAL;
 	}