regulator: lp87565: Fix missing break in switch statement
authorColin Ian King <colin.king@canonical.com>
Thu, 27 Jun 2019 13:16:39 +0000 (14:16 +0100)
committerLee Jones <lee.jones@linaro.org>
Tue, 2 Jul 2019 10:41:23 +0000 (11:41 +0100)
Currently the LP87565_DEVICE_TYPE_LP87561_Q1 case does not have a
break statement, causing it to fall through to a dev_err message.
Fix this by adding in the missing break statement.

Addresses-Coverity: ("Missing break in switch")
Fixes: 7ee63bd74750 ("regulator: lp87565: Add 4-phase lp87561 regulator support")
Signed-off-by: Colin Ian King <colin.king@canonical.com>
Reviewed-by: Keerthy <j-keerthy@ti.com>
Signed-off-by: Lee Jones <lee.jones@linaro.org>
drivers/regulator/lp87565-regulator.c

index af00d1f..142b6f1 100644 (file)
@@ -183,6 +183,7 @@ static int lp87565_regulator_probe(struct platform_device *pdev)
        case LP87565_DEVICE_TYPE_LP87561_Q1:
                min_idx = LP87565_BUCK_3210;
                max_idx = LP87565_BUCK_3210;
+               break;
        default:
                dev_err(lp87565->dev, "Invalid lp config %d\n",
                        lp87565->dev_type);