summary refs log tree commit diff
path: root/drivers/vme
diff options
context:
space:
mode:
authorUwe Kleine-König <uwe@kleine-koenig.org>2021-01-27 22:23:29 +0100
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>2021-02-09 12:15:07 +0100
commit2adc75fba3289455b9c4349dd6b95cfb7167b7ea (patch)
tree643d8131f4ea58cd70d853079dafae84b840c550 /drivers/vme
parent5f6805327982d1fd45355730e9d1adda616b995b (diff)
downloadlinux-2adc75fba3289455b9c4349dd6b95cfb7167b7ea.tar.gz
vme: make remove callback return void
The driver core ignores the return value of struct bus_type::remove()
because there is only little that can be done. To simplify the quest to
make this function return void, let struct vme_driver::remove return void,
too. There is only a single vme driver and it already returns 0
unconditionally in .remove().

Also fix the bus remove function to always return 0.

Signed-off-by: Uwe Kleine-König <uwe@kleine-koenig.org>
Link: https://lore.kernel.org/r/20210127212329.98517-1-uwe@kleine-koenig.org
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Diffstat (limited to 'drivers/vme')
-rw-r--r--drivers/vme/vme.c4
1 files changed, 2 insertions, 2 deletions
diff --git a/drivers/vme/vme.c b/drivers/vme/vme.c
index 54d7963c1078..1b15afea28ee 100644
--- a/drivers/vme/vme.c
+++ b/drivers/vme/vme.c
@@ -1997,9 +1997,9 @@ static int vme_bus_remove(struct device *dev)
 
 	driver = dev->platform_data;
 	if (driver->remove)
-		return driver->remove(vdev);
+		driver->remove(vdev);
 
-	return -ENODEV;
+	return 0;
 }
 
 struct bus_type vme_bus_type = {