Re: [PATCH v2] staging/media/rtl2832u_sdr: fix coding style problems by adding blank lines

2014-06-03 Thread Dan Carpenter
On Mon, Jun 02, 2014 at 05:38:06PM -0700, Ovidiu Toader wrote: This minor patch fixes all WARNING:SPACING style warnings in rtl2832_sdr.c The new version of the file pleases checkpatch.pl when run with --ignore LONG_LINE. Looks good. Thanks. regards, dan carpenter -- To unsubscribe

Re: Poll and empty queues

2014-06-03 Thread Hans Verkuil
Hi Nicholas, On 06/02/2014 09:47 PM, Nicolas Dufresne wrote: Hi everyone, Recently in GStreamer we notice that we where not handling the POLLERR flag at all. Though we found that what the code do, and what the doc says is slightly ambiguous. When the application did not call

Re: Is this a user discussion list anyway?

2014-06-03 Thread Hans Verkuil
On 06/02/2014 05:47 PM, Dexter Filmore wrote: Is this the place to ask questions related to linux-media? I see mostly development messages, is there a more appropriate place? If there is no dedicated mailinglist for your capture device, then this list is the fallback. So feel free to ask your

Re: [PATCH 0/2] v4l-utils: Add missing v4l2-mediabus.h header

2014-06-03 Thread Hans Verkuil
On 06/03/2014 02:44 AM, Laurent Pinchart wrote: Hello, This patch set adds the missing v4l2-mediabus.h header, required by media-ctl. Please see individual patches for details, they're pretty straightforward. Nack. The kernel headers used in v4l-utils are installed via 'make

[PATCH 1/1] v4l: subdev: Unify argument validation across IOCTLs

2014-06-03 Thread Sakari Ailus
Separate validation of different argument types. There's no reason to do this separately for every IOCTL. Signed-off-by: Sakari Ailus sakari.ai...@linux.intel.com --- drivers/media/v4l2-core/v4l2-subdev.c | 119 +- 1 file changed, 73 insertions(+), 46 deletions(-)

Re: [RFC PATCH] [media] mt9v032: Add support for mt9v022 and mt9v024

2014-06-03 Thread Philipp Zabel
Hi Laurent, Guennadi, Am Mittwoch, den 28.05.2014, 16:44 +0200 schrieb Laurent Pinchart: If you had submitted an entirely new driver for a sensor already supported by an soc-camera sensor driver, I would have told you to fix the problem on soc- camera side. As you're only expanding hardware

[PATCH v2 2/5] [media] mt9v032: register v4l2 asynchronous subdevice

2014-06-03 Thread Philipp Zabel
Add support for registering the sensor subdevice using the v4l2-async API. Signed-off-by: Philipp Zabel p.za...@pengutronix.de --- Changes since v1: - Fixed cleanup and error handling --- drivers/media/i2c/mt9v032.c | 13 - 1 file changed, 12 insertions(+), 1 deletion(-) diff --git

[PATCH v2 1/5] [media] mt9v032: reset is self clearing

2014-06-03 Thread Philipp Zabel
According to the publicly available MT9V032 data sheet, the reset bits are self clearing and the reset register always reads 0. The reset will be asserted for 15 SYSCLK cycles. Instead of writing 0 to the register, wait using ndelay. Signed-off-by: Philipp Zabel p.za...@pengutronix.de ---

[PATCH v2 3/5] [media] mt9v032: do not clear reserved bits in read mode register

2014-06-03 Thread Philipp Zabel
The read mode register bits 8 and 9 are set and marked as reserved. Don't clear them. Signed-off-by: Philipp Zabel p.za...@pengutronix.de --- Changes since v1: - Add MT9V032_READ_MODE_RESERVED #define --- drivers/media/i2c/mt9v032.c | 12 +--- 1 file changed, 9 insertions(+), 3

[PATCH v2 4/5] [media] mt9v032: add support for mt9v022 and mt9v024

2014-06-03 Thread Philipp Zabel
From the looks of it, mt9v022 and mt9v032 are very similar, as are mt9v024 and mt9v034. With minimal changes it is possible to support mt9v02[24] with the same driver. Signed-off-by: Philipp Zabel p.za...@pengutronix.de --- Changes since v1: - Removed code changes for fault pixel correction bit

[PATCH v2 0/5] mt9v032: add support for v4l2-async, mt9v02x, and regmap

2014-06-03 Thread Philipp Zabel
Hi, This series contains the fixed up mt9v032 patches I recently sent, plus an additional patch to use regmap for the i2c register access. regards Philipp Philipp Zabel (5): [media] mt9v032: reset is self clearing [media] mt9v032: register v4l2 asynchronous subdevice [media] mt9v032: do

[PATCH v2 5/5] [media] mt9v032: use regmap

