Re: [PATCH] solo6x10: Approximate frame intervals with non-standard denominator

2013-04-11 Thread Hans Verkuil
Hi Ismael! A quick review below: On Wed April 10 2013 23:28:05 Ismael Luceno wrote: Instead of falling back to 1/25 (PAL) or 1/30 (NTSC). Signed-off-by: Ismael Luceno ismael.luc...@corp.bluecherry.net # Please enter the commit message for your changes. Lines starting # with '#' will be

RE: [REVIEWv2 PATCH 12/12] hdpvr: allow g/s_std when in legacy mode.

2013-04-11 Thread leo
Hans, The current HEAD is working for both MythTV and gstreamer! Will you be doing more work on hdpvr? Should I start looking into error handling and kmallocs? Thank you, -Leo. Original Message Subject: Re: [REVIEWv2 PATCH 12/12] hdpvr: allow g/s_std when in legacy mode.

Re: [PATCH v8 1/7] media: V4L2: add temporary clock helpers

2013-04-11 Thread Guennadi Liakhovetski
Hi Barry On Thu, 11 Apr 2013, Barry Song wrote: Hi Guennadi, Typical video devices like camera sensors require an external clock source. Many such devices cannot even access their hardware registers without a running clock. These clock sources should be controlled by their consumers.

Re: [REVIEWv2 PATCH 12/12] hdpvr: allow g/s_std when in legacy mode.

2013-04-11 Thread Hans Verkuil
On Thu 11 April 2013 09:21:45 l...@lumanate.com wrote: Hans, The current HEAD is working for both MythTV and gstreamer! Will you be doing more work on hdpvr? Should I start looking into error handling and kmallocs? No, that's it. I'll post a pull request with all this tomorrow. Feel free

Re: uvcvideo: Dropping payload (out of sync)

2013-04-11 Thread André Weidemann
On 10.04.2013 08:05, André Weidemann wrote: Hi, I ran into a problem while trying to get a Microsoft LifeCam Studio(TM) (045e:0772) to work with uvccapture on a Raspberry PI running Kernel 3.6.11 under Debian Wheezy. I started grabbing a picture with: /usr/bin/uvccapture -x1920 -y1080

Re: [PATCH v8 1/7] media: V4L2: add temporary clock helpers

2013-04-11 Thread Barry Song
2013/4/11 Guennadi Liakhovetski g.liakhovet...@gmx.de: Hi Barry On Thu, 11 Apr 2013, Barry Song wrote: Hi Guennadi, Typical video devices like camera sensors require an external clock source. Many such devices cannot even access their hardware registers without a running clock. These

Re: [PATCH v8 1/7] media: V4L2: add temporary clock helpers

2013-04-11 Thread Guennadi Liakhovetski
On Thu, 11 Apr 2013, Barry Song wrote: 2013/4/11 Guennadi Liakhovetski g.liakhovet...@gmx.de: Hi Barry On Thu, 11 Apr 2013, Barry Song wrote: Hi Guennadi, Typical video devices like camera sensors require an external clock source. Many such devices cannot even access their

Re: [PATCH v8 1/7] media: V4L2: add temporary clock helpers

2013-04-11 Thread Barry Song
2013/4/11 Guennadi Liakhovetski g.liakhovet...@gmx.de: On Thu, 11 Apr 2013, Barry Song wrote: 2013/4/11 Guennadi Liakhovetski g.liakhovet...@gmx.de: Hi Barry On Thu, 11 Apr 2013, Barry Song wrote: Hi Guennadi, Typical video devices like camera sensors require an external clock

Re: [PATCH 11/30] [media] exynos: remove unnecessary header inclusions

2013-04-11 Thread Sylwester Nawrocki
On 04/11/2013 02:13 AM, Mauro Carvalho Chehab wrote: Em Thu, 11 Apr 2013 02:04:53 +0200 Arnd Bergmann a...@arndb.de escreveu: In multiplatform configurations, we cannot include headers provided by only the exynos platform. Fortunately a number of drivers that include those headers do not

Re: [PATCH 1/2] v4l2-ctl: break down the streaming_set()

2013-04-11 Thread Hans Verkuil
On Wed 10 April 2013 12:57:05 Tzu-Jung Lee wrote: This patch breaks down the streaming_set() into smaller ones, which can be resued for supporting m2m devices. Further cleanup or consolidation can be applied with separate patches, since this one tries not to modify logics. Signed-off-by:

Exact behavior of the EOS event?

2013-04-11 Thread Hans Verkuil
Hi Kamil, Roy, When implementing eos support in v4l2-ctl I started wondering about the exact timings of that. There are two cases, polling and non-polling, and I'll explain how I do it now in v4l2-ctl. Polling case: I select for both read and exceptions. When the select returns I check for

