[dm-devel] [PATCH 35/39] dm: avoid void function return statements

[Mike Snitzer]

From: Heinz Mauelshagen <hei...@redhat.com>

Signed-off-by: Heinz Mauelshagen <hei...@redhat.com>
Signed-off-by: Mike Snitzer <snit...@kernel.org>
---
 drivers/md/dm-io-rewind.c          | 2 --
 drivers/md/dm-log-userspace-base.c | 8 --------
 drivers/md/dm-zoned-target.c       | 1 -
 3 files changed, 11 deletions(-)

diff --git a/drivers/md/dm-io-rewind.c b/drivers/md/dm-io-rewind.c
index efd76412262a..7766a1c94762 100644
--- a/drivers/md/dm-io-rewind.c
+++ b/drivers/md/dm-io-rewind.c
@@ -68,7 +68,6 @@ static void dm_bio_integrity_rewind(struct bio *bio, unsigned 
int bytes_done)
 static inline void dm_bio_integrity_rewind(struct bio *bio,
                                           unsigned int bytes_done)
 {
-       return;
 }
 
 #endif
@@ -104,7 +103,6 @@ static void dm_bio_crypt_rewind(struct bio *bio, unsigned 
int bytes)
 
 static inline void dm_bio_crypt_rewind(struct bio *bio, unsigned int bytes)
 {
-       return;
 }
 
 #endif
diff --git a/drivers/md/dm-log-userspace-base.c 
b/drivers/md/dm-log-userspace-base.c
index d96a9651a462..5551bbc23cdb 100644
--- a/drivers/md/dm-log-userspace-base.c
+++ b/drivers/md/dm-log-userspace-base.c
@@ -346,8 +346,6 @@ static void userspace_dtr(struct dm_dirty_log *log)
 
        kfree(lc->usr_argv_str);
        kfree(lc);
-
-       return;
 }
 
 static int userspace_presuspend(struct dm_dirty_log *log)
@@ -661,8 +659,6 @@ static void userspace_mark_region(struct dm_dirty_log *log, 
region_t region)
        fe->region = region;
        list_add(&fe->list, &lc->mark_list);
        spin_unlock_irqrestore(&lc->flush_lock, flags);
-
-       return;
 }
 
 /*
@@ -698,8 +694,6 @@ static void userspace_clear_region(struct dm_dirty_log 
*log, region_t region)
        fe->region = region;
        list_add(&fe->list, &lc->clear_list);
        spin_unlock_irqrestore(&lc->flush_lock, flags);
-
-       return;
 }
 
 /*
@@ -756,7 +750,6 @@ static void userspace_set_region_sync(struct dm_dirty_log 
*log,
         * It would be nice to be able to report failures.
         * However, it is easy enough to detect and resolve.
         */
-       return;
 }
 
 /*
@@ -927,7 +920,6 @@ static void __exit userspace_dirty_log_exit(void)
        kmem_cache_destroy(_flush_entry_cache);
 
        DMINFO("version " DM_LOG_USERSPACE_VSN " unloaded");
-       return;
 }
 
 module_init(userspace_dirty_log_init);
diff --git a/drivers/md/dm-zoned-target.c b/drivers/md/dm-zoned-target.c
index 95b132b52f33..ad4764dcd013 100644
--- a/drivers/md/dm-zoned-target.c
+++ b/drivers/md/dm-zoned-target.c
@@ -1119,7 +1119,6 @@ static void dmz_status(struct dm_target *ti, 
status_type_t type,
                *result = '\0';
                break;
        }
-       return;
 }
 
 static int dmz_message(struct dm_target *ti, unsigned int argc, char **argv,
-- 
2.37.0 (Apple Git-136)

--
dm-devel mailing list
dm-devel@redhat.com
https://listman.redhat.com/mailman/listinfo/dm-devel