Re: Doubt regarding DMA / VMALLOC memory ops

2013-05-15 Thread anish singh
On Wed, May 15, 2013 at 4:10 PM, Chetan Nanda wrote: > Hi, > > I am new to V4L2 kernel framework. And currently trying to understand V4L2 > kernel sub-system. > In videobuf2 sub-system, before calling 'vb2_queue_init(q)', q's mem_ops is > being initialized (depending on type of buffer) as > > q->

Re: [PATCH RFC] media: OF: add field-active and sync-on-green endpoint properties

2013-05-15 Thread Prabhakar Lad
Hi Laurent, On Wed, May 15, 2013 at 6:54 PM, Laurent Pinchart wrote: > Hi Prabhakar, > > Thank you for the patch. > > On Wednesday 15 May 2013 18:22:29 Lad Prabhakar wrote: >> From: Lad, Prabhakar >> >> This patch adds "field-active" and "sync-on-green" as part of >> endpoint properties and also

[PATCH v4] V4L2: I2C: ML86V7667 video decoder driver

2013-05-15 Thread Sergei Shtylyov
From: Vladimir Barinov Add OKI Semiconductor ML86V7667 video decoder driver. Signed-off-by: Vladimir Barinov [Sergei: added v4l2_device_unregister_subdev() call to the error cleanup path of ml86v7667_probe(); some cleanup.] Signed-off-by: Sergei Shtylyov --- This patch is against the 'media_t

Re: [RFC PATCH 2/3] [media] s5p-jpeg: Add DT support to JPEG driver

2013-05-15 Thread Sylwester Nawrocki
On 05/14/2013 01:53 PM, George Joseph wrote: From: George Joseph Palathingal Adding DT support to the driver. Driver supports Exynos 4210, 4412 and 5250. Signed-off-by: George Joseph Palathingal Cc: devicetree-disc...@lists.ozlabs.org --- drivers/media/platform/s5p-jpeg/jpeg-core.c | 36 +++

Re: [RFC PATCH 1/3] [media] s5p-jpeg: Add support for Exynos4x12 and 5250

2013-05-15 Thread Sylwester Nawrocki
Hi George, Thanks for the patches. Sorry, I can't review the $subject patch in detail as is, there is way too many things done in this single patch. It looks more like a driver replacement. It is even hard to edit due to its size in my e-mail client. Hence, may I ask you to split it into several

[PATCH v4 3/3] ARM: shmobile: Marzen: enable VIN and ADV7180 in defconfig

2013-05-15 Thread Sergei Shtylyov
From: Vladimir Barinov Add the VIN and ADV7180 drivers to 'marzen_defconfig'. Signed-off-by: Vladimir Barinov Signed-off-by: Sergei Shtylyov --- arch/arm/configs/marzen_defconfig |7 +++ 1 file changed, 7 insertions(+) Index: renesas/arch/arm/configs/marzen_defconfig ===

[PATCH v4 2/3] ARM: shmobile: Marzen: add VIN and ADV7180 support

