Re: [PATCH] media: mt9p031/mt9t001/mt9v032: use V4L2_CID_TEST_PATTERN for test pattern control

2012-09-26 Thread Prabhakar Lad
Hi Laurent, Thanks for the review. On Tue, Sep 25, 2012 at 9:30 PM, Laurent Pinchart laurent.pinch...@ideasonboard.com wrote: Hi Prabhakar, Thank you for the patch. On Tuesday 25 September 2012 18:29:25 Prabhakar wrote: From: Lad, Prabhakar prabhakar@ti.com Signed-off-by: Lad,

[PATCH v2] media: mt9p031/mt9t001/mt9v032: use V4L2_CID_TEST_PATTERN for test pattern control

2012-09-26 Thread Prabhakar
From: Lad, Prabhakar prabhakar@ti.com Signed-off-by: Lad, Prabhakar prabhakar@ti.com Signed-off-by: Manjunath Hadli manjunath.ha...@ti.com Cc: Laurent Pinchart laurent.pinch...@ideasonboard.com Cc: Sakari Ailus sakari.ai...@iki.fi Cc: Paul Gortmaker paul.gortma...@windriver.com Cc: Jean

Re: [PATCHv8 18/26] v4l: add buffer exporting via dmabuf

2012-09-26 Thread Hans Verkuil
On Tue September 25 2012 18:30:43 Tomasz Stanislawski wrote: Hi Hans, Thank you for review. Please refer to the comments below. On 08/22/2012 01:41 PM, Hans Verkuil wrote: On Tue August 14 2012 17:34:48 Tomasz Stanislawski wrote: This patch adds extension to V4L2 api. It allow to export

Re: Gain controls in v4l2-ctrl framework

2012-09-26 Thread Prabhakar Lad
Hi All, On Sun, Sep 23, 2012 at 4:56 PM, Prabhakar Lad prabhakar.cse...@gmail.com wrote: Hi All, The CCD/Sensors have the capability to adjust the R/ye, Gr/Cy, Gb/G, B/Mg gain values. Since these control can be re-usable I am planning to add the following gain controls as part of the

Re: HVR 4000 and DVB-API

2012-09-26 Thread Dominique Michel
Le Tue, 25 Sep 2012 22:27:55 +0300, Rémi Denis-Courmont r...@remlab.net a écrit : Thanks for the answer. Le lundi 24 septembre 2012 10:51:23, Dominique Michel a écrit : The WinTV HVR-4000-HD is a multi-tuners TV card with 2 dvb tuners. It look like its driver doesn't have been updated to

Re: Gain controls in v4l2-ctrl framework

2012-09-26 Thread Chris MacGregor
Hi All. On 09/25/2012 11:44 PM, Prabhakar Lad wrote: Hi All, On Sun, Sep 23, 2012 at 4:56 PM, Prabhakar Lad prabhakar.cse...@gmail.com wrote: Hi All, The CCD/Sensors have the capability to adjust the R/ye, Gr/Cy, Gb/G, B/Mg gain values. Since these control can be re-usable I am planning to

Re: Gain controls in v4l2-ctrl framework

2012-09-26 Thread Prabhakar Lad
Hi Chris, On Wed, Sep 26, 2012 at 12:23 PM, Chris MacGregor ch...@cybermato.com wrote: Hi All. On 09/25/2012 11:44 PM, Prabhakar Lad wrote: Hi All, On Sun, Sep 23, 2012 at 4:56 PM, Prabhakar Lad prabhakar.cse...@gmail.com wrote: Hi All, The CCD/Sensors have the capability to adjust

Re: [RFCv1 API PATCH 4/4] tuner-core: map audmode to STEREO for radio devices.

2012-09-26 Thread Hans Verkuil
On Wed September 26 2012 04:29:33 Mauro Carvalho Chehab wrote: Em Tue, 25 Sep 2012 15:45:00 +0200 Hans Verkuil hansv...@cisco.com escreveu: On Tue 25 September 2012 15:33:40 Mauro Carvalho Chehab wrote: Em Fri, 14 Sep 2012 13:15:36 +0200 Hans Verkuil hans.verk...@cisco.com escreveu:

[PATCH 1/2] [media] exynos-gsc: Remove linux/version.h header file inclusion

2012-09-26 Thread Sachin Kamat
version.h is not needed for these files. Signed-off-by: Sachin Kamat sachin.ka...@linaro.org --- drivers/media/platform/exynos-gsc/gsc-core.c |1 - drivers/media/platform/exynos-gsc/gsc-m2m.c |1 - 2 files changed, 0 insertions(+), 2 deletions(-) diff --git

[PATCH 2/2] [media] exynos-gsc: Add missing static storage class specifiers

