Messages in this thread | | | Date | Thu, 18 Jul 2013 16:26:22 +0530 | Subject | Re: [PATCH] regulator: core: Use the power efficient workqueue for delayed powerdown | From | Viresh Kumar <> |
| |
On 18 July 2013 16:23, Mark Brown <broonie@kernel.org> wrote: > From: Mark Brown <broonie@linaro.org> > > There is no need to use a normal per-CPU workqueue for delayed power downs > as they're not timing or performance critical and waking up a core for them > would defeat some of the point. > > Signed-off-by: Mark Brown <broonie@linaro.org> > --- > drivers/regulator/core.c | 5 +++-- > 1 file changed, 3 insertions(+), 2 deletions(-) > > diff --git a/drivers/regulator/core.c b/drivers/regulator/core.c > index b6efead..6ef476f 100644 > --- a/drivers/regulator/core.c > +++ b/drivers/regulator/core.c > @@ -1891,8 +1891,9 @@ int regulator_disable_deferred(struct regulator *regulator, int ms) > rdev->deferred_disables++; > mutex_unlock(&rdev->mutex); > > - ret = schedule_delayed_work(&rdev->disable_work, > - msecs_to_jiffies(ms)); > + ret = queue_delayed_work(system_power_efficient_wq, > + &rdev->disable_work, > + msecs_to_jiffies(ms)); > if (ret < 0) > return ret; > else
Cool!! People have started to use this interface now :)
Reviewed-by: Viresh Kumar <viresh.kumar@linaro.org>
| |