2013-05-15 Thread Sergei Shtylyov
From: Vladimir Barinov Add ADV7180 platform devices on the Marzen board, configure VIN1/3 pins, and register VIN1/3 devices with the ADV7180 specific platform data. Signed-off-by: Vladimir Barinov [Sergei: removed superfluous tabulation and inserted empty lines in the macro definition, updated

[PATCH] [media] sony-btf-mpx: Drop needless newline in param description

2013-05-15 Thread Jean Delvare
Module parameter descriptions need not be terminated with a newline. Signed-off-by: Jean Delvare Cc: Hans Verkuil Cc: Mauro Carvalho Chehab --- drivers/media/i2c/sony-btf-mpx.c |2 +- 1 file changed, 1 insertion(+), 1 deletion(-) --- linux-3.10-rc1.orig/drivers/media/i2c/sony-btf-mpx.c

[PATCH v4 1/3] ARM: shmobile: r8a7779: add VIN support

2013-05-15 Thread Sergei Shtylyov
From: Vladimir Barinov Add VIN clocks and platform devices for R8A7779 SoC; add function to register the VIN platform devices. Signed-off-by: Vladimir Barinov [Sergei: added 'id' parameter check to r8a7779_add_vin_device(), used '*pdata' in *sizeof* operator there, renamed some variables, annot

[PATCH v4 0/3] R8A7779/Marzen R-Car VIN driver support

2013-05-15 Thread Sergei Shtylyov
Hello. Here's the set of 3 patches against the Simon Horman's 'renesas.git' repo, 'renesas-next-20130515v2' tag and my recent yet unapplied USB/I2C patches. Here we add the VIN driver platform code for the R8A7779/Marzen with ADV7180 I2C video decoder. [1/3] ARM: shmobile: r8a7779: add VIN sup

[PATCH 1/1] [media] uvcvideo: quirk PROBE_DEF for Alienware X51 OmniVision webcam

2013-05-15 Thread joseph . salisbury
From: Joseph Salisbury BugLink: http://bugs.launchpad.net/bugs/1180409 OminiVision webcam 0x05a9:0x2643 needs the same UVC_QUIRK_PROBE_DEF as other OmniVision models to work properly. Cc: Laurent Pinchart Cc: Mauro Carvalho Chehab Cc: linux-media@vger.kernel.org Cc: sta...@vger.kernel.org Si

Re: [PATCH v4] V4L2: soc_camera: Renesas R-Car VIN driver

2013-05-15 Thread Vladimir Barinov
Hi Guennadi, On 05/15/2013 08:44 AM, Guennadi Liakhovetski wrote: Hi Sergei, Vladimir On Wed, 15 May 2013, Sergei Shtylyov wrote: From: Vladimir Barinov Add Renesas R-Car VIN (Video In) V4L2 driver. Based on the patch by Phil Edworthy. Signed-off-by: Vladimir Barinov [Sergei: removed depre

cron job: media_tree daily build: ERRORS

2013-05-15 Thread Hans Verkuil
This message is generated daily by a cron job that builds media_tree for the kernels and architectures in the list below. Results of the daily build of media_tree: date: Wed May 15 19:59:44 CEST 2013 git branch: test git hash: 4237c09a63906b980741725da63f85e454caec02 gcc versi

Re: [PATCH v2] uvcvideo: Fix open/close race condition

2013-05-15 Thread Shawn Nematbakhsh
Hi Laurent, Thanks for the revised patch. I tested and confirmed that it fixes the issue I reported. On Wed, May 15, 2013 at 2:58 AM, Laurent Pinchart wrote: > Hi Shawn, > > Could you please confirm whether this patch fixes the issue you've reported ? > > On Tuesday 07 May 2013 13:19:37 Laurent

cron job: media_tree daily build: ERRORS

2013-05-15 Thread Hans Verkuil
This message is generated daily by a cron job that builds media_tree for the kernels and architectures in the list below. Results of the daily build of media_tree: date: Wed May 15 19:00:22 CEST 2013 git branch: test git hash: 4237c09a63906b980741725da63f85e454caec02 gcc versi

Re: [RFCv2] Motion Detection API

2013-05-15 Thread Sylwester Nawrocki
Hi Hans, On 05/13/2013 11:32 AM, Hans Verkuil wrote: This RFC looks at adding support for motion detection to V4L2. This is the main missing piece that prevents the go7007 and solo6x10 drivers from being moved into mainline from the staging directory. This is the second version of this RFC. The

Re: Introduce a new helper framework for buffer synchronization

2013-05-15 Thread Rob Clark
On Wed, May 15, 2013 at 1:19 AM, Inki Dae wrote: > > >> -Original Message- >> From: Rob Clark [mailto:robdcl...@gmail.com] >> Sent: Tuesday, May 14, 2013 10:39 PM >> To: Inki Dae >> Cc: linux-fbdev; DRI mailing list; Kyungmin Park; myungjoo.ham; YoungJun >> Cho; linux-arm-ker...@lists.infr

omap3 : isp clock a : Difference between dmesg frequency and actual frequency with 3.9

2013-05-15 Thread jean-philippe francois
Hi, I am working on a dm3730 based camera. The sensor input clock is provided by the cpu via the CAM_XCLK pin. Here is the corresponding log : [9.115966] Entering cam_set_xclk [9.119781] omap3isp omap3isp: isp_set_xclk(): cam_xclka set to 24685714 Hz [9.121337] ov10x33 1-0010: sensor

Re: [PATCH RFC] media: OF: add field-active and sync-on-green endpoint properties

2013-05-15 Thread Laurent Pinchart
Hi Prabhakar, Thank you for the patch. On Wednesday 15 May 2013 18:22:29 Lad Prabhakar wrote: > From: Lad, Prabhakar > > This patch adds "field-active" and "sync-on-green" as part of > endpoint properties and also support to parse them in the parser. > > Signed-off-by: Lad, Prabhakar > Cc: Ha

[PATCH RFC] media: OF: add field-active and sync-on-green endpoint properties

2013-05-15 Thread Lad Prabhakar
From: Lad, Prabhakar This patch adds "field-active" and "sync-on-green" as part of endpoint properties and also support to parse them in the parser. Signed-off-by: Lad, Prabhakar Cc: Hans Verkuil Cc: Laurent Pinchart Cc: Mauro Carvalho Chehab Cc: Guennadi Liakhovetski Cc: Sylwester Nawrocki

Re: [PATCH 2/6] ARM: davinci: dm365 evm: remove init_enable from ths7303 pdata

2013-05-15 Thread Prabhakar Lad
Hi Laurent, Thanks for the review. On Wed, May 15, 2013 at 5:35 PM, Laurent Pinchart wrote: > Hi Prabhakar, > > Thank you for the patch. > > On Wednesday 15 May 2013 17:27:18 Lad Prabhakar wrote: >> From: Lad, Prabhakar >> >> remove init_enable from ths7303 pdata as it is no longer exists. > >

Re: [PATCH 2/6] ARM: davinci: dm365 evm: remove init_enable from ths7303 pdata

2013-05-15 Thread Laurent Pinchart
Hi Prabhakar, Thank you for the patch. On Wednesday 15 May 2013 17:27:18 Lad Prabhakar wrote: > From: Lad, Prabhakar > > remove init_enable from ths7303 pdata as it is no longer exists. You should move this before 1/6, otherwise you will break bisection. > Signed-off-by: Lad, Prabhakar > Cc:

[PATCH 6/6] media: i2c: ths7303: add OF support

2013-05-15 Thread Lad Prabhakar
From: Lad, Prabhakar add OF support for the ths7303 driver. Signed-off-by: Lad, Prabhakar Cc: Hans Verkuil Cc: Laurent Pinchart Cc: Mauro Carvalho Chehab Cc: Guennadi Liakhovetski Cc: Sylwester Nawrocki Cc: Sakari Ailus Cc: Grant Likely Cc: Rob Herring Cc: Rob Landley Cc: devicetree-di

[PATCH 5/6] media: i2c: ths7303: add support for asynchronous probing

2013-05-15 Thread Lad Prabhakar
From: Lad, Prabhakar Both synchronous and asynchronous ths7303 subdevice probing is supported by this patch. Signed-off-by: Lad, Prabhakar Cc: Hans Verkuil Cc: Laurent Pinchart Cc: Mauro Carvalho Chehab Cc: Guennadi Liakhovetski Cc: Sylwester Nawrocki Cc: Sakari Ailus Cc: linux-ker...@vge

[PATCH 4/6] media: i2c: ths7303: make the pdata as a constant pointer

2013-05-15 Thread Lad Prabhakar
From: Lad, Prabhakar generally the pdata needs to be a constant pointer in the device state structure. This patch makes the pdata as a constant pointer and alongside returns -EINVAL when pdata is NULL. Signed-off-by: Lad, Prabhakar Cc: Hans Verkuil Cc: Laurent Pinchart Cc: Mauro Carvalho Cheh

[PATCH 3/6] media: i2c: ths7303: remove unnecessary function ths7303_setup()

2013-05-15 Thread Lad Prabhakar
From: Lad, Prabhakar the ths7303_setup() was doing the same thing as ths7303_setval() except that ths7303_setval() sets it to some particular mode. This patch removes ths7303_setup() function and calls ths7303_setval() in the probe setting the device to 480I_576I filter mode in the probe. Signed

[PATCH 2/6] ARM: davinci: dm365 evm: remove init_enable from ths7303 pdata

2013-05-15 Thread Lad Prabhakar
From: Lad, Prabhakar remove init_enable from ths7303 pdata as it is no longer exists. Signed-off-by: Lad, Prabhakar Cc: Sekhar Nori Cc: Hans Verkuil Cc: Laurent Pinchart Cc: Mauro Carvalho Chehab Cc: Guennadi Liakhovetski Cc: Sylwester Nawrocki Cc: Sakari Ailus Cc: linux-ker...@vger.kern

[PATCH 1/6] media: i2c: ths7303: remove init_enable option from pdata

2013-05-15 Thread Lad Prabhakar
From: Lad, Prabhakar This patch removes init_enable option from pdata, the init_enable was intended that the device should start streaming video immediately but ideally the bridge drivers should call s_stream explicitly for such devices to start video. Signed-off-by: Lad, Prabhakar Cc: Hans Ver

[PATCH 0/6] media: i2c: ths7303 feature enhancement and cleanup

2013-05-15 Thread Lad Prabhakar
From: Lad, Prabhakar This patch series enables the ths7303 driver for asynchronous probing, OF support with some cleanup patches. Lad, Prabhakar (6): media: i2c: ths7303: remove init_enable option from pdata ARM: davinci: dm365 evm: remove init_enable from ths7303 pdata media: i2c: ths7303

Re: [PATCH RFC v2 3/3] media: added managed v4l2 subdevice initialization

2013-05-15 Thread Hans Verkuil
On Wed 15 May 2013 10:29:16 Andrzej Hajda wrote: > On 13.05.2013 11:24, Hans Verkuil wrote: > > Hi Andrzej, > > > > On Mon May 13 2013 10:34:46 Andrzej Hajda wrote: > >> This patch adds managed versions of initialization > >> functions for v4l2 subdevices. > > I figured out what is bothering me abo

Re: [PATCH] console/font: Refactor font support code selection logic

2013-05-15 Thread Hans Verkuil
On Wed 15 May 2013 13:40:50 Geert Uytterhoeven wrote: > The current Makefile rules to build font support are messy and buggy. > Replace them by Kconfig rules: > - Introduce CONFIG_FONT_SUPPORT, which controls the building of all font > code, > - Select CONFIG_FONT_SUPPORT for all drivers th

[PATCH] console/font: Refactor font support code selection logic

2013-05-15 Thread Geert Uytterhoeven
The current Makefile rules to build font support are messy and buggy. Replace them by Kconfig rules: - Introduce CONFIG_FONT_SUPPORT, which controls the building of all font code, - Select CONFIG_FONT_SUPPORT for all drivers that use fonts, - Select CONFIG_FONT_8x16 for all drivers that d

Re: [PATCH RFC v2 2/3] media: added managed v4l2 control initialization

2013-05-15 Thread Laurent Pinchart
Hi Andrzej, Thank you for the patch. On Monday 13 May 2013 10:34:45 Andrzej Hajda wrote: > This patch adds managed versions of initialization > and cleanup functions for v4l2 control handler. > > Signed-off-by: Andrzej Hajda > Reviewed-by: Sylwester Nawrocki > Signed-off-by: Kyungmin Park > -

Doubt regarding DMA / VMALLOC memory ops

2013-05-15 Thread Chetan Nanda
Hi, I am new to V4L2 kernel framework. And currently trying to understand V4L2 kernel sub-system. In videobuf2 sub-system, before calling 'vb2_queue_init(q)', q's mem_ops is being initialized (depending on type of buffer) as q->mem_ops = &vb2_vmalloc_memops; or q->mem_ops = &vb2_dma_contig_memop

Re: [PATCH v2] uvcvideo: Fix open/close race condition

2013-05-15 Thread Laurent Pinchart
Hi Shawn, Could you please confirm whether this patch fixes the issue you've reported ? On Tuesday 07 May 2013 13:19:37 Laurent Pinchart wrote: > Maintaining the users count using an atomic variable makes sure that > access to the counter won't be racy, but doesn't serialize access to the > opera

Re: [PATCH 5/5] media: i2c: tvp7002: add OF support

2013-05-15 Thread Laurent Pinchart
Hi Prabhakar, On Wednesday 15 May 2013 10:48:05 Prabhakar Lad wrote: > On Tue, May 14, 2013 at 9:04 PM, Laurent Pinchart wrote: > > On Tuesday 14 May 2013 16:15:34 Lad Prabhakar wrote: > >> From: Lad, Prabhakar > >> > >> add OF support for the tvp7002 driver. > >> > >> Signed-off-by: Lad, Prabh

Re: [PATCH RFC v2 3/3] media: added managed v4l2 subdevice initialization

2013-05-15 Thread Andrzej Hajda
On 13.05.2013 11:24, Hans Verkuil wrote: > Hi Andrzej, > > On Mon May 13 2013 10:34:46 Andrzej Hajda wrote: >> This patch adds managed versions of initialization >> functions for v4l2 subdevices. > I figured out what is bothering me about this patch: the fact that it is > tied to the v4l2_i2c_subde