From: Matt Ranostay <matt.ranostay@konsulko.com>
Date: Fri, 27 Jul 2018 01:47:24 +0000 (-0400)
Subject: media: video-i2c: hwmon: fix return value from amg88xx_hwmon_init()
X-Git-Tag: v4.19-rc1~137^2~147
X-Git-Url: https://asedeno.scripts.mit.edu/gitweb/?a=commitdiff_plain;h=17f330ce9e434333a98b0fbc7b6d5ddbe0cc6770;p=linux.git

media: video-i2c: hwmon: fix return value from amg88xx_hwmon_init()

PTR_ERR was making any pointer passed an error pointer, and should be
replaced with PTR_ERR_OR_ZERO which checks if is an actual error condition.

Signed-off-by: Matt Ranostay <matt.ranostay@konsulko.com>
Signed-off-by: Hans Verkuil <hans.verkuil@cisco.com>
Signed-off-by: Mauro Carvalho Chehab <mchehab+samsung@kernel.org>
---

diff --git a/drivers/media/i2c/video-i2c.c b/drivers/media/i2c/video-i2c.c
index 7dc9338502e5..06d29d8f6be8 100644
--- a/drivers/media/i2c/video-i2c.c
+++ b/drivers/media/i2c/video-i2c.c
@@ -167,7 +167,7 @@ static int amg88xx_hwmon_init(struct video_i2c_data *data)
 	void *hwmon = devm_hwmon_device_register_with_info(&data->client->dev,
 				"amg88xx", data, &amg88xx_chip_info, NULL);
 
-	return PTR_ERR(hwmon);
+	return PTR_ERR_OR_ZERO(hwmon);
 }
 #else
 #define	amg88xx_hwmon_init	NULL