iio: fetch and enable regulators unconditionally
authorCrt Mori <cmo@melexis.com>
Mon, 5 Sep 2016 09:14:48 +0000 (11:14 +0200)
committerJonathan Cameron <jic23@kernel.org>
Mon, 5 Sep 2016 20:08:40 +0000 (21:08 +0100)
commit67516074884b4bde71ca8ca4724544669935cc5d
treee23f0da0bab62ab9cbcf7e828ee2ae0e77529ec2
parent70e483487db787b152da756d4be0fef917378142
iio: fetch and enable regulators unconditionally

This patch is inspired by a comment of Jonathan Cameron on patch of
Linus Walleij commit aeb55fff3891834e07a3144159a7298a19696af8 ("iio: st_sensors: fetch and enable regulators unconditionally").

The explanation for this change is same as in that patch:
"Supplies are *not* optional (optional means that the supply is
optional in the electrical sense, not the software sense) so we need to
get the and enable them at all times.

If the device tree or board file does not define suitable regulators for
the component, it will be substituted by a dummy regulator, or, if
regulators are disabled altogether, by stubs. There is no need to use the
IS_ERR_OR_NULL() check that is considered harmful.

Reported-by: Linus Wallerij <linus.walleij@linaro.org>
Suggested-by: Jonathan Cameron <jic23@kernel.org>
Signed-off-by: Crt Mori <cmo@melexis.com>
Signed-off-by: Jonathan Cameron <jic23@kernel.org>
drivers/iio/pressure/ms5611_core.c