diff options
| author | Alex Frid <afrid@nvidia.com> | 2011-07-20 15:04:06 -0700 |
|---|---|---|
| committer | Dan Willemsen <dwillemsen@nvidia.com> | 2011-11-30 21:47:49 -0800 |
| commit | 727df62ef3a7e0c53d32ba9b7a361a68defa00fc (patch) | |
| tree | 5dbc43dd58d5ce8e047de136759437a48ee71e98 /drivers/regulator/core.c | |
| parent | 6e9b68133112f094d1b82b9ce854853b84480895 (diff) | |
regulator: Expand regulator notifier events
Add regulator enable notifications (not sent by the current code).
Add voltage notifications sent only if regulator output is actually
changed (keep intact existing voltage change notification that is
sent unconditionally - even if voltage change was not allowed by
constraints, or no actual change since regulator is disabled):
PRE_ENABLE Regulator is to be enabled
POST_ENABLE Regulator was enabled
OUT_PRECHANGE Regulator is enabled and its voltage is to be changed
OUT_POSTCHANGE Regulator is enabled and its voltage was changed
Bug 853132
Original-Change-Id: I2759ada1c5d1e43f746f6c866f9b3c203b68aed6
Reviewed-on: http://git-master/r/42262
Tested-by: Aleksandr Frid <afrid@nvidia.com>
Reviewed-by: Scott Williams <scwilliams@nvidia.com>
Reviewed-by: Laxman Dewangan <ldewangan@nvidia.com>
Reviewed-by: Karan Jhavar <kjhavar@nvidia.com>
Reviewed-by: Yu-Huan Hsu <yhsu@nvidia.com>
Rebase-Id: Rdac9b19d1f999e889aae8e2eb72f9b60057dd05a
Diffstat (limited to 'drivers/regulator/core.c')
| -rw-r--r-- | drivers/regulator/core.c | 12 |
1 files changed, 12 insertions, 0 deletions
diff --git a/drivers/regulator/core.c b/drivers/regulator/core.c index 3cc55d69b67e..f3247f7ccefc 100644 --- a/drivers/regulator/core.c +++ b/drivers/regulator/core.c @@ -1371,6 +1371,8 @@ static int _regulator_enable(struct regulator_dev *rdev) } trace_regulator_enable(rdev_get_name(rdev)); + _notifier_call_chain( + rdev, REGULATOR_EVENT_PRE_ENABLE, NULL); /* Allow the regulator to ramp; it would be useful * to extend this for bulk operations so that the @@ -1388,6 +1390,8 @@ static int _regulator_enable(struct regulator_dev *rdev) udelay(delay); } + _notifier_call_chain( + rdev, REGULATOR_EVENT_POST_ENABLE, NULL); trace_regulator_enable_complete(rdev_get_name(rdev)); } else if (ret < 0) { @@ -1679,6 +1683,10 @@ static int _regulator_do_set_voltage(struct regulator_dev *rdev, min_uV += rdev->constraints->uV_offset; max_uV += rdev->constraints->uV_offset; + if (_regulator_is_enabled(rdev)) + _notifier_call_chain(rdev, REGULATOR_EVENT_OUT_PRECHANGE, + NULL); + if (rdev->desc->ops->set_voltage) { ret = rdev->desc->ops->set_voltage(rdev, min_uV, max_uV, &selector); @@ -1746,6 +1754,10 @@ static int _regulator_do_set_voltage(struct regulator_dev *rdev, _notifier_call_chain(rdev, REGULATOR_EVENT_VOLTAGE_CHANGE, NULL); + if (_regulator_is_enabled(rdev)) + _notifier_call_chain(rdev, REGULATOR_EVENT_OUT_POSTCHANGE, + NULL); + trace_regulator_set_voltage_complete(rdev_get_name(rdev), selector); return ret; |