[PATCH v3 2/3] staging: iio: ad7780: update voltage on read
jic23 at kernel.org
Sat Nov 3 17:21:25 UTC 2018
On Sat, 3 Nov 2018 13:06:19 -0300
Renato Lui Geh <renatogeh at gmail.com> wrote:
> On Thu, 1 Nov 2018 15:20:55 +0000
> "Ardelean, Alexandru" <alexandru.Ardelean at analog.com> wrote:
> > This looks wrong.
> > I admit this was done in the same way in the probe function, but that looks
> > a bit wrong as well.
> > Typically, the return value of `regulator_get_voltage()` would get checked
> > with:
> > ret = regulator_get_voltage(st->reg);
> > if (ret < 0)
> > return ret;
> > *val = ret / 1000;
> > So, negative values are errors and zero & positive values are valid voltage
> > values.
> I see. So -EINVAL is only used when sent the wrong info type?
yes. I actually misread what was there and thought we were just talking
about using a ret variable rather than returning the error via your
Definitely want to pass on the error from regulator_get_voltage as
it may have more meaning than a simple -EINVAL.
More information about the devel