Re: [PATCH] pwm: Avoid double mutex lock on pwm_enable

2015-12-10 Thread Anand Moon
Hi Krzysztof On 11 December 2015 at 09:53, Krzysztof Kozlowski wrote: > On 11.12.2015 13:07, Anand Moon wrote: >> Hi Krzysztof, >> >> On 22 November 2015 at 05:43, Krzysztof Kozlowski >> wrote: >>> 2015-11-22 3:14 GMT+09:00 Anand Moon : Hi Krzysztof, > > [...] > >>> Yes, now you pasted the

Re: [PATCH] pwm: Avoid double mutex lock on pwm_enable

2015-12-10 Thread Krzysztof Kozlowski
On 11.12.2015 13:07, Anand Moon wrote: > Hi Krzysztof, > > On 22 November 2015 at 05:43, Krzysztof Kozlowski > wrote: >> 2015-11-22 3:14 GMT+09:00 Anand Moon : >>> Hi Krzysztof, [...] >> Yes, now you pasted the same warning I did... >> >> This is still the same issue. I already wrote it: >>> 1.

Re: [PATCH] pwm: Avoid double mutex lock on pwm_enable

2015-12-10 Thread Anand Moon
Hi Krzysztof, On 11 December 2015 at 09:37, Anand Moon wrote: > Hi Krzysztof, > > On 22 November 2015 at 05:43, Krzysztof Kozlowski > wrote: >> 2015-11-22 3:14 GMT+09:00 Anand Moon : >>> Hi Krzysztof, >>> >>> On 21 November 2015 at 18:37, Krzysztof Kozlowski >>> wrote: 2015-11-21 21:11 GMT

Re: [PATCH] pwm: Avoid double mutex lock on pwm_enable

2015-12-10 Thread Anand Moon
Hi Krzysztof, On 22 November 2015 at 05:43, Krzysztof Kozlowski wrote: > 2015-11-22 3:14 GMT+09:00 Anand Moon : >> Hi Krzysztof, >> >> On 21 November 2015 at 18:37, Krzysztof Kozlowski >> wrote: >>> 2015-11-21 21:11 GMT+09:00 Anand Moon : hi Krzysztof, On 21 November 2015 at 15:22

Re: [PATCH] pwm: Avoid double mutex lock on pwm_enable

2015-11-23 Thread Jonathan Richardson
On 15-11-23 02:07 AM, Thierry Reding wrote: > On Sun, Nov 22, 2015 at 09:13:17AM +0900, Krzysztof Kozlowski wrote: >> 2015-11-22 3:14 GMT+09:00 Anand Moon : >>> Hi Krzysztof, >>> >>> On 21 November 2015 at 18:37, Krzysztof Kozlowski >>> wrote: 2015-11-21 21:11 GMT+09:00 Anand Moon : > hi

Re: [PATCH] pwm: Avoid double mutex lock on pwm_enable

2015-11-23 Thread Thierry Reding
On Sun, Nov 22, 2015 at 09:13:17AM +0900, Krzysztof Kozlowski wrote: > 2015-11-22 3:14 GMT+09:00 Anand Moon : > > Hi Krzysztof, > > > > On 21 November 2015 at 18:37, Krzysztof Kozlowski > > wrote: > >> 2015-11-21 21:11 GMT+09:00 Anand Moon : > >>> hi Krzysztof, > >>> > >>> On 21 November 2015 at 1

Re: [PATCH] pwm: Avoid double mutex lock on pwm_enable

2015-11-21 Thread Krzysztof Kozlowski
2015-11-22 3:14 GMT+09:00 Anand Moon : > Hi Krzysztof, > > On 21 November 2015 at 18:37, Krzysztof Kozlowski > wrote: >> 2015-11-21 21:11 GMT+09:00 Anand Moon : >>> hi Krzysztof, >>> >>> On 21 November 2015 at 15:22, Krzysztof Kozlowski >>> wrote: 2015-11-21 18:40 GMT+09:00 Anand Moon :

