summaryrefslogtreecommitdiff
path: root/drivers/iio/dac/ad5593r.c
diff options
context:
space:
mode:
authorUwe Kleine-König <u.kleine-koenig@pengutronix.de>2021-10-13 23:32:17 +0300
committerJonathan Cameron <Jonathan.Cameron@huawei.com>2021-10-19 10:30:45 +0300
commit72ba4505622ddc765f227fb24270fc202193aab5 (patch)
tree958dd79e81147e03559cedfbbf2e27abf7a05229 /drivers/iio/dac/ad5593r.c
parent1f10848f18555980e7379532d1859245d2d847c9 (diff)
downloadlinux-72ba4505622ddc765f227fb24270fc202193aab5.tar.xz
iio: dac: ad5592r: Make ad5592r_remove() return void
Up to now ad5592r_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> Reviewed-by: Alexandru Ardelean <ardeleanalex@gmail.com> Link: https://lore.kernel.org/r/20211013203223.2694577-10-u.kleine-koenig@pengutronix.de Signed-off-by: Jonathan Cameron <Jonathan.Cameron@huawei.com>
Diffstat (limited to 'drivers/iio/dac/ad5593r.c')
-rw-r--r--drivers/iio/dac/ad5593r.c4
1 files changed, 3 insertions, 1 deletions
diff --git a/drivers/iio/dac/ad5593r.c b/drivers/iio/dac/ad5593r.c
index 5b4df36fdc2a..64dd7a0bddf7 100644
--- a/drivers/iio/dac/ad5593r.c
+++ b/drivers/iio/dac/ad5593r.c
@@ -99,7 +99,9 @@ static int ad5593r_i2c_probe(struct i2c_client *i2c,
static int ad5593r_i2c_remove(struct i2c_client *i2c)
{
- return ad5592r_remove(&i2c->dev);
+ ad5592r_remove(&i2c->dev);
+
+ return 0;
}
static const struct i2c_device_id ad5593r_i2c_ids[] = {