Re: Exact behavior of the EOS event?

2013-04-11 Thread Tzu-Jung Lee
Hi Hans, On Thu, Apr 11, 2013 at 5:40 PM, Hans Verkuil hverk...@xs4all.nl wrote: Hi Kamil, Roy, When implementing eos support in v4l2-ctl I started wondering about the exact timings of that. There are two cases, polling and non-polling, and I'll explain how I do it now in v4l2-ctl.

Re: [PATCH v8 0/7] V4L2 clock and async patches and soc-camera example

2013-04-11 Thread Guennadi Liakhovetski
Hi all On Mon, 8 Apr 2013, Guennadi Liakhovetski wrote: Mostly just a re-spin of v7 with minor modifications. Guennadi Liakhovetski (7): media: V4L2: add temporary clock helpers media: V4L2: support asynchronous subdevice registration media: soc-camera: switch I2C subdevice drivers

Re: [PATCH 11/30] [media] exynos: remove unnecessary header inclusions

2013-04-11 Thread Arnd Bergmann
On Thursday 11 April 2013, Sylwester Nawrocki wrote: On 04/11/2013 02:13 AM, Mauro Carvalho Chehab wrote: Em Thu, 11 Apr 2013 02:04:53 +0200 Arnd Bergmann a...@arndb.de escreveu: In multiplatform configurations, we cannot include headers provided by only the exynos platform.

Re: [RFC][PATCH 1/2] media: vb2: return for polling if a buffer is available

2013-04-11 Thread Marek Szyprowski
Hello, On 4/1/2013 8:40 AM, Seung-Woo Kim wrote: The vb2_poll() does not need to wait next vb_buffer_done() if there is already a buffer in done_list of queue, but current vb2_poll() always waits. So done_list is checked before calling poll_wait(). Signed-off-by: Seung-Woo Kim

RE: Exact behavior of the EOS event?

2013-04-11 Thread Kamil Debski
Hi Hans, -Original Message- From: Hans Verkuil [mailto:hverk...@xs4all.nl] Sent: Thursday, April 11, 2013 11:41 AM To: k.deb...@samsung.com Cc: linux-media@vger.kernel.org; Tzu-Jung Lee Subject: Exact behavior of the EOS event? Hi Kamil, Roy, When implementing eos support in

Re: Exact behavior of the EOS event?

2013-04-11 Thread Hans Verkuil
On Thu 11 April 2013 14:39:44 Kamil Debski wrote: Hi Hans, -Original Message- From: Hans Verkuil [mailto:hverk...@xs4all.nl] Sent: Thursday, April 11, 2013 11:41 AM To: k.deb...@samsung.com Cc: linux-media@vger.kernel.org; Tzu-Jung Lee Subject: Exact behavior of the EOS

Re: Exact behavior of the EOS event?

2013-04-11 Thread Andrzej Hajda
Hi Hans, Kamil and Roy, On 11.04.2013 14:51, Hans Verkuil wrote: On Thu 11 April 2013 14:39:44 Kamil Debski wrote: Hi Hans, -Original Message- From: Hans Verkuil [mailto:hverk...@xs4all.nl] Sent: Thursday, April 11, 2013 11:41 AM To: k.deb...@samsung.com Cc:

Re: uvcvideo: Dropping payload (out of sync)

2013-04-11 Thread Devin Heitmueller
On Thu, Apr 11, 2013 at 3:56 AM, André Weidemann andre.weidem...@web.de wrote: For anyone who may also run into this problem here is a solution... It seems the problem is hardware related to the Raspberry Pi. The solution can be found here: https://github.com/raspberrypi/linux/issues/238

[RFC v2013-04-11] SDR API REQUIREMENT SPECIFICATION

2013-04-11 Thread Antti Palosaari
I added some new parameters as described. Comments are welcome - I haven't got almost any up to date. I will keep latest version of that document same old address: http://palosaari.fi/linux/kernel_sdr_api_requirement_specification.txt regards Antti LINUX KERNEL SDR API REQUIREMENT

[PATCH] v4l2-ctl: add is_compressed_format() helper

2013-04-11 Thread Tzu-Jung Lee
It is used to: bypass precalculate_bars() for OUTPUT device that takes encoded bitstreams. handle the last chunk of input file that has non-buffer-aligned size. Signed-off-by: Tzu-Jung Lee tj...@ambarella.com --- utils/v4l2-ctl/v4l2-ctl-streaming.cpp | 101

[PATCH] v4l2-ctl: add is_compressed_format() helper

