diff options
| author | Uwe Kleine-König <u.kleine-koenig@pengutronix.de> | 2021-10-11 15:27:43 +0200 |
|---|---|---|
| committer | Guenter Roeck <linux@roeck-us.net> | 2021-10-12 07:22:42 -0700 |
| commit | beee7890c36320fe08d9cce82afa1db848360bfb (patch) | |
| tree | c41e8751ac8b6b2d9f1799bbdc8ada257306731e /drivers/hwmon/adt7410.c | |
| parent | 8a0c75a1c3990ebe8383914781cec347c1576ae6 (diff) | |
hwmon: (adt7x10) Make adt7x10_remove() return void
Up to now adt7x10_remove() returns zero unconditionally. Make it return
void instead which makes it easier to see in the callers that there is
no error to handle.
Also the return value of i2c and spi remove callbacks is ignored anyway.
Signed-off-by: Uwe Kleine-König <u.kleine-koenig@pengutronix.de>
Link: https://lore.kernel.org/r/20211011132754.2479853-3-u.kleine-koenig@pengutronix.de
Signed-off-by: Guenter Roeck <linux@roeck-us.net>
Diffstat (limited to 'drivers/hwmon/adt7410.c')
| -rw-r--r-- | drivers/hwmon/adt7410.c | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/drivers/hwmon/adt7410.c b/drivers/hwmon/adt7410.c index 9d80895d0266..973db057427b 100644 --- a/drivers/hwmon/adt7410.c +++ b/drivers/hwmon/adt7410.c @@ -50,7 +50,8 @@ static int adt7410_i2c_probe(struct i2c_client *client) static int adt7410_i2c_remove(struct i2c_client *client) { - return adt7x10_remove(&client->dev, client->irq); + adt7x10_remove(&client->dev, client->irq); + return 0; } static const struct i2c_device_id adt7410_ids[] = { |