diff options
| author | Dan Carpenter <dan.carpenter@linaro.org> | 2025-03-05 18:01:57 +0300 |
|---|---|---|
| committer | David Sterba <dsterba@suse.com> | 2025-03-18 20:35:49 +0100 |
| commit | c01b7114b809bbd3804d8a5b7e397f87f4ce52cb (patch) | |
| tree | 243ffc9e10fa4b88f03bbe6b8641515fc2320384 /fs/btrfs/dev-replace.c | |
| parent | 477a7a9c1f46b179ccc5eb2bbf18a0966055a464 (diff) | |
btrfs: return a literal instead of a variable in btrfs_init_dev_replace()
This is just a small clean up, it doesn't change how the code works.
Originally this code had a goto so we needed to set "ret = 0;" but now
it returns directly and so we can simplify it a bit by doing a
"return 0;" and removing the assignment.
Reviewed-by: Anand Jain <anand.jain@oracle.com>
Signed-off-by: Dan Carpenter <dan.carpenter@linaro.org>
Reviewed-by: David Sterba <dsterba@suse.com>
Signed-off-by: David Sterba <dsterba@suse.com>
Diffstat (limited to 'fs/btrfs/dev-replace.c')
| -rw-r--r-- | fs/btrfs/dev-replace.c | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/fs/btrfs/dev-replace.c b/fs/btrfs/dev-replace.c index 1a82e88ec5c1..53d7d85cb4be 100644 --- a/fs/btrfs/dev-replace.c +++ b/fs/btrfs/dev-replace.c @@ -103,7 +103,6 @@ no_valid_dev_replace_entry_found: "found replace target device without a valid replace item"); return -EUCLEAN; } - ret = 0; dev_replace->replace_state = BTRFS_IOCTL_DEV_REPLACE_STATE_NEVER_STARTED; dev_replace->cont_reading_from_srcdev_mode = @@ -120,7 +119,7 @@ no_valid_dev_replace_entry_found: dev_replace->tgtdev = NULL; dev_replace->is_valid = 0; dev_replace->item_needs_writeback = 0; - return ret; + return 0; } slot = path->slots[0]; eb = path->nodes[0]; |