2013-04-11 Thread Tzu-Jung Lee
It is used to: bypass precalculate_bars() for OUTPUT device that takes encoded bitstreams. handle the last chunk of input file that has non-buffer-aligned size. Signed-off-by: Tzu-Jung Lee tj...@ambarella.com --- utils/v4l2-ctl/v4l2-ctl-streaming.cpp | 101

[GIT PULL] s5p-mfc/exynos4-is clean up

2013-04-11 Thread Sylwester Nawrocki
Hi Mauro, This change set includes mostly some cleanup/refactoring of the Exynos4 FIMC-IS driver and refactoring to make some modules from exynos4-is easier to re-use in the future Exynos5 FIMC-IS driver. The following changes since commit 81e096c8ac6a064854c2157e0bf802dc4906678c: [media]

[PATCH] v4l2-ctl: add is_compressed_format() helper

2013-04-11 Thread Tzu-Jung Lee
It is used to: bypass precalculate_bars() for OUTPUT device that takes encoded bitstreams. handle the last chunk of input file that has non-buffer-aligned size. Signed-off-by: Tzu-Jung Lee tj...@ambarella.com --- utils/v4l2-ctl/v4l2-ctl-streaming.cpp | 132

Re: [PATCH] v4l2-ctl: add is_compressed_format() helper

2013-04-11 Thread Hans Verkuil
On Thu April 11 2013 18:54:07 Tzu-Jung Lee wrote: It is used to: bypass precalculate_bars() for OUTPUT device that takes encoded bitstreams. handle the last chunk of input file that has non-buffer-aligned size. Signed-off-by: Tzu-Jung Lee tj...@ambarella.com ---

Re: [PATCH] v4l2-ctl: add is_compressed_format() helper

2013-04-11 Thread Hans Verkuil
On Thu April 11 2013 20:07:01 Tzu-Jung Lee wrote: It is used to: bypass precalculate_bars() for OUTPUT device that takes encoded bitstreams. handle the last chunk of input file that has non-buffer-aligned size. This seems to be the third version of this patch. When you post a new

cron job: media_tree daily build: WARNINGS

2013-04-11 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 Apr 11 19:00:19 CEST 2013 git branch: test git hash: 81e096c8ac6a064854c2157e0bf802dc4906678c gcc

Re: [patch] [media] radio-si476x: check different function pointers

2013-04-11 Thread Hans Verkuil
On Wed April 10 2013 13:40:51 Dan Carpenter wrote: This is a static checker where it complains if we check for one function pointer and then call a different function on the next line. In most cases, the code does the same thing before and after this patch. For example, when -phase_diversity

Re: [patch] [media] radio-si476x: check different function pointers

2013-04-11 Thread Andrey Smirnov
On Thu, Apr 11, 2013 at 11:24 AM, Hans Verkuil hverk...@xs4all.nl wrote: On Wed April 10 2013 13:40:51 Dan Carpenter wrote: This is a static checker where it complains if we check for one function pointer and then call a different function on the next line. In most cases, the code does the

Re: [PATCH] v4l2-ctl: add is_compressed_format() helper

2013-04-11 Thread Tzu-Jung Lee
On Fri, Apr 12, 2013 at 2:09 AM, Hans Verkuil hverk...@xs4all.nl wrote: On Thu April 11 2013 20:07:01 Tzu-Jung Lee wrote: It is used to: bypass precalculate_bars() for OUTPUT device that takes encoded bitstreams. handle the last chunk of input file that has non-buffer-aligned size.

Re: [PATCH v8 1/7] media: V4L2: add temporary clock helpers

2013-04-11 Thread Mike Turquette
Quoting Barry Song (2013-04-11 01:59:28) 2013/4/11 Guennadi Liakhovetski g.liakhovet...@gmx.de: On Thu, 11 Apr 2013, Barry Song wrote: 2013/4/11 Guennadi Liakhovetski g.liakhovet...@gmx.de: Hi Barry On Thu, 11 Apr 2013, Barry Song wrote: Hi Guennadi, Typical video

[PATCH v4 1/2] v4l2-ctl: add is_compressed_format() helper

2013-04-11 Thread Tzu-Jung Lee
To bypass precalculate_bars() for OUTPUT device that takes encoded bitstreams. Signed-off-by: Tzu-Jung Lee tj...@ambarella.com --- utils/v4l2-ctl/v4l2-ctl-streaming.cpp | 40 ++- 1 file changed, 39 insertions(+), 1 deletion(-) diff --git

[PATCH v4 2/2] v4l2-ctl: handle the last chunk of input file

2013-04-11 Thread Tzu-Jung Lee
For bitstream input file, it could be non-aligned to the buffer size. In this case, we still need to QBUF it to the driver. Signed-off-by: Tzu-Jung Lee tj...@ambarella.com --- utils/v4l2-ctl/v4l2-ctl-streaming.cpp | 72 +++ 1 file changed, 57 insertions(+), 15

