block_device_operations->release() should return void

The value passed is 0 in all but "it can never happen" cases (and those
only in a couple of drivers) *and* it would've been lost on the way
out anyway, even if something tried to pass something meaningful.
Just don't bother.

Signed-off-by: Al Viro <viro@zeniv.linux.org.uk>

1 files changed, 7 insertions, 11 deletions

diff --git a/drivers/s390/block/dasd.c b/drivers/s390/block/dasd.c
index 82758cbb220b..4361d9772c42 100644
--- a/drivers/s390/block/dasd.c
+++ b/drivers/s390/block/dasd.c
@@ -2997,18 +2997,14 @@ unlock:
         return rc;
 }
 
-static int dasd_release(struct gendisk *disk, fmode_t mode)
+static void dasd_release(struct gendisk *disk, fmode_t mode)
 {
-        struct dasd_device *base;
-
-        base = dasd_device_from_gendisk(disk);
-        if (!base)
-                return -ENODEV;
-
-        atomic_dec(&base->block->open_count);
-        module_put(base->discipline->owner);
-        dasd_put_device(base);
-        return 0;
+        struct dasd_device *base = dasd_device_from_gendisk(disk);
+        if (base) {
+                atomic_dec(&base->block->open_count);
+                module_put(base->discipline->owner);
+                dasd_put_device(base);
+        }
 }
 
 /*