diff options
author | Linus Walleij <linus.walleij@linaro.org> | 2016-06-29 15:08:33 +0300 |
---|---|---|
committer | Jonathan Cameron <jic23@kernel.org> | 2016-06-30 22:39:43 +0300 |
commit | 90e96fdd012925871a1a21b5eaeab4b027682d6a (patch) | |
tree | c43b758c07579642dc67871bbc976ebae7f12cb4 /arch/arm | |
parent | c5842b47b2945d6db24b4db6b2c7364e94cbc78b (diff) | |
download | linux-90e96fdd012925871a1a21b5eaeab4b027682d6a.tar.xz |
iio: magn: ak8975: fix regulator usage
IS_ERR_OR_NULL() should never be used with regulators because
a NULL pointer may be a perfectly valid dummy regulator
We should always succeed to fetch and enable a regulator, but
it may be a dummy. That is fine, so bail out for any real
errors or probe deferrals
Include the error code in the warning print so we know what
kind of problem we're dealing with (for example it is nice to
see if it is a probe deferral).
As we will bail out of probe if the regulator is erroneous,
just issue regulator_disable() on the poweroff path: it will
succeed.
Cc: Mark Brown <broonie@kernel.org>
Cc: Lars-Peter Clausen Lars-Peter Clausen <lars@metafoo.de>
Signed-off-by: Linus Walleij <linus.walleij@linaro.org>
Signed-off-by: Jonathan Cameron <jic23@kernel.org>
Diffstat (limited to 'arch/arm')
0 files changed, 0 insertions, 0 deletions