2012-09-26 Thread Sachin Kamat
Fixes the following warnings: drivers/media/platform/exynos-gsc/gsc-core.c:313:5: warning: symbol 'get_plane_info' was not declared. Should it be static? drivers/media/platform/exynos-gsc/gsc-core.c:746:28: warning: symbol 'gsc_ctrl_ops' was not declared. Should it be static?

Re: Gain controls in v4l2-ctrl framework

2012-09-26 Thread Sakari Ailus
On Wed, Sep 26, 2012 at 12:14:36PM +0530, Prabhakar Lad wrote: Hi All, On Sun, Sep 23, 2012 at 4:56 PM, Prabhakar Lad prabhakar.cse...@gmail.com wrote: Hi All, The CCD/Sensors have the capability to adjust the R/ye, Gr/Cy, Gb/G, B/Mg gain values. Since these control can be re-usable

Re: Gain controls in v4l2-ctrl framework

2012-09-26 Thread Prabhakar Lad
Hi Sakari, On Wed, Sep 26, 2012 at 1:12 PM, Sakari Ailus sakari.ai...@iki.fi wrote: On Wed, Sep 26, 2012 at 12:14:36PM +0530, Prabhakar Lad wrote: Hi All, On Sun, Sep 23, 2012 at 4:56 PM, Prabhakar Lad prabhakar.cse...@gmail.com wrote: Hi All, The CCD/Sensors have the capability to

[GIT PULL FOR v3.7] Add back lost tda9875 copyright

2012-09-26 Thread Hans Verkuil
When the separate tda9875 driver was merged into tvaudio the copyright line of the tda9875 driver was dropped inadvertently. Add it back. Regards, Hans The following changes since commit 4313902ebe33155209472215c62d2f29d117be29: [media] ivtv-alsa, ivtv: Connect ivtv PCM capture

Re: Gain controls in v4l2-ctrl framework

2012-09-26 Thread Sakari Ailus
On Wed, Sep 26, 2012 at 01:16:08PM +0530, Prabhakar Lad wrote: ... Currently I am need of following, 1: V4L2_CID_GAIN_RED 2: V4L2_CID_GAIN_GREEN_RED 3: V4L2_CID_GAIN_GREEN_BLUE 4: V4L2_CID_GAIN_BLUE 5: V4L2_CID_GAIN_OFFSET Are they analogue or digital? -- Sakari Ailus e-mail:

Re: omap3isp: wrong image after resizer with mt9v034 sensor

2012-09-26 Thread Enric Balletbò i Serra
Hi Laurent 2012/9/25 Laurent Pinchart laurent.pinch...@ideasonboard.com: Hi Enric, On Tuesday 25 September 2012 13:23:20 Enric Balletbò i Serra wrote: 2012/9/25 Laurent Pinchart laurent.pinch...@ideasonboard.com: On Tuesday 25 September 2012 09:44:42 Enric Balletbò i Serra wrote:

re: [media] v4l2-subdev: add support for the new edid ioctls

2012-09-26 Thread Dan Carpenter
Hi Hans, The patch ed45ce2cc0b3: [media] v4l2-subdev: add support for the new edid ioctls from Aug 10, 2012, needs an overflow check the same as the other cases in that switch statement. drivers/media/v4l2-core/v4l2-ioctl.c 2200 case VIDIOC_SUBDEV_G_EDID: 2201 case

Re: Gain controls in v4l2-ctrl framework

2012-09-26 Thread Prabhakar Lad
On Wed, Sep 26, 2012 at 1:24 PM, Sakari Ailus sakari.ai...@iki.fi wrote: On Wed, Sep 26, 2012 at 01:16:08PM +0530, Prabhakar Lad wrote: ... Currently I am need of following, 1: V4L2_CID_GAIN_RED 2: V4L2_CID_GAIN_GREEN_RED 3: V4L2_CID_GAIN_GREEN_BLUE 4: V4L2_CID_GAIN_BLUE 5:

Re: [media] v4l2-subdev: add support for the new edid ioctls

2012-09-26 Thread Hans Verkuil
On Wed 26 September 2012 10:01:06 Dan Carpenter wrote: Hi Hans, The patch ed45ce2cc0b3: [media] v4l2-subdev: add support for the new edid ioctls from Aug 10, 2012, needs an overflow check the same as the other cases in that switch statement. drivers/media/v4l2-core/v4l2-ioctl.c 2200

Re: [GIT PULL FOR v3.7] Add missing vidioc-subdev-g-edid.xml.

2012-09-26 Thread Hans Verkuil
On Tue 25 September 2012 13:56:34 Hans Verkuil wrote: Hi Mauro, As requested! I've respun this tree, fixing one documentation bug (the max value for 'blocks' is 256, not 255) and adding an overflow check in v4l2-ioctl.c as reported by Dan Carpenter:

Re: omap3isp: wrong image after resizer with mt9v034 sensor

2012-09-26 Thread Laurent Pinchart
Hi Enric, On Wednesday 26 September 2012 09:57:53 Enric Balletbò i Serra wrote: [snip] You had reason. Checking the data lines of the camera bus with an oscilloscope I see I had a problem, exactly in D8 /D9 data lines. I'm curious, how have you fixed that ? Now I can capture images but the

Re: [PATCH] [media] s5p-tv: Fix potential NULL pointer dereference error

2012-09-26 Thread Sylwester Nawrocki
On 09/24/2012 10:52 AM, Tomasz Stanislawski wrote: On 09/22/2012 09:39 AM, Sachin Kamat wrote: When mdev is NULL, the error print statement will try to dereference the NULL pointer. Signed-off-by: Sachin Kamat sachin.ka...@linaro.org --- drivers/media/platform/s5p-tv/mixer_drv.c |2 +-

Re: [PATCH Resend] [media] s5p-fimc: Fix incorrect condition in fimc_lite_reqbufs()

2012-09-26 Thread Sylwester Nawrocki
On 09/26/2012 05:53 AM, Sachin Kamat wrote: Fixes a typo in a conditional evaluation. Signed-off-by: Sachin Kamat sachin.ka...@linaro.org Applied, thanks! -- To unsubscribe from this list: send the line unsubscribe linux-media in the body of a message to majord...@vger.kernel.org More

Re: [PATCH v2] media: mt9p031/mt9t001/mt9v032: use V4L2_CID_TEST_PATTERN for test pattern control

2012-09-26 Thread Laurent Pinchart
Hi Prabhakar, Thanks for the patch. On Wednesday 26 September 2012 12:05:10 Prabhakar wrote: From: Lad, Prabhakar prabhakar@ti.com Signed-off-by: Lad, Prabhakar prabhakar@ti.com Signed-off-by: Manjunath Hadli manjunath.ha...@ti.com Cc: Laurent Pinchart

Re: [PATCH 1/2] [media] exynos-gsc: Remove linux/version.h header file inclusion

2012-09-26 Thread Sylwester Nawrocki
On 09/26/2012 09:18 AM, Sachin Kamat wrote: version.h is not needed for these files. Signed-off-by: Sachin Kamat sachin.ka...@linaro.org Applied both, thank you. -- To unsubscribe from this list: send the line unsubscribe linux-media in the body of a message to majord...@vger.kernel.org More

Re: [RFC] Timestamps and V4L2

2012-09-26 Thread Laurent Pinchart
Hi Sakari, On Tuesday 25 September 2012 23:12:00 Sakari Ailus wrote: Hans Verkuil wrote: On Tue 25 September 2012 12:48:01 Laurent Pinchart wrote: On Tuesday 25 September 2012 08:47:45 Hans Verkuil wrote: On Tue September 25 2012 02:00:55 Laurent Pinchart wrote: BTW, I think we should

Re: [RFCv1 API PATCH 4/4] tuner-core: map audmode to STEREO for radio devices.

2012-09-26 Thread Mauro Carvalho Chehab
Em Wed, 26 Sep 2012 09:03:13 +0200 Hans Verkuil hverk...@xs4all.nl escreveu: On Wed September 26 2012 04:29:33 Mauro Carvalho Chehab wrote: Em Tue, 25 Sep 2012 15:45:00 +0200 Hans Verkuil hansv...@cisco.com escreveu: On Tue 25 September 2012 15:33:40 Mauro Carvalho Chehab wrote: Em

[PATCH 0/5] media: ov7670: driver cleanup and support for ov7674.

2012-09-26 Thread Javier Martin
The following series includes all the changes discussed in [1] that don't affect either bridge drivers that use ov7670 or soc-camera framework For this reason they are considered non controversial and sent separately. At least 1 more series will follow in order to implement all features described

[PATCH 1/5] media: ov7670: add support for ov7675.

2012-09-26 Thread Javier Martin
ov7675 and ov7670 share the same registers but there is no way to distinguish them at runtime. However, they require different tweaks to achieve the desired resolution. For this reason this patch adds a new ov7675 entry to the ov7670_id table. Signed-off-by: Javier Martin

[PATCH 2/5] media: ov7670: make try_fmt() consistent with 'min_height' and 'min_width'.

2012-09-26 Thread Javier Martin
'min_height' and 'min_width' are variables that allow to specify the minimum resolution that the sensor will achieve. This patch make v4l2 fmt callbacks consider this parameters in order to return valid data to user space. Signed-off-by: Javier Martin javier.mar...@vista-silicon.com ---

[PATCH 4/5] media: ov7670: add possibility to bypass pll for ov7675.

