diff options
author | Michal Orzel <michalorzel.eng@gmail.com> | 2022-04-23 13:38:09 +0200 |
---|---|---|
committer | Jens Axboe <axboe@kernel.dk> | 2022-04-23 07:15:26 -0600 |
commit | 834726828b47c08e84df02f975e30c5c65bf316b (patch) | |
tree | bf888f639b906e309205f3f2a8dfdc158c50c1e9 /block/partitions/acorn.c | |
parent | 7ab89db979017255c2163880de5c63d8c9726aef (diff) | |
download | lwn-834726828b47c08e84df02f975e30c5c65bf316b.tar.gz lwn-834726828b47c08e84df02f975e30c5c65bf316b.zip |
block/partitions/acorn: Remove redundant assignments
Get rid of redundant assignments to a variable slot from function
adfspart_check_ADFS. It is being assigned a value that is never read
as we are breaking out from the switch case and the function ends.
Reported by clang-tidy [deadcode.DeadStores]
Signed-off-by: Michal Orzel <michalorzel.eng@gmail.com>
Link: https://lore.kernel.org/r/20220423113811.13335-3-michalorzel.eng@gmail.com
Signed-off-by: Jens Axboe <axboe@kernel.dk>
Diffstat (limited to 'block/partitions/acorn.c')
-rw-r--r-- | block/partitions/acorn.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/block/partitions/acorn.c b/block/partitions/acorn.c index 2c381c694c57..d2fc122d7426 100644 --- a/block/partitions/acorn.c +++ b/block/partitions/acorn.c @@ -282,13 +282,13 @@ int adfspart_check_ADFS(struct parsed_partitions *state) #ifdef CONFIG_ACORN_PARTITION_RISCIX case PARTITION_RISCIX_SCSI: case PARTITION_RISCIX_MFM: - slot = riscix_partition(state, start_sect, slot, + riscix_partition(state, start_sect, slot, nr_sects); break; #endif case PARTITION_LINUX: - slot = linux_partition(state, start_sect, slot, + linux_partition(state, start_sect, slot, nr_sects); break; } |