Re: [PATCH v3 3/3] leds: leds-pwm: Defer led_pwm_set() if PWM can sleep

2013-03-15 Thread Bryan Wu
On Fri, Mar 15, 2013 at 1:09 PM, Florian Vaussard wrote: > Hi, > > > On 03/15/2013 07:59 PM, Thierry Reding wrote: >>> >>> >>> Hi Florian and Thierry, >>> >>> Sorry, guys. My bad! I'm using GMAIL filtering my emails. This email >>> goes into my arm-linux mail list archive and I totally missed it.

Re: [PATCH v3 3/3] leds: leds-pwm: Defer led_pwm_set() if PWM can sleep

2013-03-15 Thread Florian Vaussard
Hi, On 03/15/2013 07:59 PM, Thierry Reding wrote: Hi Florian and Thierry, Sorry, guys. My bad! I'm using GMAIL filtering my emails. This email goes into my arm-linux mail list archive and I totally missed it. I can merge this patch into my for-next branch, but it will be in 3.10 not 3.9-rcX.

Re: [PATCH v3 3/3] leds: leds-pwm: Defer led_pwm_set() if PWM can sleep

2013-03-15 Thread Thierry Reding
On Fri, Mar 15, 2013 at 10:08:57AM -0700, Bryan Wu wrote: > On Tue, Mar 12, 2013 at 3:06 AM, Florian Vaussard > wrote: > > Hi Bryan, > > > > > > > > Bryan, I assume that you'll be taking this? It doesn't apply cleanly to > > my tree, probably because of Peter's recent changes that you

Re: [PATCH v3 3/3] leds: leds-pwm: Defer led_pwm_set() if PWM can sleep

2013-03-15 Thread Bryan Wu
On Tue, Mar 12, 2013 at 3:06 AM, Florian Vaussard wrote: > Hi Bryan, > > > > Bryan, I assume that you'll be taking this? It doesn't apply cleanly to > my tree, probably because of Peter's recent changes that you took > through your tree and Florian based his patches on top of that.

Re: [PATCH v3 3/3] leds: leds-pwm: Defer led_pwm_set() if PWM can sleep

2013-03-12 Thread Florian Vaussard
Hi Bryan, Bryan, I assume that you'll be taking this? It doesn't apply cleanly to my tree, probably because of Peter's recent changes that you took through your tree and Florian based his patches on top of that. The conflict resolution should be trivial, though. It seems that this patch was

Re: [PATCH v3 3/3] leds: leds-pwm: Defer led_pwm_set() if PWM can sleep

2013-03-05 Thread Florian Vaussard
On 03/04/2013 11:58 AM, Thierry Reding wrote: On Mon, Mar 04, 2013 at 11:44:22AM +0100, Florian Vaussard wrote: Hi Bryan, Richard, On 01/30/2013 09:17 AM, Thierry Reding wrote: On Mon, Jan 28, 2013 at 03:00:59PM +0100, Florian Vaussard wrote: Call to led_pwm_set() can happen inside atomic c

Re: [PATCH v3 3/3] leds: leds-pwm: Defer led_pwm_set() if PWM can sleep

2013-03-04 Thread Thierry Reding
On Mon, Mar 04, 2013 at 11:44:22AM +0100, Florian Vaussard wrote: > Hi Bryan, Richard, > > >On 01/30/2013 09:17 AM, Thierry Reding wrote: > >>On Mon, Jan 28, 2013 at 03:00:59PM +0100, Florian Vaussard wrote: > >>>Call to led_pwm_set() can happen inside atomic context, like triggers. > >>>If the PW

Re: [PATCH v3 3/3] leds: leds-pwm: Defer led_pwm_set() if PWM can sleep

2013-03-04 Thread Florian Vaussard
Hi Bryan, Richard, On 01/30/2013 09:17 AM, Thierry Reding wrote: On Mon, Jan 28, 2013 at 03:00:59PM +0100, Florian Vaussard wrote: Call to led_pwm_set() can happen inside atomic context, like triggers. If the PWM call can sleep, defer using a worker. Cc: Peter Ujfalusi Signed-off-by: Florian

Re: [PATCH v3 3/3] leds: leds-pwm: Defer led_pwm_set() if PWM can sleep

2013-02-27 Thread Florian Vaussard
Hi Bryan, On 01/30/2013 09:17 AM, Thierry Reding wrote: On Mon, Jan 28, 2013 at 03:00:59PM +0100, Florian Vaussard wrote: Call to led_pwm_set() can happen inside atomic context, like triggers. If the PWM call can sleep, defer using a worker. Cc: Peter Ujfalusi Signed-off-by: Florian Vaussard

Re: [PATCH v3 3/3] leds: leds-pwm: Defer led_pwm_set() if PWM can sleep

2013-02-04 Thread Florian Vaussard
On 01/30/2013 09:17 AM, Thierry Reding wrote: On Mon, Jan 28, 2013 at 03:00:59PM +0100, Florian Vaussard wrote: Call to led_pwm_set() can happen inside atomic context, like triggers. If the PWM call can sleep, defer using a worker. Cc: Peter Ujfalusi Signed-off-by: Florian Vaussard --- driv

Re: [PATCH v3 3/3] leds: leds-pwm: Defer led_pwm_set() if PWM can sleep

2013-01-30 Thread Thierry Reding
On Mon, Jan 28, 2013 at 03:00:59PM +0100, Florian Vaussard wrote: > Call to led_pwm_set() can happen inside atomic context, like triggers. > If the PWM call can sleep, defer using a worker. > > Cc: Peter Ujfalusi > Signed-off-by: Florian Vaussard > --- > drivers/leds/leds-pwm.c | 50 +

Re: [PATCH v3 3/3] leds: leds-pwm: Defer led_pwm_set() if PWM can sleep

2013-01-29 Thread Peter Ujfalusi
On 01/28/2013 03:00 PM, Florian Vaussard wrote: > Call to led_pwm_set() can happen inside atomic context, like triggers. > If the PWM call can sleep, defer using a worker. > > Cc: Peter Ujfalusi > Signed-off-by: Florian Vaussard Reviewed-by: Peter Ujfalusi > --- > drivers/leds/leds-pwm.c |