2014-06-03 Thread Philipp Zabel
This switches all register accesses to use regmap. It allows to use the regmap cache, tracing, and debug register dump facilities, and removes the need to open code read-modify-writes. Signed-off-by: Philipp Zabel p.za...@pengutronix.de --- This patch was not included before. ---

Re: [PATCH 0/2] v4l-utils: Add missing v4l2-mediabus.h header

2014-06-03 Thread Laurent Pinchart
Hi Hans, On Tuesday 03 June 2014 08:52:29 Hans Verkuil wrote: On 06/03/2014 02:44 AM, Laurent Pinchart wrote: Hello, This patch set adds the missing v4l2-mediabus.h header, required by media-ctl. Please see individual patches for details, they're pretty straightforward. Nack. The

Re: [PATCH 0/2] v4l-utils: Add missing v4l2-mediabus.h header

2014-06-03 Thread Hans Verkuil
On 06/03/14 11:46, Laurent Pinchart wrote: Hi Hans, On Tuesday 03 June 2014 08:52:29 Hans Verkuil wrote: On 06/03/2014 02:44 AM, Laurent Pinchart wrote: Hello, This patch set adds the missing v4l2-mediabus.h header, required by media-ctl. Please see individual patches for details, they're

[PATCH] Update sync-with-kernel to use installed kernel headers

2014-06-03 Thread Laurent Pinchart
Kernel headers exported to userspace can contain kernel-specific statements (such as __user annotations) that are removed when installing the headers with 'make headers_install' in the kernel sources. Only those headers must be used by userspace, raw headers are private to the kernel. Update the

Re: [PATCH 0/2] v4l-utils: Add missing v4l2-mediabus.h header

2014-06-03 Thread Laurent Pinchart
Hi Hans, On Tuesday 03 June 2014 11:48:14 Hans Verkuil wrote: On 06/03/14 11:46, Laurent Pinchart wrote: On Tuesday 03 June 2014 08:52:29 Hans Verkuil wrote: On 06/03/2014 02:44 AM, Laurent Pinchart wrote: Hello, This patch set adds the missing v4l2-mediabus.h header, required by

Re: [PATCH v3 1/3] v4l: Add test pattern colour component controls

2014-06-03 Thread Laurent Pinchart
Hi Hans, On Friday 30 May 2014 14:28:16 Hans Verkuil wrote: On 05/29/2014 05:01 PM, Laurent Pinchart wrote: On Thursday 29 May 2014 17:58:59 Sakari Ailus wrote: Laurent Pinchart wrote: On Thursday 29 May 2014 17:40:46 Sakari Ailus wrote: In many cases the test pattern has selectable

Re: [PATCH v3 1/3] v4l: Add test pattern colour component controls

2014-06-03 Thread Sakari Ailus
Hi Laurent, Laurent Pinchart wrote: Hi Hans, On Friday 30 May 2014 14:28:16 Hans Verkuil wrote: On 05/29/2014 05:01 PM, Laurent Pinchart wrote: On Thursday 29 May 2014 17:58:59 Sakari Ailus wrote: Laurent Pinchart wrote: On Thursday 29 May 2014 17:40:46 Sakari Ailus wrote: In many cases

Re: [PATCH 1/3] media-ctl: libv4l2subdev: Add DV timings support

2014-06-03 Thread Sakari Ailus
Hi Laurent, On Mon, Jun 02, 2014 at 05:10:02PM +0200, Laurent Pinchart wrote: Expose the pad-level get caps, query, get and set DV timings ioctls. Signed-off-by: Laurent Pinchart laurent.pinch...@ideasonboard.com --- utils/media-ctl/libv4l2subdev.c | 72

Re: [PATCH v3 1/3] v4l: Add test pattern colour component controls

2014-06-03 Thread Hans Verkuil
On 06/03/14 14:21, Laurent Pinchart wrote: Hi Hans, On Friday 30 May 2014 14:28:16 Hans Verkuil wrote: On 05/29/2014 05:01 PM, Laurent Pinchart wrote: On Thursday 29 May 2014 17:58:59 Sakari Ailus wrote: Laurent Pinchart wrote: On Thursday 29 May 2014 17:40:46 Sakari Ailus wrote: In many

[PATCH] libv4lconvert: Fix a regression when converting from Y10B

