Re: [PATCHSET] kthread_worker: reimplement flush_kthread_work() to allow freeing during execution

2012-09-27 Thread Greg KH
On Mon, Sep 17, 2012 at 01:28:50PM -0700, Greg KH wrote: > On Mon, Sep 17, 2012 at 12:40:16PM -0700, Tejun Heo wrote: > > On Fri, Sep 14, 2012 at 03:50:40PM -0700, Colin Cross wrote: > > > This patch set fixes a reproducible crash I'm seeing on a 3.4.10 > > > kernel. flush_kthread_worker (which is

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

2012-09-27 Thread Tony Lindgren
* Laurent Pinchart [120927 02:52]: > Hi Tony, > > On Wednesday 26 September 2012 15:00:19 Tony Lindgren wrote: > > Moi Sakari > > > > * Sakari Ailus [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

Re: [PATCH] omap3isp: Replace cpu_is_omap3630() with ISP revision check

2012-09-27 Thread Tony Lindgren
* Laurent Pinchart [120927 07:38]: > Drivers must not rely on cpu_is_omap* macros (they will soon become > private). Use the ISP revision instead to identify the hardware. > > Signed-off-by: Laurent Pinchart Great, thanks for fixing this: Acked-by: Tony Lindgren -- To unsubscribe from this li

Re: omap3isp: wrong image after resizer with mt9v034 sensor

2012-09-27 Thread Laurent Pinchart
Hi Enric, On Thursday 27 September 2012 18:05:56 Enric Balletbò i Serra wrote: > 2012/9/27 Laurent Pinchart : > > On Wednesday 26 September 2012 16:15:35 Enric Balletbò i Serra wrote: > >> 2012/9/26 Laurent Pinchart : > >> > On Wednesday 26 September 2012 09:57:53 Enric Balletbò i Serra wrote: > >

Re: [PATCH v2] omap3isp: Use monotonic timestamps for statistics buffers

2012-09-27 Thread Laurent Pinchart
Hi Mauro, On Thursday 27 September 2012 22:49:34 Sakari Ailus wrote: > Mauro Carvalho Chehab wrote: > > Laurent Pinchart escreveu: > >> V4L2 buffers use the monotonic clock, while statistics buffers use wall > >> time. This makes it difficult to correlate video frames and statistics. > >> > >> S

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

2012-09-27 Thread Laurent Pinchart
Hi Sylwester, On Thursday 27 September 2012 19:18:13 Sylwester Nawrocki wrote: > On 09/27/2012 01:10 PM, Laurent Pinchart wrote: > > On Wednesday 26 September 2012 17:54:10 Sylwester Nawrocki wrote: > >> This patch adds definition of the Samsung S5C73M3 camera specific > >> image format. V4L2_PIX_

Re: tda18271 driver power consumption

2012-09-27 Thread Antti Palosaari
On 09/28/2012 01:55 AM, Michael Krufky wrote: On Thu, Sep 27, 2012 at 6:46 PM, Antti Palosaari wrote: On 09/28/2012 01:43 AM, Michael Krufky wrote: On Thu, Sep 27, 2012 at 6:26 PM, Antti Palosaari wrote: On 09/28/2012 12:58 AM, Michael Krufky wrote: On Thu, Sep 27, 2012 at 5:38 PM, Antt

Re: tda18271 driver power consumption

2012-09-27 Thread Michael Krufky
On Thu, Sep 27, 2012 at 6:46 PM, Antti Palosaari wrote: > On 09/28/2012 01:43 AM, Michael Krufky wrote: >> >> On Thu, Sep 27, 2012 at 6:26 PM, Antti Palosaari wrote: >>> >>> On 09/28/2012 12:58 AM, Michael Krufky wrote: On Thu, Sep 27, 2012 at 5:38 PM, Antti Palosaari wrote: >

[PATCH] mt9v022: support required register settings in snapshot mode

2012-09-27 Thread Anatolij Gustschin
Some camera systems cannot operate mt9v022 in normal mode and use only the snapshot mode. The TechNote for mt9v022 (TN0960) and mt9v024 (TN-09-225) describes required register settings when configuring the snapshot operation. The snapshot mode requires that certain automatic functions of the image

Re: tda18271 driver power consumption

2012-09-27 Thread Antti Palosaari
On 09/28/2012 01:43 AM, Michael Krufky wrote: On Thu, Sep 27, 2012 at 6:26 PM, Antti Palosaari wrote: On 09/28/2012 12:58 AM, Michael Krufky wrote: On Thu, Sep 27, 2012 at 5:38 PM, Antti Palosaari wrote: On 09/28/2012 12:20 AM, Michael Krufky wrote: On Thu, Sep 27, 2012 at 3:59 PM, Antt

Re: tda18271 driver power consumption

2012-09-27 Thread Michael Krufky
On Thu, Sep 27, 2012 at 6:26 PM, Antti Palosaari wrote: > On 09/28/2012 12:58 AM, Michael Krufky wrote: >> >> On Thu, Sep 27, 2012 at 5:38 PM, Antti Palosaari wrote: >>> >>> On 09/28/2012 12:20 AM, Michael Krufky wrote: On Thu, Sep 27, 2012 at 3:59 PM, Antti Palosaari wrote: >

Re: tda18271 driver power consumption

2012-09-27 Thread Antti Palosaari
On 09/28/2012 12:58 AM, Michael Krufky wrote: On Thu, Sep 27, 2012 at 5:38 PM, Antti Palosaari wrote: On 09/28/2012 12:20 AM, Michael Krufky wrote: On Thu, Sep 27, 2012 at 3:59 PM, Antti Palosaari wrote: On 09/27/2012 10:19 PM, Mauro Carvalho Chehab wrote: Em Thu, 26 Jul 2012 08:48:58 -

[GIT PULL FOR v3.7] all the rest patches!

2012-09-27 Thread Antti Palosaari
Mauro, New attempt. I really want that "PCTV 520e workaround for DRX-K fw loading" in too or find out other fix quickly. I have answered too many bug reports according to it currently. Will take debugs now... regards Antti The following changes since commit 8928b6d1568eb9104cc9e2e6627d708643

[PATCH v2 3/3] mt9v022: set y_skip_top field to zero as default

2012-09-27 Thread Anatolij Gustschin
Set "y_skip_top" to zero and revise comment as I do not see this line corruption on two different mt9v022 setups. The first read-out line is perfectly fine. Add mt9v022 platform data configuring y_skip_top for platforms that have issues with the first read-out line. Set y_skip_top to 1 for pcm990 b

[PATCH v2 2/3] mt9v022: fix the V4L2_CID_EXPOSURE control

2012-09-27 Thread Anatolij Gustschin
Since the MT9V022_TOTAL_SHUTTER_WIDTH register is controlled in manual mode by V4L2_CID_EXPOSURE control, it shouldn't be written directly in mt9v022_s_crop(). In manual mode this register should be set to the V4L2_CID_EXPOSURE control value. Changing this register directly and outside of the actua

[PATCH v2 1/3] mt9v022: add v4l2 controls for blanking

2012-09-27 Thread Anatolij Gustschin
Add controls for horizontal and vertical blanking. Also add an error message for case that the control handler init failed. Since setting the blanking registers is done by controls now, we shouldn't change these registers outside of the control function. Use v4l2_ctrl_s_ctrl() to set them. Signed-

Re: ISPsupport

2012-09-27 Thread Gary Thomas
On 2012-09-27 15:32, John Tobias wrote: Hi Laurent, I am using Pandaboard ES. I got it working now, although I am still trying to figure out how to use the pipe, resizer, uyvy output instead of SGRBG10. Do you have some example for creating a pipe/link in media-ctl from SGRBG10 to UYVY?. You c

Re: tda18271 driver power consumption

2012-09-27 Thread Michael Krufky
On Thu, Sep 27, 2012 at 5:38 PM, Antti Palosaari wrote: > On 09/28/2012 12:20 AM, Michael Krufky wrote: >> >> On Thu, Sep 27, 2012 at 3:59 PM, Antti Palosaari wrote: >>> >>> On 09/27/2012 10:19 PM, Mauro Carvalho Chehab wrote: Em Thu, 26 Jul 2012 08:48:58 -0400 Michael Krufky

Re: tda18271 driver power consumption

2012-09-27 Thread Antti Palosaari
On 09/28/2012 12:20 AM, Michael Krufky wrote: On Thu, Sep 27, 2012 at 3:59 PM, Antti Palosaari wrote: On 09/27/2012 10:19 PM, Mauro Carvalho Chehab wrote: Em Thu, 26 Jul 2012 08:48:58 -0400 Michael Krufky escreveu: Antti, This small patch should do the trick -- can you test it? The foll

Re: ISPsupport

2012-09-27 Thread John Tobias
Hi Laurent, I am using Pandaboard ES. I got it working now, although I am still trying to figure out how to use the pipe, resizer, uyvy output instead of SGRBG10. Do you have some example for creating a pipe/link in media-ctl from SGRBG10 to UYVY?. Regards, John On Wed, Sep 26, 2012 at 4:33 A

UK Blackhill Transmitter

2012-09-27 Thread Alistair Fitchie
Updated transmitter and channel information for UK Blackhill transmitter (Central Scotland). Post Digital Switch Over Day. # kaffeine channels.dvb automatically generated by w_scan # see (http://wirbel.htpc-forum.de/w_scan/index2.html) # TV(C)/RA(C)|name|vpid|apids|ttpid|sid|tsid|{S/C/T/A}|freq|

Re: tda18271 driver power consumption

2012-09-27 Thread Michael Krufky
On Thu, Sep 27, 2012 at 3:59 PM, Antti Palosaari wrote: > On 09/27/2012 10:19 PM, Mauro Carvalho Chehab wrote: >> >> Em Thu, 26 Jul 2012 08:48:58 -0400 >> Michael Krufky escreveu: >> >>> Antti, >>> >>> This small patch should do the trick -- can you test it? >>> >>> >>> The following changes sinc

Re: [PATCH 1/3] mt9v022: add v4l2 controls for blanking and other register settings

2012-09-27 Thread Anatolij Gustschin
Hi Guennadi, On Tue, 11 Sep 2012 10:24:23 +0200 (CEST) Guennadi Liakhovetski wrote: > Hi Anatolij > > On Tue, 28 Aug 2012, Anatolij Gustschin wrote: > > > Hi Guennadi, > > > > On Fri, 24 Aug 2012 23:23:37 +0200 (CEST) > > Guennadi Liakhovetski wrote: > > ... > > > > Every time the sensor is

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

2012-09-27 Thread Sakari Ailus
Hi Laurent, Thanks for the review. Laurent Pinchart wrote: On Thursday 27 September 2012 00:50:36 Sakari Ailus wrote: 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 bo

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

2012-09-27 Thread Sakari Ailus
Hi Laurent, Thanks for the review! Laurent Pinchart wrote: Hi Sakari, Thanks for the patch. On Thursday 27 September 2012 00:50:35 Sakari Ailus wrote: 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

Re: tda18271 driver power consumption

2012-09-27 Thread Antti Palosaari
On 09/27/2012 10:19 PM, Mauro Carvalho Chehab wrote: Em Thu, 26 Jul 2012 08:48:58 -0400 Michael Krufky escreveu: Antti, This small patch should do the trick -- can you test it? The following changes since commit 0c7d5a6da75caecc677be1fda207b7578936770d: Linux 3.5-rc5 (2012-07-03 22:57:4

Re: [PATCH 2/3] iommu/omap: Merge iommu2.h into iommu.h

2012-09-27 Thread Tony Lindgren
* Tony Lindgren [120927 12:54]: > Hi Ido, > > * Ido Yariv [120920 22:15]: > > Since iommu is not currently supported on OMAP1, merge plat/iommu2.h into > > iommu.h so only one file would have to move to platform_data/ as part of the > > single zImage effort. > > Looks like you need patch 2.5/3

Re: [PATCH 2/3] iommu/omap: Merge iommu2.h into iommu.h

2012-09-27 Thread Tony Lindgren
Hi Ido, * Ido Yariv [120920 22:15]: > Since iommu is not currently supported on OMAP1, merge plat/iommu2.h into > iommu.h so only one file would have to move to platform_data/ as part of the > single zImage effort. Looks like you need patch 2.5/3 in this series too that makes some of the things

Re: [PATCH v2] omap3isp: Use monotonic timestamps for statistics buffers

2012-09-27 Thread Sakari Ailus
Hi Mauro, Mauro Carvalho Chehab wrote: Em Fri, 14 Sep 2012 23:57:48 +0200 Laurent Pinchart escreveu: V4L2 buffers use the monotonic clock, while statistics buffers use wall time. This makes it difficult to correlate video frames and statistics. Switch statistics buffers to the monotonic cloc

Re: [PATCH] usb id addition for Terratec Cinergy T Stick Dual rev. 2

2012-09-27 Thread Antti Palosaari
On 09/27/2012 10:06 PM, Damien Bally wrote: This patch adds support for new Terratec Cinergy T Stick Dual rev. 2. Signed-off-by: Damien Bally I will NACK that initially because that USB ID already used by AF9015 driver. You have to explain / study what happens when AF9015 driver claims that

Re: qv4l2-bug / libv4lconvert API issue

2012-09-27 Thread Hans de Goede
Hi, On 09/27/2012 03:20 PM, Frank Schäfer wrote: What you've found is a qv4l2 bug (do you have the latest version?) Of course, I'm using the latest developer version. Even if this is just a qv4l2-bug: how do you want to fix it without removing the format selction feature ? Well, if qv4l2

Re: [PATCH 3/3] arm: omap: Move iommu/iovmm headers to platform_data

2012-09-27 Thread Mauro Carvalho Chehab
Em Fri, 21 Sep 2012 01:14:08 -0400 Ido Yariv escreveu: > Move iommu/iovmm headers from plat/ to platform_data/ as part of the > single zImage work. > > Signed-off-by: Ido Yariv > --- > arch/arm/mach-omap2/devices.c | 2 +- > arch/arm/mach-omap2/iommu2.c

Re: tda18271 driver power consumption

2012-09-27 Thread Mauro Carvalho Chehab
Em Thu, 26 Jul 2012 08:48:58 -0400 Michael Krufky escreveu: > Antti, > > This small patch should do the trick -- can you test it? > > > The following changes since commit 0c7d5a6da75caecc677be1fda207b7578936770d: > > Linux 3.5-rc5 (2012-07-03 22:57:41 +0300) > > are available in the git re

Re: cron job: media_tree daily build: ERRORS

2012-09-27 Thread Sylwester Nawrocki
Hi, On 09/27/2012 08:49 PM, Hans Verkuil wrote: > 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:Thu Sep 27 19:00:23 CEST 2012 > git hash:058fef68ff0814

[PATCH] usb id addition for Terratec Cinergy T Stick Dual rev. 2

2012-09-27 Thread Damien Bally
This patch adds support for new Terratec Cinergy T Stick Dual rev. 2. Signed-off-by: Damien Bally --- drivers/media/usb/dvb-usb-v2/it913x.c.orig 2012-09-17 19:04:59.819197365 +0200 +++ drivers/media/usb/dvb-usb-v2/it913x.c 2012-09-17 19:07:27.314015768 +0200 @@ -773,6 +773,9 @@ static const str

Re: [PATCH v2 27/34] media: mx2_camera: remove cpu_is_xxx by using platform_device_id

2012-09-27 Thread Mauro Carvalho Chehab
Em Thu, 20 Sep 2012 14:45:40 +0800 Shawn Guo escreveu: > It changes the driver to use platform_device_id rather than cpu_is_xxx > to determine the controller type, and updates the platform code > accordingly. > > As the result, mach/hardware.h inclusion gets removed from the driver. > > Signed-

cron job: media_tree daily build: ERRORS

2012-09-27 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:Thu Sep 27 19:00:23 CEST 2012 git hash:058fef68ff081401f6325a76a37356c2584ea029 gcc version: i686-linux-gcc (GC

Re: qv4l2-bug / libv4lconvert API issue

2012-09-27 Thread Frank Schäfer
Am 27.09.2012 16:47, schrieb Hans Verkuil: > On Thu 27 September 2012 15:20:21 Frank Schäfer wrote: >> Hi, >> >> Am 27.09.2012 12:26, schrieb Hans de Goede: >>> Hi, >>> >>> On 09/26/2012 11:04 PM, Frank Schäfer wrote: Hi, I've noticed the following issues/bugs while playing with qv4l

Re: [PATCH 4/6] em28xx: implement FE set_lna() callback

2012-09-27 Thread Mauro Carvalho Chehab
Em Thu, 27 Sep 2012 14:41:25 -0300 Mauro Carvalho Chehab escreveu: > Em Fri, 17 Aug 2012 04:35:08 +0300 > Antti Palosaari escreveu: > > > Make LNA run-time switching possible for PCTV nanoStick T2 290e! > > > > Signed-off-by: Antti Palosaari > > --- > > drivers/media/usb/em28xx/em28xx-dvb.c

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

2012-09-27 Thread Sylwester Nawrocki
Hi Sakari, On 09/26/2012 10:49 PM, Sakari Ailus 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 pass it to the user space where it's needed. > It'd also > play nicel

Re: [PATCH] em28xx: PCTV 520e workaround for DRX-K fw loading

2012-09-27 Thread Mauro Carvalho Chehab
Em Thu, 20 Sep 2012 19:06:04 +0300 Antti Palosaari escreveu: > This is hack to make device working again. > Looks like we need to wait DRX-K fw loading is ready until tuner > is attached as tuner is behind demod I2C bus. > > For some reason it still crash when device is re-plugged without > modu

Re: [PATCH 4/6] em28xx: implement FE set_lna() callback

2012-09-27 Thread Mauro Carvalho Chehab
Em Fri, 17 Aug 2012 04:35:08 +0300 Antti Palosaari escreveu: > Make LNA run-time switching possible for PCTV nanoStick T2 290e! > > Signed-off-by: Antti Palosaari > --- > drivers/media/usb/em28xx/em28xx-dvb.c | 40 > ++- > 1 file changed, 35 insertions(+), 5 de

Hauppauge HVR 900 H (usb-id 2040:6600) - error: only half picture visible

2012-09-27 Thread Dr. Thomas Bastian
Hi, with the above card, I see only half of the picture: http://www.flickr.com/photos/23627029@N02/8030217620/in/photostream/lightbox/ Below you find some information out of /var/log/messages the xawtv- and the mplayer-output. What could I try to make it work? Thanks in advance -- Felix cat /

Re: [PATCH 08/14] media: soc-camera: use managed devm_regulator_bulk_get()

2012-09-27 Thread Sachin Kamat
Hi Guennadi, On 27 September 2012 19:37, Guennadi Liakhovetski wrote: > Using device-managed devm_regulator_bulk_get() eliminates the need to > release regulators explicitly. > > Signed-off-by: Guennadi Liakhovetski > --- > drivers/media/platform/soc_camera/soc_camera.c |8 ++-- > 1 fil

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

2012-09-27 Thread Sylwester Nawrocki
Hi Laurent, On 09/27/2012 12:29 PM, Laurent Pinchart wrote: > On Thursday 27 September 2012 00:00:20 Sylwester Nawrocki wrote: >> On 09/26/2012 10:32 PM, Mauro Carvalho Chehab wrote: >>> Em Thu, 13 Sep 2012 12:02:14 +0100 Sangwook Lee escreveu: This patch adds driver for S5K4ECGX sensor with

Re: tda18271 driver power consumption

2012-09-27 Thread Antti Palosaari
On 09/22/2012 08:21 PM, Antti Palosaari wrote: On 09/20/2012 08:49 PM, Michael Krufky wrote: On Thu, Sep 20, 2012 at 1:47 PM, Michael Krufky wrote: On Mon, Aug 6, 2012 at 3:13 PM, Antti Palosaari wrote: On 08/06/2012 09:57 PM, Michael Krufky wrote: On Mon, Aug 6, 2012 at 2:35 PM, Devin Hei

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

2012-09-27 Thread Sylwester Nawrocki
Hi Laurent, On 09/27/2012 01:10 PM, Laurent Pinchart wrote: > On Wednesday 26 September 2012 17:54:10 Sylwester Nawrocki wrote: >> 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 inter

[GIT PULL FOR v3.7] all the rest patches!

2012-09-27 Thread Antti Palosaari
Mauro, I just rebased my local tree to your latest tree and here is all the rest patch which are waiting. regards Antti The following changes since commit 058fef68ff081401f6325a76a37356c2584ea029: [media] radio-si470x: convert struct i2c_msg initialization to C99 format (2012-09-27 07:40:

Re: [PATCH v2] omap3isp: Use monotonic timestamps for statistics buffers

2012-09-27 Thread Mauro Carvalho Chehab
Em Fri, 14 Sep 2012 23:57:48 +0200 Laurent Pinchart escreveu: > V4L2 buffers use the monotonic clock, while statistics buffers use wall > time. This makes it difficult to correlate video frames and statistics. > > Switch statistics buffers to the monotonic clock to fix this, and > replace struct

RE: [PATCH v6 6/6] [media] s5p-mfc: Update MFC v4l2 driver to support MFC6.x

2012-09-27 Thread Kamil Debski
Hi Arun, Please find my comment inline. It seems that we're very close to an ACK :) PS - you missed me in CC when posting this patch (6/6). Best wishes, -- Kamil Debski Linux Platform Group Samsung Poland R&D Center > Subject: [PATCH v6 6/6] [media] s5p-mfc: Update MFC v4l2 driver to support >

Re: omap3isp: wrong image after resizer with mt9v034 sensor

2012-09-27 Thread Enric Balletbò i Serra
Hi Laurent, 2012/9/27 Laurent Pinchart : > Hi Enric, > > On Wednesday 26 September 2012 16:15:35 Enric Balletbò i Serra wrote: >> 2012/9/26 Laurent Pinchart : >> > On Wednesday 26 September 2012 09:57:53 Enric Balletbò i Serra wrote: >> > >> > [snip] >> > >> >> You had reason. Checking the data li

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

2012-09-27 Thread Javier Martin
Some bridge drivers captures pixels during blanking periods if pixclk is enabled. In order to avoid capturing bogus data we need to disable pixclk in the sensor during those blanking periods. Acked-by: Jonathan Corbet Signed-off-by: Javier Martin --- Changes since v1: - Added changelog. --- d

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

2012-09-27 Thread Javier Martin
For a frame rate of 30 fps a pixclk of 24MHz is needed. For those cases where the ov7670 has a clean 24MHz input (xvclk) the PLL can be bypassed. This will result in a value of clkrc of 1, which means that in practice pixclk = xvclk (input clock) Acked-by: Jonathan Corbet Signed-off-by: Javier M

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

2012-09-27 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. Acked-by: Jonathan Corbet Signed-off-by: Javier Martin --- drivers/media

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

2012-09-27 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 wi

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

2012-09-27 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 --- Changes since v1: -

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

2012-09-27 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 i

Re: [PULL] soc-camera 3.7-rc7 fixes

2012-09-27 Thread Guennadi Liakhovetski
Mauro, that should have been "3.6-rc7" of course... Please don't delay until 3.7-rc7;-) I think, I'll rename the branch _after_ you pull it. Thanks Guennadi On Tue, 25 Sep 2012, Guennadi Liakhovetski wrote: > Hi Mauro > > There go the 3 fixes, that popped up (at least in my INBOX;-)) last > w

Re: [PATCHv8 21/26] v4l: vb2-dma-contig: add reference counting for a device from allocator context

2012-09-27 Thread Tomasz Stanislawski
Hi Laurent, On 08/15/2012 10:04 PM, Laurent Pinchart wrote: > Hi Tomasz, > > Thanks for the patch. > > On Tuesday 14 August 2012 17:34:51 Tomasz Stanislawski wrote: >> This patch adds taking reference to the device for MMAP buffers. >> >> Such buffers, may be exported using DMABUF mechanism. If

[PATCH 2/6] uvcvideo: Return -EACCES when trying to access a read/write-only control

2012-09-27 Thread Laurent Pinchart
The proper return code according to the V4L2 specification is -EACCES, not -EINVAL. Signed-off-by: Laurent Pinchart --- drivers/media/usb/uvc/uvc_ctrl.c |4 ++-- 1 files changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/media/usb/uvc/uvc_ctrl.c b/drivers/media/usb/uvc/uvc_ctrl.c

[PATCH 6/6] uvcvideo: Add VIDIOC_[GS]_PRIORITY support

2012-09-27 Thread Laurent Pinchart
Signed-off-by: Laurent Pinchart --- drivers/media/usb/uvc/uvc_driver.c |3 ++ drivers/media/usb/uvc/uvc_v4l2.c | 45 drivers/media/usb/uvc/uvcvideo.h |1 + 3 files changed, 49 insertions(+), 0 deletions(-) diff --git a/drivers/media/usb/uvc/uvc_

[PATCH 5/6] uvcvideo: Return -ENOTTY for unsupported ioctls

2012-09-27 Thread Laurent Pinchart
The proper error code is -ENOTTY instead of -EINVAL. This issue was found by the v4l2-compliance tool. Signed-off-by: Laurent Pinchart --- drivers/media/usb/uvc/uvc_v4l2.c |4 ++-- 1 files changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/media/usb/uvc/uvc_v4l2.c b/drivers/medi

[PATCH 4/6] uvcvideo: Set device_caps in VIDIOC_QUERYCAP

2012-09-27 Thread Laurent Pinchart
Set the capabilities field to global capabilities, and the device_caps field to the video node capabilities. This issue was found by the v4l2-compliance tool. Signed-off-by: Laurent Pinchart --- drivers/media/usb/uvc/uvc_driver.c |5 + drivers/media/usb/uvc/uvc_v4l2.c | 10 ++---

[PATCH 3/6] uvcvideo: Don't fail when an unsupported format is requested

2012-09-27 Thread Laurent Pinchart
Return the default format in that case. This issue was found by the v4l2-compliance tool. Signed-off-by: Laurent Pinchart --- drivers/media/usb/uvc/uvc_v4l2.c | 11 ++- drivers/media/usb/uvc/uvc_video.c |1 + drivers/media/usb/uvc/uvcvideo.h |1 + 3 files changed, 8 insertio

[PATCH 1/6] uvcvideo: Set error_idx properly for extended controls API failures

2012-09-27 Thread Laurent Pinchart
When one of the requested controls doesn't exist the error_idx field must reflect that situation. For G_EXT_CTRLS and S_EXT_CTRLS, error_idx must be set to the control count. For TRY_EXT_CTRLS, it must be set to the index of the unexisting control. This issue was found by the v4l2-compliance tool.

[PATCH 0/6] uvcvideo: V4L2 compliance fixes

2012-09-27 Thread Laurent Pinchart
Hi everybody, Here are 6 patches that fix V4L2 compliance issues in the uvcvideo driver found by the v4l2-compliance tool. I'm working on the last remaining issue (control classes not implemented). Laurent Pinchart (6): uvcvideo: Set error_idx properly for extended controls API failures uvcv

[PATCH] uvcvideo: Mark first output terminal as default video node

2012-09-27 Thread Laurent Pinchart
The MEDIA_ENT_FL_DEFAULT identifies entities that are considered as default for their group and type. For video nodes, it identifies the node that application should use by default. UVC devices don't report which output terminal should be considered as the default, use the first one. Most devices

[PATCH] uvcvideo: Fix control value clamping for unsigned integer controls

2012-09-27 Thread Laurent Pinchart
V4L2 integer controls are stored in signed 32-bit values. However, UVC controls can be either signed or unsigned. Take the UVC control signedness into account when clamping the control value to the min-max range. Signed-off-by: Laurent Pinchart --- drivers/media/usb/uvc/uvc_ctrl.c |8 ++-

[PATCH 03/14] OF: make a function pointer argument const

2012-09-27 Thread Guennadi Liakhovetski
The "struct device_node *" argument of of_parse_phandle_*() can be const. Signed-off-by: Guennadi Liakhovetski --- drivers/of/base.c |4 ++-- include/linux/of.h |6 +++--- 2 files changed, 5 insertions(+), 5 deletions(-) diff --git a/drivers/of/base.c b/drivers/of/base.c index d4a1c9a.

[PATCH 02/14] of: add a dummy inline function for when CONFIG_OF is not defined

2012-09-27 Thread Guennadi Liakhovetski
If CONFIG_OF isn't defined, no declaration of of_get_parent will be found and compilation can fail. This patch adds a dummy inline function definition to fix the problem. Signed-off-by: Guennadi Liakhovetski --- include/linux/of.h |5 + 1 files changed, 5 insertions(+), 0 deletions(-) d

[PATCH 01/14] i2c: add dummy inline functions for when CONFIG_OF_I2C(_MODULE) isn't defined

2012-09-27 Thread Guennadi Liakhovetski
If CONFIG_OF_I2C and CONFIG_OF_I2C_MODULE are undefined no declaration of of_find_i2c_device_by_node and of_find_i2c_adapter_by_node will be available. Add dummy inline functions to avoid compilation breakage. Signed-off-by: Guennadi Liakhovetski Cc: Ben Dooks --- include/linux/of_i2c.h | 12

[PATCH 00/14] V4L2 DT support

2012-09-27 Thread Guennadi Liakhovetski
The last version of the V4L2 DT binding RFC only raised some cosmetic improvement suggestions, which are addressed in the respective patch in this series. The rest of patches add a simple V4L2 DT parser for common properties, and an implementation for soc-camera, the sh_mobile_ceu_camera host driv

[PATCH] omap3isp: Replace cpu_is_omap3630() with ISP revision check

2012-09-27 Thread Laurent Pinchart
Drivers must not rely on cpu_is_omap* macros (they will soon become private). Use the ISP revision instead to identify the hardware. Signed-off-by: Laurent Pinchart --- drivers/media/platform/omap3isp/isp.c | 25 - 1 files changed, 16 insertions(+), 9 deletions(-) diff

[PATCH v2] drivers/media: Remove unnecessary semicolon

2012-09-27 Thread Peter Senna Tschudin
Remove unnecessary semicolon And: drivers/media/dvb-frontends/stv0900_core.c: remove unnecessary whitespace before a quoted newline Found by http://coccinelle.lip6.fr/ Signed-off-by: Peter Senna Tschudin --- drivers/media/dvb-core/dvb_frontend.c | 2 +- drivers/media/dvb-frontends/a8

Update on the CEC API

2012-09-27 Thread Hans Verkuil
Hi all, During the Linux Plumbers Conference we (i.e. V4L2 and DRM developers) had a discussion on how to handle the CEC protocol that's part of the HDMI standard. The decision was made to create a CEC bus with CEC clients, each represented by a /dev/cecX device. So this is independent of the V4L

[PATCH 11/14] media: sh-mobile-ceu-camera: runtime PM suspending doesn't have to be synchronous

2012-09-27 Thread Guennadi Liakhovetski
In both error and clean up cases there is no need to wait for runtime PM to finish suspending the device. Signed-off-by: Guennadi Liakhovetski --- .../platform/soc_camera/sh_mobile_ceu_camera.c |4 ++-- 1 files changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/media/platform

[PATCH 09/14] media: mt9t112: support deferred probing

2012-09-27 Thread Guennadi Liakhovetski
If .probe() is called without our private data, this might mean, that the host driver hasn't probed yet. Defer probing in this case. Signed-off-by: Guennadi Liakhovetski --- drivers/media/i2c/soc_camera/mt9t112.c |5 - 1 files changed, 4 insertions(+), 1 deletions(-) diff --git a/driver

[PATCH 07/14] media: soc-camera: support deferred probing of clients

2012-09-27 Thread Guennadi Liakhovetski
Currently soc-camera doesn't work with independently registered I2C client devices, it has to register them itself. This patch adds support for such configurations, in which case client drivers have to request deferred probing until their host becomes available and enables the data interface. Sign

[PATCH 06/14] media: soc-camera: prepare for asynchronous client probing

2012-09-27 Thread Guennadi Liakhovetski
Currently the soc-camera core uses the fact, that client's .probe() method is run inside of soc_camera_probe(). In such a case, the core can first attach the client to a host, then trigger client's probing, check its result and detach the client from the host to allow further clients to be probed a

[PATCH 10/14] media: soc-camera: support OF cameras

2012-09-27 Thread Guennadi Liakhovetski
With OF we aren't getting platform data any more. To minimise changes we create all the missing data ourselves, including compulsory struct soc_camera_link objects. Host-client linking is now done, based on the OF data. Media bus numbers also have to be assigned dynamically. Signed-off-by: Guennad

[PATCH 04/14] media: add V4L2 DT binding documentation

2012-09-27 Thread Guennadi Liakhovetski
This patch adds a document, describing common V4L2 device tree bindings. Co-authored-by: Sylwester Nawrocki Signed-off-by: Guennadi Liakhovetski --- Documentation/devicetree/bindings/media/v4l2.txt | 162 ++ 1 files changed, 162 insertions(+), 0 deletions(-) create mode 10

[PATCH 14/14] media: sh_mobile_ceu_camera: support all standard V4L2 DT properties

2012-09-27 Thread Guennadi Liakhovetski
Additionally to the basic DT support, added to the driver in previous patches, this patch implements complete interface configuration from DT. Signed-off-by: Guennadi Liakhovetski --- .../platform/soc_camera/sh_mobile_ceu_camera.c | 126 1 files changed, 78 insertions(+

[PATCH 13/14] media: sh-mobile-ceu-driver: support max width and height in DT

2012-09-27 Thread Guennadi Liakhovetski
Some CEU implementations have non-standard (larger) maximum supported width and height values. Add two OF properties to specify them. Signed-off-by: Guennadi Liakhovetski --- .../platform/soc_camera/sh_mobile_ceu_camera.c | 19 +++ 1 files changed, 19 insertions(+), 0 delet

[PATCH 08/14] media: soc-camera: use managed devm_regulator_bulk_get()

2012-09-27 Thread Guennadi Liakhovetski
Using device-managed devm_regulator_bulk_get() eliminates the need to release regulators explicitly. Signed-off-by: Guennadi Liakhovetski --- drivers/media/platform/soc_camera/soc_camera.c |8 ++-- 1 files changed, 2 insertions(+), 6 deletions(-) diff --git a/drivers/media/platform/soc_

[PATCH 05/14] media: add a V4L2 OF parser

2012-09-27 Thread Guennadi Liakhovetski
Add a V4L2 OF parser, implementing bindings, documented in Documentation/devicetree/bindings/media/v4l2.txt. Signed-off-by: Guennadi Liakhovetski --- drivers/media/v4l2-core/Makefile |3 + drivers/media/v4l2-core/v4l2-of.c | 190 + include/media/v4l2-of.

[PATCH 12/14] media: sh-mobile-ceu-camera: add primitive OF support

2012-09-27 Thread Guennadi Liakhovetski
Add an OF hook to sh_mobile_ceu_camera.c, no properties so far. Booting with DT also requires platform data to be optional. Signed-off-by: Guennadi Liakhovetski --- .../platform/soc_camera/sh_mobile_ceu_camera.c | 34 ++-- 1 files changed, 24 insertions(+), 10 deletions(-)

Re: [PULL FOR v3.7] OMAP3 ISP patches

2012-09-27 Thread Mauro Carvalho Chehab
Hi Laurent, Em Wed, 19 Sep 2012 12:03:17 +0200 Laurent Pinchart escreveu: > Hi Mauro, > > The following changes since commit 4313902ebe33155209472215c62d2f29d117be29: > > [media] ivtv-alsa, ivtv: Connect ivtv PCM capture stream to ivtv-alsa > interface driver (2012-09-18 13:29:07 -0300) >

Re: qv4l2-bug / libv4lconvert API issue

2012-09-27 Thread Hans Verkuil
On Thu 27 September 2012 15:20:21 Frank Schäfer wrote: > Hi, > > Am 27.09.2012 12:26, schrieb Hans de Goede: > > Hi, > > > > On 09/26/2012 11:04 PM, Frank Schäfer wrote: > >> Hi, > >> > >> I've noticed the following issues/bugs while playing with qv4l: > >> 1.) with pac7302-webcams, only the RGB3

Re: qv4l2-bug / libv4lconvert API issue

2012-09-27 Thread Frank Schäfer
Am 27.09.2012 15:20, schrieb Frank Schäfer: > Hi, > > Am 27.09.2012 12:26, schrieb Hans de Goede: >> Hi, >> >> On 09/26/2012 11:04 PM, Frank Schäfer wrote: >>> Hi, >>> >>> I've noticed the following issues/bugs while playing with qv4l: >>> 1.) with pac7302-webcams, only the RGB3 (RGB24) format is w

Re: qv4l2-bug / libv4lconvert API issue

2012-09-27 Thread Frank Schäfer
Hi, Am 27.09.2012 12:26, schrieb Hans de Goede: > Hi, > > On 09/26/2012 11:04 PM, Frank Schäfer wrote: >> 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 ups

Re: [PATCH] drivers/media: Remove unnecessary semicolon

2012-09-27 Thread Peter Senna Tschudin
On Thu, Sep 27, 2012 at 1:55 PM, Prabhakar Lad wrote: > Hi Peter, > > Thanks for the patch. > > On Thu, Sep 27, 2012 at 5:00 PM, Peter Senna Tschudin > wrote: >> Remove unnecessary semicolon >> >> And: >> drivers/media/dvb-frontends/stv0900_core.c: remove unnecessary whitespace >> before a >> qu

Re: [PATCH v2] drivers/media: Remove unnecessary semicolon

2012-09-27 Thread Antti Palosaari
On 09/27/2012 02:55 PM, Peter Senna Tschudin wrote: Remove unnecessary semicolon And: drivers/media/dvb-frontends/stv0900_core.c: remove unnecessary whitespace before a quoted newline Found by http://coccinelle.lip6.fr/ Signed-off-by: Peter Senna Tschudin For my drivers a8293, af9013, af90

Re: [PATCH] drivers/media: Remove unnecessary semicolon

2012-09-27 Thread Prabhakar Lad
Hi Peter, Thanks for the patch. On Thu, Sep 27, 2012 at 5:00 PM, Peter Senna Tschudin wrote: > Remove unnecessary semicolon > > And: > drivers/media/dvb-frontends/stv0900_core.c: remove unnecessary whitespace > before a > quoted newline > > Found by http://coccinelle.lip6.fr/ > > Signed-off-by:

[PATCH v2] drivers/media: Remove unnecessary semicolon

2012-09-27 Thread Peter Senna Tschudin
Remove unnecessary semicolon And: drivers/media/dvb-frontends/stv0900_core.c: remove unnecessary whitespace before a quoted newline Found by http://coccinelle.lip6.fr/ Signed-off-by: Peter Senna Tschudin --- drivers/media/dvb-core/dvb_frontend.c | 2 +- drivers/media/dvb-frontends/a8

Re: [PATCH 4/7] rtl28xxu: move tuner probing to .read_config()

2012-09-27 Thread Antti Palosaari
On 09/27/2012 01:35 PM, Mauro Carvalho Chehab wrote: Em Mon, 17 Sep 2012 23:58:09 +0300 Antti Palosaari escreveu: Move rtl2831u tuner probing correct place. Huh? Patch 1/7 has exactly the same subject and description... what's the difference between them? Look the description. [PATCH 1/7

[PATCH] drivers/media: Remove unnecessary semicolon

2012-09-27 Thread Peter Senna Tschudin
Remove unnecessary semicolon And: drivers/media/dvb-frontends/stv0900_core.c: remove unnecessary whitespace before a quoted newline Found by http://coccinelle.lip6.fr/ Signed-off-by: Peter Senna Tschudin --- drivers/media/dvb-core/dvb_frontend.c |2 drivers/media/dvb-frontends/

Re: omap3isp: wrong image after resizer with mt9v034 sensor

2012-09-27 Thread Laurent Pinchart
Hi Enric, On Wednesday 26 September 2012 16:15:35 Enric Balletbò i Serra wrote: > 2012/9/26 Laurent Pinchart : > > 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 s

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

2012-09-27 Thread Laurent Pinchart
Hi Sylwester, Thank you for the patch. On Wednesday 26 September 2012 17:54:10 Sylwester Nawrocki wrote: > 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

Re: [RFC] Timestamps and V4L2

2012-09-27 Thread Laurent Pinchart
Hi Sakari, On Wednesday 26 September 2012 22:17:22 Sakari Ailus wrote: > 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

Re: [RFC] Timestamps and V4L2

2012-09-27 Thread Laurent Pinchart
Hi Sylwester, On Thursday 27 September 2012 00:30:49 Sylwester Nawrocki wrote: > 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

  1   2   >