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 16:22:42 +0200 |
| commit | beee7890c36320fe08d9cce82afa1db848360bfb (patch) | |
| tree | c41e8751ac8b6b2d9f1799bbdc8ada257306731e /drivers/hwmon/adt7310.c | |
| parent | hwmon: (dell-smm) Remove unnecessary includes (diff) | |
| download | linux-beee7890c36320fe08d9cce82afa1db848360bfb.tar.xz linux-beee7890c36320fe08d9cce82afa1db848360bfb.zip | |
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/adt7310.c')
| -rw-r--r-- | drivers/hwmon/adt7310.c | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/drivers/hwmon/adt7310.c b/drivers/hwmon/adt7310.c index 9fad01191620..c40cac16af68 100644 --- a/drivers/hwmon/adt7310.c +++ b/drivers/hwmon/adt7310.c @@ -90,7 +90,8 @@ static int adt7310_spi_probe(struct spi_device *spi) static int adt7310_spi_remove(struct spi_device *spi) { - return adt7x10_remove(&spi->dev, spi->irq); + adt7x10_remove(&spi->dev, spi->irq); + return 0; } static const struct spi_device_id adt7310_id[] = { |