2014-06-03 Thread Antonio Ospite
Fix a regression introduced in commit efc29f1764a30808ebf7b3e1d9bfa27b909bf641 (libv4lconvert: Reject too short source buffer before accessing it). The old code: case V4L2_PIX_FMT_Y10BPACK: ... if (result == 0 src_size (width * height * 10 / 8)) {

Re: [PATCH] libv4lconvert: Fix a regression when converting from Y10B

2014-06-03 Thread Antonio Ospite
On Tue, 3 Jun 2014 15:48:46 +0200 Antonio Ospite a...@ao2.it wrote: Fix a regression introduced in commit efc29f1764a30808ebf7b3e1d9bfa27b909bf641 (libv4lconvert: Reject too short source buffer before accessing it). The old code: case V4L2_PIX_FMT_Y10BPACK: ... if (result

Re: Poll and empty queues

2014-06-03 Thread Nicolas Dufresne
Le mardi 03 juin 2014 à 08:38 +0200, Hans Verkuil a écrit : Hi Nicholas, On 06/02/2014 09:47 PM, Nicolas Dufresne wrote: Hi everyone, Recently in GStreamer we notice that we where not handling the POLLERR flag at all. Though we found that what the code do, and what the doc says is

Re: Poll and empty queues

2014-06-03 Thread Laurent Pinchart
Hi Nicolas, On Tuesday 03 June 2014 10:37:50 Nicolas Dufresne wrote: Le mardi 03 juin 2014 à 08:38 +0200, Hans Verkuil a écrit : On 06/02/2014 09:47 PM, Nicolas Dufresne wrote: Hi everyone, Recently in GStreamer we notice that we where not handling the POLLERR flag at all. Though

Re: [PATCH 1/3] media-ctl: libv4l2subdev: Add DV timings support

2014-06-03 Thread Laurent Pinchart
Hi Sakari, On Tuesday 03 June 2014 15:32:24 Sakari Ailus wrote: On Mon, Jun 02, 2014 at 05:10:02PM +0200, Laurent Pinchart wrote: Expose the pad-level get caps, query, get and set DV timings ioctls. Signed-off-by: Laurent Pinchart laurent.pinch...@ideasonboard.com ---

Re: [PATCH 1/1] v4l: subdev: Unify argument validation across IOCTLs

2014-06-03 Thread Laurent Pinchart
Hi Sakari, Thank you for the patch. On Tuesday 03 June 2014 12:25:16 Sakari Ailus wrote: Separate validation of different argument types. There's no reason to do this separately for every IOCTL. Signed-off-by: Sakari Ailus sakari.ai...@linux.intel.com ---

[PATCH]cxd2820r: TS clock inversion in config

2014-06-03 Thread CrazyCat
TS clock inversion in config. Signed-off-by: Evgeny Plehov evgenyple...@ukr.net --- drivers/media/dvb-frontends/cxd2820r.h| 6 ++ drivers/media/dvb-frontends/cxd2820r_c.c | 1 + drivers/media/dvb-frontends/cxd2820r_t.c | 1 + drivers/media/dvb-frontends/cxd2820r_t2.c | 1 + 4 files

[PATCH] dw2102: Geniatech T220 init fixed

2014-06-03 Thread CrazyCat
Geniatech T220 init fixed - reset cmd from windows driver and fixed TS bus config for cxd2820r. Signed-off-by: Evgeny Plehov evgenyple...@ukr.net --- drivers/media/usb/dvb-usb/dw2102.c | 14 +++--- 1 file changed, 11 insertions(+), 3 deletions(-) diff --git

Re: Poll and empty queues

2014-06-03 Thread Nicolas Dufresne
Le mardi 03 juin 2014 à 18:11 +0200, Laurent Pinchart a écrit : Hi Nicolas, On Tuesday 03 June 2014 10:37:50 Nicolas Dufresne wrote: Le mardi 03 juin 2014 à 08:38 +0200, Hans Verkuil a écrit : On 06/02/2014 09:47 PM, Nicolas Dufresne wrote: Hi everyone, Recently in GStreamer

Re: [PATCH 1/3] media-ctl: libv4l2subdev: Add DV timings support

2014-06-03 Thread Sakari Ailus
Hi Laurent, Laurent Pinchart wrote: Hi Sakari, On Tuesday 03 June 2014 15:32:24 Sakari Ailus wrote: On Mon, Jun 02, 2014 at 05:10:02PM +0200, Laurent Pinchart wrote: Expose the pad-level get caps, query, get and set DV timings ioctls. Signed-off-by: Laurent Pinchart

Re: Poll and empty queues

2014-06-03 Thread Laurent Pinchart
Hi Nicolas, On Tuesday 03 June 2014 13:39:54 Nicolas Dufresne wrote: Le mardi 03 juin 2014 à 18:11 +0200, Laurent Pinchart a écrit : On Tuesday 03 June 2014 10:37:50 Nicolas Dufresne wrote: Le mardi 03 juin 2014 à 08:38 +0200, Hans Verkuil a écrit : On 06/02/2014 09:47 PM, Nicolas Dufresne

Trouble scanning either of my cards, error dvb_fe_set_parms failed (Invalid argument)

2014-06-03 Thread Brian Caine
Hi, So, I have two different tv tuner cards that I'm trying to get working. The kernel modules seem to be loading, no errors there. The /dev/dvb files are showing up. Everything has the right permissions. (Well, and I'm doing my testing as root too, just to get it working in the first place.)

cron job: media_tree daily build: OK

2014-06-03 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 Jun 4 04:00:50 CEST 2014 git branch: test git hash: 5ea878796f0a1d9649fe43a6a09df53d3915c0ef gcc