iio: dac: ad5380: Don't set error code to st->vref
authorAxel Lin <axel.lin@ingics.com>
Fri, 14 Dec 2012 07:48:00 +0000 (07:48 +0000)
committerJonathan Cameron <jic23@kernel.org>
Thu, 27 Dec 2012 11:00:08 +0000 (11:00 +0000)
regulator_get_voltage() may return negative error code.
Add error checking to avoid setting error code to st->vref_uv.

Signed-off-by: Axel Lin <axel.lin@ingics.com>
Acked-by: Lars-Peter Clausen <lars@metafoo.de>
Signed-off-by: Jonathan Cameron <jic23@kernel.org>
drivers/iio/dac/ad5380.c

index 14991ac55f2680ded5543eb76d436ef61cc87832..4aca1895ce52cf735fe9691ab0467947cb5218da 100644 (file)
@@ -406,7 +406,11 @@ static int __devinit ad5380_probe(struct device *dev, struct regmap *regmap,
                        goto error_free_reg;
                }
 
-               st->vref = regulator_get_voltage(st->vref_reg);
+               ret = regulator_get_voltage(st->vref_reg);
+               if (ret < 0)
+                       goto error_disable_reg;
+
+               st->vref = ret;
        } else {
                st->vref = st->chip_info->int_vref;
                ctrl |= AD5380_CTRL_INT_VREF_EN;