diff options
| author | Uwe Kleine-König <u.kleine-koenig@baylibre.com> | 2025-12-15 18:49:25 +0100 |
|---|---|---|
| committer | Vinod Koul <vkoul@kernel.org> | 2025-12-23 12:12:49 +0530 |
| commit | 478f3890709a092a97a0218f61af19ac9b725573 (patch) | |
| tree | 132dbc99fa1b056885c347960596e50e9ac27f04 /drivers/soundwire | |
| parent | 59946373755d71dbd7614ba235e0093159f80b69 (diff) | |
soundwire: Make remove function return no value
All remove functions return zero and the driver core ignores any other
returned value (just emits a warning about it being ignored). So make all
remove callbacks return void instead of an ignored int. This is in line
with most other subsystems.
Signed-off-by: Uwe Kleine-König <u.kleine-koenig@baylibre.com>
Link: https://patch.msgid.link/20251215174925.1327021-5-u.kleine-koenig@baylibre.com
Signed-off-by: Vinod Koul <vkoul@kernel.org>
Diffstat (limited to 'drivers/soundwire')
| -rw-r--r-- | drivers/soundwire/bus_type.c | 5 |
1 files changed, 2 insertions, 3 deletions
diff --git a/drivers/soundwire/bus_type.c b/drivers/soundwire/bus_type.c index 91e70cb46fb5..c40de98f1485 100644 --- a/drivers/soundwire/bus_type.c +++ b/drivers/soundwire/bus_type.c @@ -168,7 +168,6 @@ static int sdw_drv_remove(struct device *dev) { struct sdw_slave *slave = dev_to_sdw_dev(dev); struct sdw_driver *drv = drv_to_sdw_driver(dev->driver); - int ret = 0; mutex_lock(&slave->sdw_dev_lock); @@ -177,11 +176,11 @@ static int sdw_drv_remove(struct device *dev) mutex_unlock(&slave->sdw_dev_lock); if (drv->remove) - ret = drv->remove(slave); + drv->remove(slave); ida_free(&slave->bus->slave_ida, slave->index); - return ret; + return 0; } static void sdw_drv_shutdown(struct device *dev) |