2012-09-26 Thread Javier Martin
Signed-off-by: Javier Martin javier.mar...@vista-silicon.com --- drivers/media/i2c/ov7670.c | 24 ++-- include/media/ov7670.h |1 + 2 files changed, 23 insertions(+), 2 deletions(-) diff --git a/drivers/media/i2c/ov7670.c b/drivers/media/i2c/ov7670.c index

[PATCH 3/5] media: ov7670: calculate framerate properly for ov7675.

2012-09-26 Thread Javier Martin
According to the datasheet ov7675 uses a formula to achieve the desired framerate that is different from the operations done in the current code. In fact, this formula should apply to ov7670 too. This would mean that current code is wrong but, in order to preserve compatibility, the new formula

[PATCH 5/5] media: ov7670: Add possibility to disable pixclk during hblank.

2012-09-26 Thread Javier Martin
Signed-off-by: Javier Martin javier.mar...@vista-silicon.com --- drivers/media/i2c/ov7670.c |8 include/media/ov7670.h |1 + 2 files changed, 9 insertions(+) diff --git a/drivers/media/i2c/ov7670.c b/drivers/media/i2c/ov7670.c index 54fb535..f7e4341 100644 ---

Re: [PATCHv8 18/26] v4l: add buffer exporting via dmabuf

2012-09-26 Thread Tomasz Stanislawski
Hi Hans, Thank your for your comments. On 09/26/2012 08:39 AM, Hans Verkuil wrote: On Tue September 25 2012 18:30:43 Tomasz Stanislawski wrote: Hi Hans, Thank you for review. Please refer to the comments below. On 08/22/2012 01:41 PM, Hans Verkuil wrote: On Tue August 14 2012 17:34:48

Re: [RFCv1 API PATCH 4/4] tuner-core: map audmode to STEREO for radio devices.

2012-09-26 Thread Hans Verkuil
On Wed 26 September 2012 11:35:27 Mauro Carvalho Chehab wrote: Em Wed, 26 Sep 2012 09:03:13 +0200 Hans Verkuil hverk...@xs4all.nl escreveu: On Wed September 26 2012 04:29:33 Mauro Carvalho Chehab wrote: Em Tue, 25 Sep 2012 15:45:00 +0200 Hans Verkuil hansv...@cisco.com escreveu:

Re: HVR 4000 and DVB-API

2012-09-26 Thread Dominique Michel
Le Wed, 26 Sep 2012 09:48:41 +0200, Dominique Michel dominique.mic...@vtxnet.ch a écrit : Le Tue, 25 Sep 2012 22:27:55 +0300, Rémi Denis-Courmont r...@remlab.net a écrit : Thanks for the answer. Le lundi 24 septembre 2012 10:51:23, Dominique Michel a écrit : The WinTV HVR-4000-HD is

[PATCH] em28xx: Replace memcpy with struct assignment

2012-09-26 Thread Ezequiel Garcia
This kind of memcpy() is error-prone and its replacement with a struct assignment is prefered. Signed-off-by: Ezequiel Garcia elezegar...@gmail.com --- drivers/media/usb/em28xx/em28xx-cards.c |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git

Re: [RFCv1 API PATCH 2/4] v4l2-ctrls: add a notify callback.

2012-09-26 Thread Laurent Pinchart
Hi Hans, Thanks for the patch. On Friday 14 September 2012 13:15:34 Hans Verkuil wrote: Sometimes platform/bridge drivers need to be notified when a control from a subdevice changes value. In order to support this a notify callback was added. Signed-off-by: Hans Verkuil

Re: [PATCH] media-ctl: Fix build error with newer autotools

