diff options
| author |   Fuqian Huang <huangfq.daxian@gmail.com> | 2019-07-08 20:33:41 +0800 |
|---|---|---|
| committer |   Jonathan Cameron <Jonathan.Cameron@huawei.com> | 2019-07-27 20:11:14 +0100 |
| commit | d284feaf982aa881ca78cb4d81954d7240d5d2b9 (patch) | |
| tree | 09bb9c38fc2a586d859f8e485ac528add7654f14 /drivers/iio/imu/inv_mpu6050/inv_mpu_core.c | |
| parent | iio: adc: sc27xx: Replace devm_add_action() followed by failure action with devm_add_action_or_reset() (diff) | |
| download | wireguard-linux-d284feaf982aa881ca78cb4d81954d7240d5d2b9.tar.xz wireguard-linux-d284feaf982aa881ca78cb4d81954d7240d5d2b9.zip | |
iio: imu: mpu6050: Replace devm_add_action() followed by failure action with devm_add_action_or_reset()
devm_add_action_or_reset() is introduced as a helper function which
internally calls devm_add_action(). If devm_add_action() fails
then it will execute the action mentioned and return the error code.
This reduce source code size (avoid writing the action twice)
and reduce the likelyhood of bugs.
Signed-off-by: Fuqian Huang <huangfq.daxian@gmail.com>
Signed-off-by: Jonathan Cameron <Jonathan.Cameron@huawei.com>
Diffstat (limited to 'drivers/iio/imu/inv_mpu6050/inv_mpu_core.c')
| -rw-r--r-- | drivers/iio/imu/inv_mpu6050/inv_mpu_core.c | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/drivers/iio/imu/inv_mpu6050/inv_mpu_core.c b/drivers/iio/imu/inv_mpu6050/inv_mpu_core.c index 53a59957cc54..e067927317bd 100644 --- a/drivers/iio/imu/inv_mpu6050/inv_mpu_core.c +++ b/drivers/iio/imu/inv_mpu6050/inv_mpu_core.c @@ -1096,10 +1096,9 @@ int inv_mpu_core_probe(struct regmap *regmap, int irq, const char *name, if (result) return result; - result = devm_add_action(dev, inv_mpu_core_disable_regulator_action, + result = devm_add_action_or_reset(dev, inv_mpu_core_disable_regulator_action, st); if (result) { - inv_mpu_core_disable_regulator_action(st); dev_err(dev, "Failed to setup regulator cleanup action %d\n", result); return result; |