[kconfig] update: results of some syntactical checks

2013-10-19 Thread Martin Walch
This is an update to the syntactic results that I sent, back in July. With kernel 3.12 nearing completion, I would like to point to new sections in Kconfig files with potential problems: drivers/media/common/siano/Kconfig:21-26 > config SMS_SIANO_DEBUGFS > bool "Enable debugfs for smsdvb" >

Re: [RFD] use-counting V4L2 clocks

2013-10-19 Thread Sylwester Nawrocki
Hi All, On 10/15/2013 11:37 PM, Guennadi Liakhovetski wrote: On Tue, 15 Oct 2013, Laurent Pinchart wrote: > On Tuesday 15 October 2013 10:05:45 Guennadi Liakhovetski wrote: > > On Thu, 10 Oct 2013, Laurent Pinchart wrote: > > > On Tuesday 08 October 2013 23:57:55 Guennadi Liakhovetski w

Re: [PATCH v2] videobuf2: Add missing lock held on vb2_fop_relase

2013-10-19 Thread Ricardo Ribalda Delgado
Hello Sylwester On Sat, Oct 19, 2013 at 8:27 PM, Sylwester Nawrocki wrote: > On 10/19/2013 06:07 PM, Ricardo Ribalda wrote: > [...] >> >> --- >> drivers/media/platform/exynos4-is/fimc-capture.c | 2 +- >> drivers/media/platform/exynos4-is/fimc-lite.c| 2 +- >> drivers/media/usb/em28xx/

Re: [PATCH] [media] v4l: vb2-dma-contig: add support for file access mode flags for DMABUF exporting

2013-10-19 Thread Sylwester Nawrocki
Hi Philipp, On 10/08/2013 12:06 PM, Philipp Zabel wrote: Ping? This patch is needed to map videobuf2 exported dmabuf fds writeable from userspace, for example for timestamp software rendering into frames passing from a v4l2 capture device to a v4l2 output device. I have added this patch to my

Re: [PATCH v2] videobuf2: Add missing lock held on vb2_fop_relase

2013-10-19 Thread Sylwester Nawrocki
On 10/19/2013 06:07 PM, Ricardo Ribalda wrote: [...] --- drivers/media/platform/exynos4-is/fimc-capture.c | 2 +- drivers/media/platform/exynos4-is/fimc-lite.c| 2 +- drivers/media/usb/em28xx/em28xx-video.c | 2 +- drivers/media/v4l2-core/videobuf2-core.c | 18 +++

Re: ivtv 1.4.2/1.4.3 broken in recent kernels?

2013-10-19 Thread Andy Walls
On Wed, 2013-10-16 at 01:10 +0100, Rajil Saraswat wrote: > I was finally able to carry out a git bisect. Had to do a git pull on > a fast internet hooked machine and ftp the files over to the remote > machine. > > I started with 'git bisect bad v2.6.36.4' and 'git bisect good v2.6.35.10'. > > And

Re: [PATCH] em28xx: make sure that all subdevices are powered on when needed

2013-10-19 Thread Guennadi Liakhovetski
Hi Frank On Sat, 19 Oct 2013, Frank SchÀfer wrote: > Am 18.10.2013 22:30, schrieb Guennadi Liakhovetski: > > Hi Frank > > > > Thanks for the patch > > > > On Wed, 16 Oct 2013, Frank SchÀfer wrote: > > > >> Commit 622b828ab7 ("v4l2_subdev: rename tuner s_standby operation to > >> core s_power")

Re: cx23885: Add basic analog radio support

2013-10-19 Thread Miroslav Slugeň
Hi, i am not using those devices anymore for analog, we do change our devices to SDR and SW demodulation, but i am sending my latest patches all for kernel 3.2.XX Feel free to resubmit any kind of my code by your name ;) Miroslav Slugeň +420 724 825 885 Alfredo Jesús Delaiti napsal(a): Hi al

Re: [PATCH] videobuf2: Add missing lock held on vb2_fop_relase

2013-10-19 Thread Ricardo Ribalda Delgado
Hello Sylwester I have just posted a new version. Please take a look to it, it should fix your issue. I havent tried it in hw because I am out of the office. Regards! On Sat, Oct 19, 2013 at 12:51 PM, Sylwester Nawrocki wrote: > On 10/19/2013 12:22 PM, Ricardo Ribalda Delgado wrote: >> >> On S

[PATCH v2] videobuf2: Add missing lock held on vb2_fop_relase

2013-10-19 Thread Ricardo Ribalda
vb2_fop_relase does not held the lock although it is modifying the queue->owner field. This could lead to race conditions on the vb2_perform_io function when multiple applications are accessing the video device via read/write API: [ 308.297741] BUG: unable to handle kernel NULL pointer dereferenc

Re: [PATCH] videobuf2: Add missing lock held on vb2_fop_relase

2013-10-19 Thread Sylwester Nawrocki
On 10/19/2013 12:22 PM, Ricardo Ribalda Delgado wrote: On Sat, Oct 19, 2013 at 11:55 AM, Sylwester Nawrocki wrote: > On 10/14/2013 09:41 AM, Ricardo Ribalda Delgado wrote: >> >> vb2_fop_relase does not held the lock although it is modifying the >> queue->owner field. > [...] >> diff --

Re: [PATCH] videobuf2: Add missing lock held on vb2_fop_relase

2013-10-19 Thread Ricardo Ribalda Delgado
Hello Sylwester On Sat, Oct 19, 2013 at 11:55 AM, Sylwester Nawrocki wrote: > Hi Ricardo, > > > On 10/14/2013 09:41 AM, Ricardo Ribalda Delgado wrote: >> >> vb2_fop_relase does not held the lock although it is modifying the >> queue->owner field. > > [...] > >> diff --git a/drivers/media/v4l2-cor

Re: [PATCH] em28xx: make sure that all subdevices are powered on when needed

2013-10-19 Thread Frank Schäfer
Am 18.10.2013 22:30, schrieb Guennadi Liakhovetski: > Hi Frank > > Thanks for the patch > > On Wed, 16 Oct 2013, Frank Schäfer wrote: > >> Commit 622b828ab7 ("v4l2_subdev: rename tuner s_standby operation to >> core s_power") replaced the tuner s_standby call in the em28xx driver with >> a (s_power

Re: [PATCH] videobuf2: Add missing lock held on vb2_fop_relase

2013-10-19 Thread Sylwester Nawrocki
Hi Ricardo, On 10/14/2013 09:41 AM, Ricardo Ribalda Delgado wrote: vb2_fop_relase does not held the lock although it is modifying the queue->owner field. [...] diff --git a/drivers/media/v4l2-core/videobuf2-core.c b/drivers/media/v4l2-core/videobuf2-core.c index 9fc4bab..3a961ee 100644 --- a/