On 17/02/16 17:52, Gregor Boirie wrote: > From: Grégor Boirie <gregor.boirie@xxxxxxxxxx> > > Add support for an optional regulator which, if found into device-tree, > will power on device at probing time. > The regulator is declared into ms5611 DTS entry as a "vdd-supply" property. > > Signed-off-by: Gregor Boirie <gregor.boirie@xxxxxxxxxx> This usually only gets done when it first matters to someone. Guessing it matters to you! Applied to the togreg branch of iio.git Thanks, Jonathan > --- > drivers/iio/pressure/ms5611_core.c | 13 +++++++++++++ > 1 file changed, 13 insertions(+) > > diff --git a/drivers/iio/pressure/ms5611_core.c b/drivers/iio/pressure/ms5611_core.c > index acd8e37..992ad8d 100644 > --- a/drivers/iio/pressure/ms5611_core.c > +++ b/drivers/iio/pressure/ms5611_core.c > @@ -16,6 +16,7 @@ > #include <linux/module.h> > #include <linux/iio/iio.h> > #include <linux/delay.h> > +#include <linux/regulator/consumer.h> > > #include <linux/iio/buffer.h> > #include <linux/iio/triggered_buffer.h> > @@ -290,6 +291,18 @@ static const struct iio_info ms5611_info = { > static int ms5611_init(struct iio_dev *indio_dev) > { > int ret; > + struct regulator *vdd = devm_regulator_get(indio_dev->dev.parent, > + "vdd"); > + > + /* Enable attached regulator if any. */ > + if (!IS_ERR(vdd)) { > + ret = regulator_enable(vdd); > + if (ret) { > + dev_err(indio_dev->dev.parent, > + "failed to enable Vdd supply: %d\n", ret); > + return ret; > + } > + } > > ret = ms5611_reset(indio_dev); > if (ret < 0) > -- To unsubscribe from this list: send the line "unsubscribe linux-iio" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html