Re: [PATCH] pwm: Avoid double mutex lock on pwm_enable

2015-11-21 Thread Anand Moon
Hi Krzysztof, On 21 November 2015 at 18:37, Krzysztof Kozlowski wrote: > 2015-11-21 21:11 GMT+09:00 Anand Moon : >> hi Krzysztof, >> >> On 21 November 2015 at 15:22, Krzysztof Kozlowski >> wrote: >>> 2015-11-21 18:40 GMT+09:00 Anand Moon : hi Krzysztof, On 21 November 2015 at 09:5

Re: [PATCH] pwm: Avoid double mutex lock on pwm_enable

2015-11-21 Thread Anand Moon
Hi Krzysztof, On 21 November 2015 at 18:37, Krzysztof Kozlowski wrote: > 2015-11-21 21:11 GMT+09:00 Anand Moon : >> hi Krzysztof, >> >> On 21 November 2015 at 15:22, Krzysztof Kozlowski >> wrote: >>> 2015-11-21 18:40 GMT+09:00 Anand Moon : hi Krzysztof, On 21 November 2015 at 09:5

Re: [PATCH] pwm: Avoid double mutex lock on pwm_enable

2015-11-21 Thread Krzysztof Kozlowski
2015-11-21 21:11 GMT+09:00 Anand Moon : > hi Krzysztof, > > On 21 November 2015 at 15:22, Krzysztof Kozlowski > wrote: >> 2015-11-21 18:40 GMT+09:00 Anand Moon : >>> hi Krzysztof, >>> >>> On 21 November 2015 at 09:56, Krzysztof Kozlowski >>> wrote: W dniu 21.11.2015 o 01:59, Anand Moon

Re: [PATCH] pwm: Avoid double mutex lock on pwm_enable

2015-11-21 Thread Anand Moon
hi Krzysztof, On 21 November 2015 at 15:22, Krzysztof Kozlowski wrote: > 2015-11-21 18:40 GMT+09:00 Anand Moon : >> hi Krzysztof, >> >> On 21 November 2015 at 09:56, Krzysztof Kozlowski >> wrote: >>> >>> W dniu 21.11.2015 o 01:59, Anand Moon pisze: >>> > Commit d1cd21427747f15920cd726f5f67a07880

Re: [PATCH] pwm: Avoid double mutex lock on pwm_enable

2015-11-21 Thread Krzysztof Kozlowski
2015-11-21 18:40 GMT+09:00 Anand Moon : > hi Krzysztof, > > On 21 November 2015 at 09:56, Krzysztof Kozlowski > wrote: >> >> W dniu 21.11.2015 o 01:59, Anand Moon pisze: >> > Commit d1cd21427747f15920cd726f5f67a07880e7dee4 >> > (pwm: Set enable state properly on failed call to enable) >> > introdu

Re: [PATCH] pwm: Avoid double mutex lock on pwm_enable

2015-11-20 Thread Krzysztof Kozlowski
W dniu 21.11.2015 o 01:59, Anand Moon pisze: > Commit d1cd21427747f15920cd726f5f67a07880e7dee4 > (pwm: Set enable state properly on failed call to enable) > introduce double lock of mutex on pwm_enable > > Changes fix the following debug lock warning. > > [2.701720] WARNING: CPU: 3 PID: 0 at

[PATCH] pwm: Avoid double mutex lock on pwm_enable

2015-11-20 Thread Anand Moon
Commit d1cd21427747f15920cd726f5f67a07880e7dee4 (pwm: Set enable state properly on failed call to enable) introduce double lock of mutex on pwm_enable Changes fix the following debug lock warning. [2.701720] WARNING: CPU: 3 PID: 0 at kernel/locking/mutex.c:526 __mutex_lock_slowpath+0x3bc/0x4