diff options
| author | Uwe Kleine-König <u.kleine-koenig@pengutronix.de> | 2022-03-31 14:31:32 +0100 |
|---|---|---|
| committer | Mauro Carvalho Chehab <mchehab@kernel.org> | 2022-04-24 08:26:45 +0100 |
| commit | b4657e00115d0c87970a2386338b70fa95ef4ac6 (patch) | |
| tree | fd4f064e8d8bc8a4f1a075a1d9fcd33115958555 /drivers/media/i2c/dw9714.c | |
| parent | 02276e18defa2fccf16413b44440277d98c2b1ea (diff) | |
media: i2c: dw9714: Return zero in remove callback
The only effect of returning an error code in an i2c remove callback is
that the i2c core emits a generic warning and still removes the device.
So even if disabling the regulator fails it's sensible to further cleanup
and then return zero to only emit a single error message.
This patch is a preparation for making i2c remove callbacks return void.
Signed-off-by: Uwe Kleine-König <u.kleine-koenig@pengutronix.de>
Signed-off-by: Sakari Ailus <sakari.ailus@linux.intel.com>
Signed-off-by: Mauro Carvalho Chehab <mchehab@kernel.org>
Diffstat (limited to 'drivers/media/i2c/dw9714.c')
| -rw-r--r-- | drivers/media/i2c/dw9714.c | 1 |
1 files changed, 0 insertions, 1 deletions
diff --git a/drivers/media/i2c/dw9714.c b/drivers/media/i2c/dw9714.c index 8c5797ba57d4..206d74338b9c 100644 --- a/drivers/media/i2c/dw9714.c +++ b/drivers/media/i2c/dw9714.c @@ -202,7 +202,6 @@ static int dw9714_remove(struct i2c_client *client) if (ret) { dev_err(&client->dev, "Failed to disable vcc: %d\n", ret); - return ret; } } pm_runtime_set_suspended(&client->dev); |