regulator: pfuze100: add condition check for 'ramp_delay = 0'

Checking for 'ramp_delay = 0' to avoid Coverity warning report.

Signed-off-by: Robin Gong <yibin.gong@nxp.com>
Link: https://lore.kernel.org/r/1592920870-12693-1-git-send-email-yibin.gong@nxp.com
Signed-off-by: Mark Brown <broonie@kernel.org>

diff --git a/drivers/regulator/pfuze100-regulator.c b/drivers/regulator/pfuze100-regulator.c
index 4c8e8b4722872a0bfc47755dd2b89f0635b4067b..7e8ba9246167c1cc71d0f6dee4e6c0b70f0036e2 100644 (file)
--- a/drivers/regulator/pfuze100-regulator.c
+++ b/drivers/regulator/pfuze100-regulator.c
@@ -128,7 +128,7 @@ static int pfuze100_set_ramp_delay(struct regulator_dev *rdev, int ramp_delay)
        struct pfuze_chip *pfuze100 = rdev_get_drvdata(rdev);
        int id = rdev_get_id(rdev);
        bool reg_has_ramp_delay;
-       unsigned int ramp_bits;
+       unsigned int ramp_bits = 0;
        int ret;
 
        switch (pfuze100->chip_id) {
@@ -149,8 +149,11 @@ static int pfuze100_set_ramp_delay(struct regulator_dev *rdev, int ramp_delay)
        }
 
        if (reg_has_ramp_delay) {
-               ramp_delay = 12500 / ramp_delay;
-               ramp_bits = (ramp_delay >> 1) - (ramp_delay >> 3);
+               if (ramp_delay > 0) {
+                       ramp_delay = 12500 / ramp_delay;
+                       ramp_bits = (ramp_delay >> 1) - (ramp_delay >> 3);
+               }
+
                ret = regmap_update_bits(pfuze100->regmap,
                                         rdev->desc->vsel_reg + 4,
                                         0xc0, ramp_bits << 6);