regmap: mmio: replace return 0 with break in switch statement
authorColin Ian King <colin.i.king@gmail.com>
Thu, 22 Sep 2022 08:04:45 +0000 (09:04 +0100)
committerMark Brown <broonie@kernel.org>
Thu, 22 Sep 2022 11:33:18 +0000 (12:33 +0100)
Variable min_stride is assigned a value that is never read, fix this by
replacing the return 0 with a break statement. This also makes the case
statement consistent with the other cases in the switch statement.

Signed-off-by: Colin Ian King <colin.i.king@gmail.com>
Link: https://lore.kernel.org/r/20220922080445.818020-1-colin.i.king@gmail.com
Signed-off-by: Mark Brown <broonie@kernel.org>
drivers/base/regmap/regmap-mmio.c

index 8bd8d42..3ccdd86 100644 (file)
@@ -48,7 +48,7 @@ static int regmap_mmio_get_min_stride(size_t val_bits)
        case 8:
                /* The core treats 0 as 1 */
                min_stride = 0;
-               return 0;
+               break;
        case 16:
                min_stride = 2;
                break;