diff options
| author | Uwe Kleine-König <u.kleine-koenig@baylibre.com> | 2025-12-10 11:31:37 +0300 |
|---|---|---|
| committer | Thierry Reding <treding@nvidia.com> | 2026-01-13 14:25:38 +0300 |
| commit | ba3588410cedb1696cfe56ebefcc4401c6d0bb36 (patch) | |
| tree | 05633b127c3c182cc91d707205931328dafd7a13 /drivers/staging | |
| parent | 4a768c544f64eaa2fc7cfa91e46f43aa4aad0c40 (diff) | |
| download | linux-ba3588410cedb1696cfe56ebefcc4401c6d0bb36.tar.xz | |
host1x: Make remove callback return void
The return value of struct device_driver::remove is ignored by the core
(see device_remove() in drivers/base/dd.c). So it doesn't make sense to
let the host1x remove callback return an int just to ignore it later.
So make the callback return void. All current implementors return 0, so
they are easily converted.
Signed-off-by: Uwe Kleine-König <u.kleine-koenig@baylibre.com>
Tested-by: Luca Ceresoli <luca.ceresoli@bootlin.com> # tegra20 tegra-video
Reviewed-by: Luca Ceresoli <luca.ceresoli@bootlin.com>
Signed-off-by: Thierry Reding <treding@nvidia.com>
Link: https://patch.msgid.link/d364fd4ec043d36ee12e46eaef98c57658884f63.1765355236.git.u.kleine-koenig@baylibre.com
Diffstat (limited to 'drivers/staging')
| -rw-r--r-- | drivers/staging/media/tegra-video/video.c | 4 |
1 files changed, 1 insertions, 3 deletions
diff --git a/drivers/staging/media/tegra-video/video.c b/drivers/staging/media/tegra-video/video.c index 074ad0dc56ca..68783d5ffeb1 100644 --- a/drivers/staging/media/tegra-video/video.c +++ b/drivers/staging/media/tegra-video/video.c @@ -107,7 +107,7 @@ cleanup: return ret; } -static int host1x_video_remove(struct host1x_device *dev) +static void host1x_video_remove(struct host1x_device *dev) { struct tegra_video_device *vid = dev_get_drvdata(&dev->dev); @@ -118,8 +118,6 @@ static int host1x_video_remove(struct host1x_device *dev) /* This calls v4l2_dev release callback on last reference */ v4l2_device_put(&vid->v4l2_dev); - - return 0; } static const struct of_device_id host1x_video_subdevs[] = { |