subject:"\[PATCH 2\/3\] leds\: lp5521\/5523\: Remove duplicate mutex"

Re: [PATCH 2/3] leds: lp5521/5523: Remove duplicate mutex

2013-12-02 Thread Milo Kim

Hi Bryan, On 12/03/2013 04:40 AM, Bryan Wu wrote: On Wed, Nov 20, 2013 at 10:14 PM, Milo Kim wrote: It can be a problem when a pattern is loaded via the firmware interface. LP55xx common driver has already locked the mutex in 'lp55xx_firmware_loaded()'. So it should be deleted. Looks like

Re: [PATCH 2/3] leds: lp5521/5523: Remove duplicate mutex

2013-12-02 Thread Bryan Wu

On Wed, Nov 20, 2013 at 10:14 PM, Milo Kim wrote: > It can be a problem when a pattern is loaded via the firmware interface. > LP55xx common driver has already locked the mutex in > 'lp55xx_firmware_loaded()'. > So it should be deleted. > Looks like lp5521_update_program_memory() will be called

Re: [PATCH 2/3] leds: lp5521/5523: Remove duplicate mutex

2013-12-02 Thread Bryan Wu

On Wed, Nov 20, 2013 at 10:14 PM, Milo Kim milo@ti.com wrote: It can be a problem when a pattern is loaded via the firmware interface. LP55xx common driver has already locked the mutex in 'lp55xx_firmware_loaded()'. So it should be deleted. Looks like lp5521_update_program_memory() will

Re: [PATCH 2/3] leds: lp5521/5523: Remove duplicate mutex

2013-12-02 Thread Milo Kim

Hi Bryan, On 12/03/2013 04:40 AM, Bryan Wu wrote: On Wed, Nov 20, 2013 at 10:14 PM, Milo Kim milo@ti.com wrote: It can be a problem when a pattern is loaded via the firmware interface. LP55xx common driver has already locked the mutex in 'lp55xx_firmware_loaded()'. So it should be deleted.