summaryrefslogtreecommitdiff
path: root/drivers/iio/adc
diff options
context:
space:
mode:
authorLinus Walleij <linus.walleij@linaro.org>2016-06-29 15:08:33 +0300
committerJonathan Cameron <jic23@kernel.org>2016-06-30 22:39:43 +0300
commit90e96fdd012925871a1a21b5eaeab4b027682d6a (patch)
treec43b758c07579642dc67871bbc976ebae7f12cb4 /drivers/iio/adc
parentc5842b47b2945d6db24b4db6b2c7364e94cbc78b (diff)
downloadlinux-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 'drivers/iio/adc')
0 files changed, 0 insertions, 0 deletions