[PATCH] em28xx: fix snapshot button support

2013-04-11 Thread Frank Schäfer
The snapshot button support is currently broken, because module em28xx-rc is loaded only if the device has remote control support. Fix it by also loading this module if the device has a snapshot button. Signed-off-by: Frank Schäfer fschaefer@googlemail.com ---

[PATCH] em28xx: improve em2710/em2820 distinction

2013-04-11 Thread Frank Schäfer
Chip id 18 is used by the em2710 and em2820. The current code assumes that if the device is a camera, the chip is an em2710 and an em2820 otherwise. But it turned out that the em2820 is also used in camera devices. Silvercrest 1.3 MPix webcams for example are available with both chips.

Re: [PATCH v8 0/7] V4L2 clock and async patches and soc-camera example

2013-04-11 Thread Sylwester Nawrocki
Hi Guennadi, On 04/11/2013 11:59 AM, Guennadi Liakhovetski wrote: Hi all On Mon, 8 Apr 2013, Guennadi Liakhovetski wrote: Mostly just a re-spin of v7 with minor modifications. Guennadi Liakhovetski (7): media: V4L2: add temporary clock helpers media: V4L2: support asynchronous

Re: [PATCH v8 1/7] media: V4L2: add temporary clock helpers

2013-04-11 Thread Sylwester Nawrocki
Hi, On 04/11/2013 08:52 PM, Mike Turquette wrote: [...] So, you enable CFF, it provides its own clk_* implementation like clk_get_rate() etc. Now, PXA already has it defined in arch/arm/mach-pxa/clock.c. Don't think this is going to fly. agree. Hi, I came into this thread late and don't

[PATCH 1/2] adv8170: fix querystd() method for no input signal

2013-04-11 Thread Sergei Shtylyov
From: Vladimir Barinov vladimir.bari...@cogentembedded.com When the input signal is not detected querystd() method should return V4L2_STD_UNKNOWN instead of previously latched analog video standard. Signed-off-by: Vladimir Barinov vladimir.bari...@cogentembedded.com Signed-off-by: Sergei

[PATCH v2 1/2] adv7180: fix querystd() method for no input signal

2013-04-11 Thread Sergei Shtylyov
From: Vladimir Barinov vladimir.bari...@cogentembedded.com When the input signal is not detected querystd() method should return V4L2_STD_UNKNOWN instead of previously latched analog video standard. Signed-off-by: Vladimir Barinov vladimir.bari...@cogentembedded.com Signed-off-by: Sergei

[PATCH v2 2/2] adv7180: add more subdev video ops

2013-04-11 Thread Sergei Shtylyov
From: Vladimir Barinov vladimir.bari...@cogentembedded.com Add subdev video ops for ADV7180 video decoder. This makes decoder usable on the soc-camera drivers. Signed-off-by: Vladimir Barinov vladimir.bari...@cogentembedded.com Signed-off-by: Sergei Shtylyov sergei.shtyl...@cogentembedded.com

Re: [PATCH v8 1/7] media: V4L2: add temporary clock helpers

2013-04-11 Thread Laurent Pinchart
Hi Mike, On Thursday 11 April 2013 11:52:58 Mike Turquette wrote: Quoting Barry Song (2013-04-11 01:59:28) 2013/4/11 Guennadi Liakhovetski g.liakhovet...@gmx.de: On Thu, 11 Apr 2013, Barry Song wrote: 2013/4/11 Guennadi Liakhovetski g.liakhovet...@gmx.de: On Thu, 11 Apr 2013, Barry

Keene

2013-04-11 Thread Antti Palosaari
Hello Hans, That device is working very, thank you for it. Anyhow, I noticed two things. 1) it does not start transmitting just after I plug it - I have to retune it! Output says it is tuned to 95.16 MHz by default, but it is not. After I issue retune, just to same channel it starts

[PATCH] media: vb2: add length check for mmap

2013-04-11 Thread Seung-Woo Kim
The length of mmap() can be bigger than length of vb2 buffer, so it should be checked. Signed-off-by: Seung-Woo Kim sw0312@samsung.com --- drivers/media/v4l2-core/videobuf2-core.c |5 + 1 files changed, 5 insertions(+), 0 deletions(-) diff --git

[PATCH Resend 1/1] Revert of/exynos_g2d: Add Bindings for exynos G2D driver

2013-04-11 Thread Sachin Kamat
This reverts commit 09495dda6a62c74b13412a63528093910ef80edd. The description is incomplete and the location of this file is incorrect. Based on discussion with the Samsung media and DRM subsystem maintainers, the documentaion of Samsung G2D bindings has been placed at: