Re: [PATCH v2 2/2] regulator: core: Replace direct ops->disable usage

From: Mark Brown
Date: Thu Feb 20 2014 - 07:52:03 EST


On Thu, Feb 20, 2014 at 12:07:52PM +0100, Markus Pargmann wrote:

> --- a/drivers/regulator/core.c
> +++ b/drivers/regulator/core.c
> @@ -1908,8 +1908,6 @@ static int _regulator_do_disable(struct regulator_dev *rdev)
>
> trace_regulator_disable_complete(rdev_get_name(rdev));
>
> - _notifier_call_chain(rdev, REGULATOR_EVENT_DISABLE,
> - NULL);
> return 0;
> }
>
> @@ -1933,6 +1931,8 @@ static int _regulator_disable(struct regulator_dev *rdev)
> rdev_err(rdev, "failed to disable\n");
> return ret;
> }
> + _notifier_call_chain(rdev, REGULATOR_EVENT_DISABLE,
> + NULL);
> }
>
> rdev->use_count = 0;

This change doesn't seem obviously related?

Attachment: signature.asc
Description: Digital signature