2012-09-26 Thread Laurent Pinchart
Hi Gary, On Monday 24 September 2012 08:34:24 Gary Thomas wrote: Rename configure.in to be configure.ac - required for newer versions of autotools (older versions silently handled this, now it's an error) Signed-off-by: Gary Thomas g...@mlbassoc.com Thanks for the patch. Applied and

Re: [PATCH 4/4] uvc: Add return code check at vb2_queue_init()

2012-09-26 Thread Laurent Pinchart
Hi Ezequiel, Thanks for the patch. On Monday 17 September 2012 10:49:50 Ezequiel Garcia wrote: This function returns an integer and it's mandatory to check the return code. Signed-off-by: Ezequiel Garcia elezegar...@gmail.com --- drivers/media/usb/uvc/uvc_queue.c |8 ++--

[PATCH v2] uvc: Add return code check at vb2_queue_init()

2012-09-26 Thread Ezequiel Garcia
This function returns an integer and it's mandatory to check the return code. Cc: Laurent Pinchart laurent.pinch...@ideasonboard.com Signed-off-by: Ezequiel Garcia elezegar...@gmail.com --- Changes from v1: * Change variable name rc - ret * Add return 0 on successful uvc_queue_init

Re: ISPsupport

2012-09-26 Thread Laurent Pinchart
Hi John, On Monday 10 September 2012 20:00:54 John Tobias wrote: Hi all, I tried devel-ISPSUPPORT-IPIPE and devel-ISPSUPPORT, It would help you you told use what hardware you're running on, what kernel version you're using, and what devel-ISPSUPPORT-IPIPE and devel-ISPSUPPORT are. the

Re: RFC: use of timestamp/sequence in v4l2_buffer

2012-09-26 Thread Laurent Pinchart
Hi, On Thursday 13 September 2012 21:44:48 Sakari Ailus wrote: On Tue, Sep 04, 2012 at 12:38:06PM +0200, Hans Verkuil wrote: Hi all, During the Media Workshop last week we discussed how the timestamp and sequence fields in struct v4l2_buffer should be used. While trying to document

Re: Integrate camera interface of OMAP3530 in Angstrom Linux

2012-09-26 Thread Laurent Pinchart
Hi Andreas, On Thursday 06 September 2012 20:00:04 Andreas Nagel wrote: Hello, I am using an embedded module called TAO-3530 from Technexion, which has an OMAP3530 processor. This processor has a camera interface, which is part of the ISP submodule. For an ongoing project I want to capture

Re: [PATCH v2] uvc: Add return code check at vb2_queue_init()

2012-09-26 Thread Laurent Pinchart
On Wednesday 26 September 2012 08:30:34 Ezequiel Garcia wrote: This function returns an integer and it's mandatory to check the return code. Cc: Laurent Pinchart laurent.pinch...@ideasonboard.com Signed-off-by: Ezequiel Garcia elezegar...@gmail.com Acked-by: Laurent Pinchart

Re: [PATCH 0/5] media: ov7670: driver cleanup and support for ov7674.

2012-09-26 Thread javier Martin
On 26 September 2012 11:47, Javier Martin javier.mar...@vista-silicon.com wrote: The following series includes all the changes discussed in [1] that don't affect either bridge drivers that use ov7670 or soc-camera framework For this reason they are considered non controversial and sent

[PATCH] dvb-usb: print small buffers via %*ph

2012-09-26 Thread Andy Shevchenko
Signed-off-by: Andy Shevchenko andriy.shevche...@linux.intel.com --- drivers/media/usb/dvb-usb/a800.c |2 +- drivers/media/usb/dvb-usb/cinergyT2-core.c |3 +-- drivers/media/usb/dvb-usb/dibusb-common.c |2 +- drivers/media/usb/dvb-usb/digitv.c |2 +-

Re: omap3isp: wrong image after resizer with mt9v034 sensor

2012-09-26 Thread Enric Balletbò i Serra
Hi Laurent, 2012/9/26 Laurent Pinchart laurent.pinch...@ideasonboard.com: Hi Enric, On Wednesday 26 September 2012 09:57:53 Enric Balletbò i Serra wrote: [snip] You had reason. Checking the data lines of the camera bus with an oscilloscope I see I had a problem, exactly in D8 /D9 data

Re: Gain controls in v4l2-ctrl framework

2012-09-26 Thread Chris MacGregor
Hi. On 09/26/2012 12:42 AM, Sakari Ailus wrote: On Wed, Sep 26, 2012 at 12:14:36PM +0530, Prabhakar Lad wrote: Hi All, On Sun, Sep 23, 2012 at 4:56 PM, Prabhakar Lad prabhakar.cse...@gmail.com wrote: Hi All, The CCD/Sensors have the capability to adjust the R/ye, Gr/Cy, Gb/G, B/Mg gain

Re: Gain controls in v4l2-ctrl framework

2012-09-26 Thread Laurent Pinchart
Hi Chris, On Wednesday 26 September 2012 07:42:41 Chris MacGregor wrote: On 09/26/2012 12:42 AM, Sakari Ailus wrote: On Wed, Sep 26, 2012 at 12:14:36PM +0530, Prabhakar Lad wrote: On Sun, Sep 23, 2012 at 4:56 PM, Prabhakar Lad wrote: Hi All, The CCD/Sensors have the capability to

[PATCH RFC v3 0/5] s5p-fimc: Add interleaved image data capture support

2012-09-26 Thread Sylwester Nawrocki
Hi All, This patch series adds device/vendor specific media bus pixel code section and defines S5C73MX camera specific media bus pixel code, along with corresponding fourcc. The third patch adds support for MIPI-CSI2 Embedded Data capture in Samsung S5P/Exynos MIPI-CSIS device. It depends on

[PATCH RFC v3 1/5] V4L: Add V4L2_MBUS_FMT_S5C_UYVY_JPEG_1X8 media bus format

2012-09-26 Thread Sylwester Nawrocki
This patch adds media bus pixel code for the interleaved JPEG/UYVY image format used by S5C73MX Samsung cameras. This interleaved image data is transferred on MIPI-CSI2 bus as User Defined Byte-based Data. It also defines an experimental vendor and device specific media bus formats section and

[PATCH RFC v3 2/5] V4L: Add V4L2_PIX_FMT_S5C_UYVY_JPG fourcc definition

2012-09-26 Thread Sylwester Nawrocki
This patch adds definition of the Samsung S5C73M3 camera specific image format. V4L2_PIX_FMT_S5C_UYVY_JPG is a two-planar format, the first plane contains interleaved UYVY and JPEG data followed by meta-data. The second plane contains additional meta-data needed for extracting JPEG and UYVY data

[PATCH RFC v3 3/5] s5p-csis: Add support for non-image data packets capture

2012-09-26 Thread Sylwester Nawrocki
MIPI-CSI has internal memory mapped buffers for the frame embedded (non-image) data. There are two buffers, for even and odd frames which need to be saved after an interrupt is raised. The packet data buffers size is 4 KiB and there is no status register in the hardware where the actual non-image

[PATCH RFC v3 4/5] s5p-fimc: Add support for V4L2_PIX_FMT_S5C_UYVY_JPG fourcc

2012-09-26 Thread Sylwester Nawrocki
The V4L2_PIX_FMT_S5C_YUYV_JPG image formats consists of 2 planes, the first containing interleaved JPEG/YUYV data and the second containing meta data describing the interleaving method. The image data is transferred with MIPI-CSI User Defined Byte-Based Data 1 type and is captured to memory by

[PATCH RFC v3 5/5] m5mols: Implement .get_frame_desc subdev callback

2012-09-26 Thread Sylwester Nawrocki
.get_frame_desc can be used by host interface driver to query properties of captured frames, e.g. required memory buffer size. Signed-off-by: Sylwester Nawrocki s.nawro...@samsung.com Signed-off-by: Kyungmin Park kyungmin.p...@samsung.com --- drivers/media/i2c/m5mols/m5mols.h | 9 ++

Re: [PATCH 1/5] media: ov7670: add support for ov7675.

2012-09-26 Thread Jonathan Corbet
This is going to have to be quick, sorry... On Wed, 26 Sep 2012 11:47:53 +0200 Javier Martin javier.mar...@vista-silicon.com wrote: +static struct ov7670_win_size ov7670_win_sizes[2][4] = { + /* ov7670 */ I must confess I don't like this; now we've got constants in an array that was

Re: [PATCH 2/5] media: ov7670: make try_fmt() consistent with 'min_height' and 'min_width'.

2012-09-26 Thread Jonathan Corbet
On Wed, 26 Sep 2012 11:47:54 +0200 Javier Martin javier.mar...@vista-silicon.com wrote: 'min_height' and 'min_width' are variables that allow to specify the minimum resolution that the sensor will achieve. This patch make v4l2 fmt callbacks consider this parameters in order to return valid

Re: [PATCH 3/5] media: ov7670: calculate framerate properly for ov7675.

2012-09-26 Thread Jonathan Corbet
On Wed, 26 Sep 2012 11:47:55 +0200 Javier Martin javier.mar...@vista-silicon.com wrote: According to the datasheet ov7675 uses a formula to achieve the desired framerate that is different from the operations done in the current code. In fact, this formula should apply to ov7670 too. This

Re: [PATCH 4/5] media: ov7670: add possibility to bypass pll for ov7675.

2012-09-26 Thread Jonathan Corbet
On Wed, 26 Sep 2012 11:47:56 +0200 Javier Martin javier.mar...@vista-silicon.com wrote: Signed-off-by: Javier Martin javier.mar...@vista-silicon.com This one needs a changelog - what does bypassing the PLL do and why might you want to do it? Otherwise: Acked-by: Jonathan Corbet

Re: [PATCH 5/5] media: ov7670: Add possibility to disable pixclk during hblank.

2012-09-26 Thread Jonathan Corbet
On Wed, 26 Sep 2012 11:47:57 +0200 Javier Martin javier.mar...@vista-silicon.com wrote: Signed-off-by: Javier Martin javier.mar...@vista-silicon.com --- drivers/media/i2c/ov7670.c |8 include/media/ov7670.h |1 + 2 files changed, 9 insertions(+) Again, needs a

Re: mmotm 2012-09-25-17-06 uploaded (media/tuners/fc2580)

2012-09-26 Thread Randy Dunlap
On 09/25/2012 05:07 PM, a...@linux-foundation.org wrote: The mm-of-the-moment snapshot 2012-09-25-17-06 has been uploaded to http://www.ozlabs.org/~akpm/mmotm/ mmotm-readme.txt says README for mm-of-the-moment: http://www.ozlabs.org/~akpm/mmotm/ on i386: ERROR: __udivdi3

cron job: media_tree daily build: ERRORS

2012-09-26 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 Sep 26 19:00:22 CEST 2012 git hash:3f70e1f598a6be4277e71516a98457fd3bddfbd0 gcc version: i686-linux-gcc

Re: mmotm 2012-09-25-17-06 uploaded (media/tuners/fc2580)

2012-09-26 Thread Gianluca Gennari
Il 26/09/2012 19:26, Randy Dunlap ha scritto: On 09/25/2012 05:07 PM, a...@linux-foundation.org wrote: The mm-of-the-moment snapshot 2012-09-25-17-06 has been uploaded to http://www.ozlabs.org/~akpm/mmotm/ mmotm-readme.txt says README for mm-of-the-moment:

Re: [RFC] Timestamps and V4L2

2012-09-26 Thread Sakari Ailus
Hi Laurent, Laurent Pinchart wrote: On Tuesday 25 September 2012 23:12:00 Sakari Ailus wrote: Hans Verkuil wrote: On Tue 25 September 2012 12:48:01 Laurent Pinchart wrote: On Tuesday 25 September 2012 08:47:45 Hans Verkuil wrote: On Tue September 25 2012 02:00:55 Laurent Pinchart wrote: BTW,

Re: Gain controls in v4l2-ctrl framework

2012-09-26 Thread Sakari Ailus
Hi Laurent and Chris, Laurent Pinchart wrote: On Wednesday 26 September 2012 07:42:41 Chris MacGregor wrote: ... Sorry to make this more complicated, but the Aptina MT9P031, for instance (datasheet at http://www.aptina.com/assets/downloadDocument.do?id=865 - see page 35), has Digital Gain, an

Re: [PATCH] media: davinci: vpif: add check for NULL handler

2012-09-26 Thread Mauro Carvalho Chehab
Em Thu, 16 Aug 2012 19:32:00 +0530 Prabhakar Lad prabhakar@ti.com escreveu: It is amazing how many SOB's/acks are in this patch and nobody asked you to provide a patch description... the subject just tells what the code is also telling. Could you please provide a better patch description?

Re: [PATCH 0/1 v2] media/video: vpif: fixing function name start to vpif_config_params

2012-09-26 Thread Mauro Carvalho Chehab
Em Mon, 13 Aug 2012 19:43:29 +0530 Manjunath Hadli manjunath.ha...@ti.com escreveu: Hi Dror, Thanks for the patch. Mauro, I'll queue this patch for v3.7 through my tree. Sure. On Thursday 09 August 2012 12:03 PM, Dror Cohen wrote: This patch address the issue that a function

Re: [PATCH, RFC, updated] Fix DVB ioctls failing if frontend open/closed too fast

2012-09-26 Thread Juergen Lock
On Wed, Aug 01, 2012 at 12:22:16AM +0200, Juergen Lock wrote: That likely fixes this MythTV ticket: http://code.mythtv.org/trac/ticket/10830 (which btw affects all usb tuners I tested as well, pctv452e, dib0700, af9015) pctv452e is still possibly broken with MythTV even after this

Hang in Technotrend TT-connect CT-3650

2012-09-26 Thread Håkon Alstadheim
Hi all, I'm new to the list so feel free to direct me somewhere else if the following is OT. I get a hard lock on my machine, no response to Alt-Sysrq T, dead to the world, when running the mentioned card with a Conax cam to record encrypted DVB-T channels. I have had the thing running

Re: [RFC PATCH v8] media: add v4l2 subdev driver for S5K4ECGX sensor

2012-09-26 Thread Mauro Carvalho Chehab
Em Thu, 13 Sep 2012 12:02:14 +0100 Sangwook Lee sangwook@linaro.org escreveu: This patch adds driver for S5K4ECGX sensor with embedded ISP SoC, S5K4ECGX, which is a 5M CMOS Image sensor from Samsung The driver implements preview mode of the S5K4ECGX sensor. capture (snapshot) operation,

Re: [PATCH RFC] V4L: Add s_rx_buffer subdev video operation

2012-09-26 Thread Sakari Ailus
Hi Sylwester, Sylwester Nawrocki wrote: On 09/24/2012 08:26 PM, Sakari Ailus wrote: On Mon, Sep 24, 2012 at 06:51:41PM +0200, Sylwester Nawrocki wrote: On 09/24/2012 03:44 PM, Sakari Ailus wrote: How about useing a separate video buffer queue for the purpose? That would provide a nice way to

cron job: media_tree daily build: ERRORS

2012-09-26 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 Sep 26 20:33:55 CEST 2012 git hash:3f70e1f598a6be4277e71516a98457fd3bddfbd0 gcc version: i686-linux-gcc

[PATCH v2 0/2] OMAP 3 CSI-2 configuration

2012-09-26 Thread Sakari Ailus
Hi Paul and Laurent, This is an update to an old patchset for CSI-2 configuration for OMAP 3430 and 3630, and the corresponding patch to the ISP driver. Both have been tested on the 3630 only so far. Additional patches for the N9(50) camera support that mostly aren't yet upstreamable, are

[PATCH v2 2/2] omap3isp: Configure CSI-2 phy based on platform data

2012-09-26 Thread Sakari Ailus
Configure CSI-2 phy based on platform data in the ISP driver. For that, the new V4L2_CID_IMAGE_SOURCE_PIXEL_RATE control is used. Previously the same was configured from the board code. This patch is dependent on omap3: Provide means for changing CSI2 PHY configuration. Signed-off-by: Sakari

[PATCH v2 1/2] omap3: Provide means for changing CSI2 PHY configuration

2012-09-26 Thread Sakari Ailus
The OMAP 3630 has configuration how the ISP CSI-2 PHY pins are connected to the actual CSI-2 receivers outside the ISP itself. Allow changing this configuration from the ISP driver. Signed-off-by: Sakari Ailus sakari.ai...@iki.fi --- arch/arm/mach-omap2/control.c | 86

Re: [RFC PATCH v8] media: add v4l2 subdev driver for S5K4ECGX sensor

2012-09-26 Thread Sylwester Nawrocki
On 09/26/2012 10:32 PM, Mauro Carvalho Chehab wrote: Em Thu, 13 Sep 2012 12:02:14 +0100 Sangwook Leesangwook@linaro.org escreveu: This patch adds driver for S5K4ECGX sensor with embedded ISP SoC, S5K4ECGX, which is a 5M CMOS Image sensor from Samsung The driver implements preview mode

Re: [PATCH v2 2/2] omap3isp: Configure CSI-2 phy based on platform data

2012-09-26 Thread Tony Lindgren
Moi Sakari * Sakari Ailus sakari.ai...@iki.fi [120926 14:51]: Configure CSI-2 phy based on platform data in the ISP driver. For that, the new V4L2_CID_IMAGE_SOURCE_PIXEL_RATE control is used. Previously the same was configured from the board code. This patch is dependent on omap3: Provide

qv4l2-bug / libv4lconvert API issue

2012-09-26 Thread Frank Schäfer
Hi, I've noticed the following issues/bugs while playing with qv4l: 1.) with pac7302-webcams, only the RGB3 (RGB24) format is working. BGR3, YU12 and YV12 are broken 2.) for upside-down-mounted devices with an entry in libv4lconvert, automatic h/v-flipping doesn't work with some formats I've

Re: [RFC] Timestamps and V4L2

2012-09-26 Thread Sylwester Nawrocki
Hi Laurent, On 09/25/2012 02:35 AM, Laurent Pinchart wrote: Does the clock type need to be selectable for mem-to-mem devices ? Do device- specific timestamps make sense there ? I'd like to clarify one thing here, i.e. if we select device-specific timestamps how should the

HD-PVR fails consistently on Linux, works on Windows

2012-09-26 Thread Keith Pyle
I recently purchased a Hauppauge HD-PVR (the 1212 version, label on bottom 49001LF, Rev F2). I have consistent capture failures on Linux where data from the device simply stops, generally within a few minutes of starting a capture. Yet, the device works flawlessly on Windows with the same

Re: [PATCH 0/1 v2] media/video: vpif: fixing function name start to vpif_config_params

2012-09-26 Thread Prabhakar Lad
Hi Dror, On Thu, Sep 27, 2012 at 1:50 AM, Mauro Carvalho Chehab mche...@infradead.org wrote: Em Mon, 13 Aug 2012 19:43:29 +0530 Manjunath Hadli manjunath.ha...@ti.com escreveu: Hi Dror, Thanks for the patch. Mauro, I'll queue this patch for v3.7 through my tree. Sure. On Thursday

Re: [PATCH] media: davinci: vpif: add check for NULL handler

2012-09-26 Thread Prabhakar Lad
Hi Mauro, On Thu, Sep 27, 2012 at 1:37 AM, Mauro Carvalho Chehab mche...@infradead.org wrote: Em Thu, 16 Aug 2012 19:32:00 +0530 Prabhakar Lad prabhakar@ti.com escreveu: It is amazing how many SOB's/acks are in this patch and nobody asked you to provide a patch description... the subject

[PATCH v2] media: davinci: vpif: add check for NULL handler

2012-09-26 Thread Prabhakar
From: Lad, Prabhakar prabhakar@ti.com for da850/omap-l138, there is no need to setup_input_channel_mode() and set_clock(), to avoid adding dummy code in board file just returning zero add a check in the driver itself to call the handler only if its not NULL. Signed-off-by: Lad, Prabhakar