diff options
| author |  Uwe Kleine-König <u.kleine-koenig@pengutronix.de>
| 2022-01-23 18:52:00 +0100 |
|---|---|---|
| committer |  Mark Brown <broonie@kernel.org>
| 2022-02-09 13:00:43 +0000 |
| commit | afb0a80e63d67e957b5d0eb4ade301aff6e13c8c (patch) | |
| tree | 2e60f2d853c906b964039c5c9dd441d8c1094505 /drivers/platform/chrome/cros_ec_i2c.c | |
| parent | tpm: st33zp24: Make st33zp24_remove() a void function (diff) | |
| download | linux-afb0a80e63d67e957b5d0eb4ade301aff6e13c8c.tar.gz linux-afb0a80e63d67e957b5d0eb4ade301aff6e13c8c.tar.bz2 linux-afb0a80e63d67e957b5d0eb4ade301aff6e13c8c.zip | |
platform/chrome: cros_ec: Make cros_ec_unregister() return void
Up to now cros_ec_unregister() 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, platform and spi remove callbacks is
ignored anyway.
Reviewed-by: Guenter Roeck <groeck@chromium.org>
Link: https://lore.kernel.org/r/20211020071753.wltjslmimb6wtlp5@pengutronix.de
Signed-off-by: Uwe Kleine-König <u.kleine-koenig@pengutronix.de>
Acked-by: Lee Jones <lee.jones@linaro.org>
Link: https://lore.kernel.org/r/20220123175201.34839-5-u.kleine-koenig@pengutronix.de
Signed-off-by: Mark Brown <broonie@kernel.org>
Diffstat (limited to 'drivers/platform/chrome/cros_ec_i2c.c')
| -rw-r--r-- | drivers/platform/chrome/cros_ec_i2c.c | 4 |
1 files changed, 3 insertions, 1 deletions
diff --git a/drivers/platform/chrome/cros_ec_i2c.c b/drivers/platform/chrome/cros_ec_i2c.c index 30c8938c27d5..22feb0fd4ce7 100644 --- a/drivers/platform/chrome/cros_ec_i2c.c +++ b/drivers/platform/chrome/cros_ec_i2c.c @@ -313,7 +313,9 @@ static int cros_ec_i2c_remove(struct i2c_client *client) { struct cros_ec_device *ec_dev = i2c_get_clientdata(client); - return cros_ec_unregister(ec_dev); + cros_ec_unregister(ec_dev); + + return 0; } #ifdef CONFIG_PM_SLEEP |