[PATCH 1/2] extcon: gpio: Use power efficient workqueue for debounce

[PATCH 1/2] extcon: gpio: Use power efficient workqueue for debounce

[Mark Brown]

From: Mark Brown <broonie@linaro.org>

The debounce timeout is generally quite long and the work not performance
critical so allow the scheduler to run the work anywhere rather than in
the normal per-CPU workqueue.

Signed-off-by: Mark Brown <broonie@linaro.org>
---
 drivers/extcon/extcon-gpio.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/extcon/extcon-gpio.c b/drivers/extcon/extcon-gpio.c
index 02bec32..f874c30 100644
--- a/drivers/extcon/extcon-gpio.c
+++ b/drivers/extcon/extcon-gpio.c
@@ -56,7 +56,7 @@ static irqreturn_t gpio_irq_handler(int irq, void *dev_id)
 {
 	struct gpio_extcon_data *extcon_data = dev_id;
 
-	schedule_delayed_work(&extcon_data->work,
+	queue_delayed_work(system_power_efficient_wq, &extcon_data->work,
 			      extcon_data->debounce_jiffies);
 	return IRQ_HANDLED;
 }
-- 
1.8.3.2

[PATCH 2/2] extcon: adc-jack: Use power efficient workqueue

[Mark Brown]

From: Mark Brown <broonie@linaro.org>

The debounce timeout is generally quite long and the work not performance
critical so allow the scheduler to run the work anywhere rather than in
the normal per-CPU workqueue.

Signed-off-by: Mark Brown <broonie@linaro.org>
---
 drivers/extcon/extcon-adc-jack.c | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/drivers/extcon/extcon-adc-jack.c b/drivers/extcon/extcon-adc-jack.c
index d0233cd..5985807 100644
--- a/drivers/extcon/extcon-adc-jack.c
+++ b/drivers/extcon/extcon-adc-jack.c
@@ -87,7 +87,8 @@ static irqreturn_t adc_jack_irq_thread(int irq, void *_data)
 {
 	struct adc_jack_data *data = _data;
 
-	schedule_delayed_work(&data->handler, data->handling_delay);
+	queue_delayed_work(system_power_efficient_wq,
+			   &data->handler, data->handling_delay);
 	return IRQ_HANDLED;
 }
 
-- 
1.8.3.2

Re: [PATCH 1/2] extcon: gpio: Use power efficient workqueue for debounce

[Viresh Kumar]

On 19 July 2013 23:17, Mark Brown <broonie@kernel.org> wrote:
> From: Mark Brown <broonie@linaro.org>
>
> The debounce timeout is generally quite long and the work not performance
> critical so allow the scheduler to run the work anywhere rather than in
> the normal per-CPU workqueue.
>
> Signed-off-by: Mark Brown <broonie@linaro.org>
> ---
>  drivers/extcon/extcon-gpio.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)

Acked-by: Viresh Kumar <viresh.kumar@linaro.org>

Re: [PATCH 2/2] extcon: adc-jack: Use power efficient workqueue

[Viresh Kumar]

On 19 July 2013 23:17, Mark Brown <broonie@kernel.org> wrote:
> From: Mark Brown <broonie@linaro.org>
>
> The debounce timeout is generally quite long and the work not performance
> critical so allow the scheduler to run the work anywhere rather than in
> the normal per-CPU workqueue.
>
> Signed-off-by: Mark Brown <broonie@linaro.org>
> ---
>  drivers/extcon/extcon-adc-jack.c | 3 ++-
>  1 file changed, 2 insertions(+), 1 deletion(-)

Acked-by: Viresh Kumar <viresh.kumar@linaro.org>

Re: [PATCH 1/2] extcon: gpio: Use power efficient workqueue for debounce

[Chanwoo Choi]

Hi Mark,

On 07/22/2013 01:27 PM, Viresh Kumar wrote:
> On 19 July 2013 23:17, Mark Brown <broonie@kernel.org> wrote:
>> From: Mark Brown <broonie@linaro.org>
>>
>> The debounce timeout is generally quite long and the work not performance
>> critical so allow the scheduler to run the work anywhere rather than in
>> the normal per-CPU workqueue.
>>
>> Signed-off-by: Mark Brown <broonie@linaro.org>
>> ---
>>  drivers/extcon/extcon-gpio.c | 2 +-
>>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> Acked-by: Viresh Kumar <viresh.kumar@linaro.org>
> 

This patchset is applied.

Thanks,
Chanwoo Choi