Re: [PATCH v2 2/2] leds-lp5523: set the brightness to 0 forcely on removing the driver

2012-08-22 Thread Bryan Wu
On Wed, Aug 22, 2012 at 3:32 PM, Kim, Milo wrote: > Turning off the brightness of each channel is required > when removing the driver. > > So use flush_work() rather than cancel_work_sync() to execute > remaining brightness works. > Thanks, merged to my for-next -Bryan > Signed-off-by:

[PATCH v2 2/2] leds-lp5523: set the brightness to 0 forcely on removing the driver

2012-08-22 Thread Kim, Milo
Turning off the brightness of each channel is required when removing the driver. So use flush_work() rather than cancel_work_sync() to execute remaining brightness works. Signed-off-by: Milo(Woogyom) Kim --- drivers/leds/leds-lp5523.c |4 ++-- 1 files changed, 2 insertions(+), 2

[PATCH v2 2/2] leds-lp5523: set the brightness to 0 forcely on removing the driver

2012-08-22 Thread Kim, Milo
Turning off the brightness of each channel is required when removing the driver. So use flush_work() rather than cancel_work_sync() to execute remaining brightness works. Signed-off-by: Milo(Woogyom) Kim milo@ti.com --- drivers/leds/leds-lp5523.c |4 ++-- 1 files changed, 2

Re: [PATCH v2 2/2] leds-lp5523: set the brightness to 0 forcely on removing the driver

2012-08-22 Thread Bryan Wu
On Wed, Aug 22, 2012 at 3:32 PM, Kim, Milo milo@ti.com wrote: Turning off the brightness of each channel is required when removing the driver. So use flush_work() rather than cancel_work_sync() to execute remaining brightness works. Thanks, merged to my for-next -Bryan Signed-off-by: