Fwd: Re: BUG: double mutex_unlock in drivers/media/video/tlg2300/pd-video.c

2011-01-21 Thread Alexander Strakh
--- Begin Message --- Hello, > Hi Strakh: > > Thanks for your patch. > > But I prefer to remove the mutex_unlock() in the pd_vidioc_s_fmt(), > since the pd_vidioc_s_fmt() is also called in restore_v4l2_context(). > > would you please change the patch? > I will ack it. > > Best Regards > Huang

Re: BUG: double mutex_unlock in drivers/media/video/tlg2300/pd-video.c

2010-12-22 Thread Mauro Carvalho Chehab
Em 13-12-2010 13:59, iceberg escreveu: > KERNEL_VERSION: 2.6.36 > SUBJECT: double mutex_lock in drivers/media/video/tlg2300/pd-video.c > in function vidioc_s_fmt > SUBSCRIBE: > First mutex_unlock in function pd_vidioc_s_fmt in line 767: > > 764ret |= send_set_req(pd

Re: BUG: double mutex_unlock in drivers/media/video/tlg2300/pd-video.c

2010-12-13 Thread Huang Shijie
Hi Strakh: Thanks for your patch. But I prefer to remove the mutex_unlock() in the pd_vidioc_s_fmt(), since the pd_vidioc_s_fmt() is also called in restore_v4l2_context(). would you please change the patch? I will ack it. Best Regards Huang Shijie 2010/12/13 Alexander Strakh : >        KERNEL

BUG: double mutex_unlock in drivers/media/video/tlg2300/pd-video.c

2010-12-13 Thread Alexander Strakh
KERNEL_VERSION: 2.6.36 SUBJECT: double mutex_lock in drivers/media/video/tlg2300/pd-video.c in function vidioc_s_fmt SUBSCRIBE: First mutex_unlock in function pd_vidioc_s_fmt in line 767: 764ret |= send_set_req(pd, VIDEO_ROSOLU_SEL, 765