Re: [RFC PATCH 0/7] saa7134: improve v4l2-compliance

2013-01-28 Thread Hans Verkuil
On Mon January 28 2013 23:40:59 Ondrej Zary wrote: > On Monday 28 January 2013 11:56:59 Hans Verkuil wrote: > > On Sun January 27 2013 20:45:05 Ondrej Zary wrote: > > > Hello, > > > this patch series improves v4l2-compliance of saa7134 driver. There are > > > still some problems. Controls require c

[PATCH 1/1] [media] s5c73m3: Staticize some symbols

2013-01-28 Thread Sachin Kamat
Fixes the following sparse warnings: drivers/media/i2c/s5c73m3/s5c73m3-core.c:42:5: warning: symbol 'boot_from_rom' was not declared. Should it be static? drivers/media/i2c/s5c73m3/s5c73m3-core.c:45:5: warning: symbol 'update_fw' was not declared. Should it be static? drivers/media/i2c/s5c73m3/s5c7

Re: [PATCH 1/1] v4l: Document timestamp behaviour to correspond to reality

2013-01-28 Thread Laurent Pinchart
Hi Sakari, On Tuesday 29 January 2013 01:02:20 Sakari Ailus wrote: > On Mon, Jan 28, 2013 at 08:56:21PM +0100, Laurent Pinchart wrote: > > On Monday 28 January 2013 10:55:14 Hans Verkuil wrote: > > > On Fri January 25 2013 19:03:29 Sakari Ailus wrote: > > > > Document that monotonic timestamps are

Re: [PATCH 1/1] v4l: Document timestamp behaviour to correspond to reality

2013-01-28 Thread Sakari Ailus
Hi Laurent, On Mon, Jan 28, 2013 at 08:56:21PM +0100, Laurent Pinchart wrote: > On Monday 28 January 2013 10:55:14 Hans Verkuil wrote: > > On Fri January 25 2013 19:03:29 Sakari Ailus wrote: > > > Document that monotonic timestamps are taken after the corresponding frame > > > has been received, n

Re: [RFC PATCH 0/7] saa7134: improve v4l2-compliance

2013-01-28 Thread Ondrej Zary
On Monday 28 January 2013 11:56:59 Hans Verkuil wrote: > On Sun January 27 2013 20:45:05 Ondrej Zary wrote: > > Hello, > > this patch series improves v4l2-compliance of saa7134 driver. There are > > still some problems. Controls require conversion to control framework > > which I was unable to fini

[PATCH 1/2] [media] Documentation/media-framework.txt: fix a sentence

2013-01-28 Thread Antonio Ospite
From: Antonio Ospite Signed-off-by: Antonio Ospite --- Hi, Actually I am not 100% sure whether the old form was correct English already or not but the new one sounds better to me. Thanks, Antonio Documentation/media-framework.txt |2 +- 1 file changed, 1 insertion(+), 1 deletion(-)

[PATCH 2/2] [media] Documentation/DocBook/media/v4l/subdev-formats.xml: fix a typo

2013-01-28 Thread Antonio Ospite
From: Antonio Ospite s/lisst/lists/ Signed-off-by: Antonio Ospite --- Should it also be s/packet/packed/ on the same line? If so I'll send another patch for that, there are a couple more of these. Thanks, Antonio Documentation/DocBook/media/v4l/subdev-formats.xml |2 +- 1 file change

[PATCH 0/2] A couple documentation fixes for media stuff

2013-01-28 Thread Antonio Ospite
From: Antonio Ospite Take a look at the per-patch annotations too, please. Thanks, Antonio Antonio Ospite (2): [media] Documentation/media-framework.txt: fix a sentence [media] Documentation/DocBook/media/v4l/subdev-formats.xml: fix a typo Documentation/DocBook/media/v4l/subdev-for

cron job: media_tree daily build: WARNINGS

2013-01-28 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: Mon Jan 28 19:00:21 CET 2013 git branch: for_v3.9 git hash: a32f7d1ad3744914273c6907204c2ab3b5d496a0 gcc ve

Re: mb86a20s and cx23885

2013-01-28 Thread Alfredo Jesús Delaiti
Hi El 28/01/13 07:23, Mauro Carvalho Chehab escribió: Em Sun, 27 Jan 2013 18:48:57 -0300 Alfredo Jesús Delaiti escreveu: Hi El 27/01/13 13:16, Mauro Carvalho Chehab escribió: Em Sun, 27 Jan 2013 12:27:21 -0300 Alfredo Jesús Delaiti escreveu: Hi all I'm trying to run the digital part of t

Re: [PATCH] video: drm: exynos: Adds display-timing node parsing using video helper function

2013-01-28 Thread Sean Paul
On Mon, Jan 28, 2013 at 12:02 PM, Leela Krishna Amudala wrote: > On Mon, Jan 28, 2013 at 9:24 PM, Sean Paul wrote: >> >> On Mon, Jan 28, 2013 at 12:45 AM, Vikas Sajjan >> wrote: >> > This patch adds display-timing node parsing using video helper function >> > >> > Signed-off-by: Leela Krishna Am

Re: [PATCH 1/1] v4l: Document timestamp behaviour to correspond to reality

2013-01-28 Thread Laurent Pinchart
On Monday 28 January 2013 10:55:14 Hans Verkuil wrote: > On Fri January 25 2013 19:03:29 Sakari Ailus wrote: > > Document that monotonic timestamps are taken after the corresponding frame > > has been received, not when the reception has begun. This corresponds to > > the reality of current drivers

Re: [PATCH] video: drm: exynos: Adds display-timing node parsing using video helper function

2013-01-28 Thread Leela Krishna Amudala
On Mon, Jan 28, 2013 at 9:24 PM, Sean Paul wrote: > > On Mon, Jan 28, 2013 at 12:45 AM, Vikas Sajjan > wrote: > > This patch adds display-timing node parsing using video helper function > > > > Signed-off-by: Leela Krishna Amudala > > Signed-off-by: Vikas Sajjan > > --- > > drivers/gpu/drm/exy

Re: [PATCH] video: drm: exynos: Adds display-timing node parsing using video helper function

2013-01-28 Thread Sean Paul
On Mon, Jan 28, 2013 at 12:45 AM, Vikas Sajjan wrote: > This patch adds display-timing node parsing using video helper function > > Signed-off-by: Leela Krishna Amudala > Signed-off-by: Vikas Sajjan > --- > drivers/gpu/drm/exynos/exynos_drm_fimd.c | 35 > -- > 1 f

Re: HI

2013-01-28 Thread Alexey Klimov
Hello Igor, On Mon, Jan 28, 2013 at 3:14 AM, Igor Stamatovski wrote: > Im trying to use ADS tech instantFM music USB card. > > dmesg reports this after machine reset (USB stays on machine) > > [6.387624] USB radio driver for Si470x FM Radio Receivers, Version 1.0.10 > [6.930228] radio-si4

Re: [media] staging: go7007: fix test for V4L2_STD_SECAM

2013-01-28 Thread Hans Verkuil
On Mon January 28 2013 15:19:42 Dan Carpenter wrote: > The current test doesn't make a lot of sense. It's likely to be true, > which is what we would want in most cases. From looking at how this is > handled in other drivers, I think "&" was intended instead of "*". > It's an easy mistake to mak

Re: [PATCH 9/7] saa7134: v4l2-compliance: initialize VBI structure

2013-01-28 Thread Hans Verkuil
On Mon January 28 2013 15:11:53 Ondrej Zary wrote: > Make saa7134 driver more V4L2 compliant: clear VBI structure completely > before assigning values to make sure any reserved space is cleared > > Signed-off-by: Ondrej Zary > --- > drivers/media/pci/saa7134/saa7134-video.c |1 + > 1 files c

[media] staging: go7007: fix test for V4L2_STD_SECAM

2013-01-28 Thread Dan Carpenter
The current test doesn't make a lot of sense. It's likely to be true, which is what we would want in most cases. From looking at how this is handled in other drivers, I think "&" was intended instead of "*". It's an easy mistake to make because they are next to each other on the keyboard. Signe

Re: [PATCH 8/7] saa7134: v4l2-compliance: remove bogus g_parm

2013-01-28 Thread Hans Verkuil
On Mon January 28 2013 15:11:50 Ondrej Zary wrote: > Make saa7134 driver more V4L2 compliant: remove empty g_parm function > > Signed-off-by: Ondrej Zary Acked-by: Hans Verkuil > --- > drivers/media/pci/saa7134/saa7134-video.c |7 --- > 1 files changed, 0 insertions(+), 7 deletions(-)

[PATCH 9/7] saa7134: v4l2-compliance: initialize VBI structure

2013-01-28 Thread Ondrej Zary
Make saa7134 driver more V4L2 compliant: clear VBI structure completely before assigning values to make sure any reserved space is cleared Signed-off-by: Ondrej Zary --- drivers/media/pci/saa7134/saa7134-video.c |1 + 1 files changed, 1 insertions(+), 0 deletions(-) diff --git a/drivers/med

[PATCH 8/7] saa7134: v4l2-compliance: remove bogus g_parm

2013-01-28 Thread Ondrej Zary
Make saa7134 driver more V4L2 compliant: remove empty g_parm function Signed-off-by: Ondrej Zary --- drivers/media/pci/saa7134/saa7134-video.c |7 --- 1 files changed, 0 insertions(+), 7 deletions(-) diff --git a/drivers/media/pci/saa7134/saa7134-video.c b/drivers/media/pci/saa7134/saa

On MIPI-CSI2 YUV420 formats and V4L2 Media Bus formats

2013-01-28 Thread Antonio Ospite
Hi, looking at the MIPI Alliance Specification for Camera Serial Interface 2 (I'll call it MIPI-CSI2 from now on, the document I am referring to is mentioned at [1] and available at [2]), I see there is an YUV420 8 bit format (MIPI Data Type 0x18) specified with interleaved components in the form

Re: terratec h5 rev. 3?

2013-01-28 Thread Roland Scheidegger
Am 21.12.2012 06:38, schrieb linux-media-ow...@vger.kernel.org: > Hi, > > I've recently got a terratec h5 for dvb-c and thought it would be > supported but it looks like it's a newer revision not recognized by em28xx. > After using the new_id hack it gets recognized and using various htc > cards (

[PATCH v2] media: add support for decoder as one of media entity types

2013-01-28 Thread Prabhakar Lad
From: Manjunath Hadli A lot of SOCs including Texas Instruments Davinci family mainly use video decoders as input devices. This patch adds a flag 'MEDIA_ENT_T_V4L2_SUBDEV_DECODER' media entity type for decoder's. Along side updates the documentation for this media entity type. Signed-off-by: Man

Re: 3.8.0-rc4+ - Oops on removing WinTV-HVR-1400 expresscard TV Tuner

2013-01-28 Thread Yijing Wang
>> Then use #modprobe pciehp pciehp_force=1 pciehp_debug=1 to load pciehp >> modules. >> After above actions, enter /sys/bus/pci_express/devices/ directory and >> /sys/bus/pci/slots/ >> Some slots and pcie port devices should be there now. >> > Sorry, I've tried your suggestion, but the two direc

Re: cron job: media_tree daily build: WARNINGS

2013-01-28 Thread Hans Verkuil
On Mon January 28 2013 12:00:50 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. OK, the daily build is back in business! I've improved my scripts substantially. In particular, if you download the scri

cron job: media_tree daily build: WARNINGS

2013-01-28 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: Mon Jan 28 08:49:09 CET 2013 git branch: for_v3.9 git hash: a32f7d1ad3744914273c6907204c2ab3b5d496a0 gcc ve

Re: [PATCH 6/7] saa7134: v4l2-compliance: remove V4L2_IN_ST_NO_SYNC from enum_input

2013-01-28 Thread Hans Verkuil
On Mon January 28 2013 11:30:46 Mauro Carvalho Chehab wrote: > Em Sun, 27 Jan 2013 20:45:11 +0100 > Ondrej Zary escreveu: > > > Make saa7134 driver more V4L2 compliant: don't set bogus V4L2_IN_ST_NO_SYNC > > flag in enum_input as it's for digital video only > > > > Signed-off-by: Ondrej Zary >

Re: [RFC PATCH 0/7] saa7134: improve v4l2-compliance

2013-01-28 Thread Hans Verkuil
On Sun January 27 2013 20:45:05 Ondrej Zary wrote: > Hello, > this patch series improves v4l2-compliance of saa7134 driver. There are still > some problems. Controls require conversion to control framework which I was > unable to finish (because the driver accesses other controls and also the > fil

Re: 3.8.0-rc4+ - Oops on removing WinTV-HVR-1400 expresscard TV Tuner

2013-01-28 Thread Martin Mokrejs
Chris Clayton wrote: > Hi Yijing, > > On 01/28/13 02:40, Yijing Wang wrote: >> Hi Chris, >> Sorry for the delay reply. It seems like my reply last night was missed. >> >> From the sysinfo you provide, there are no pcie port devices under >> /sys/bus/pci_express/devices. >> Maybe because th

Re: [PATCH 7/7] saa7134: v4l2-compliance: remove bogus audio input support

2013-01-28 Thread Hans Verkuil
On Sun January 27 2013 20:45:12 Ondrej Zary wrote: > Make saa7134 driver more V4L2 compliant: remove empty g_audio and s_audio > functions and don't set audioset in enum_input > > Signed-off-by: Ondrej Zary Acked-by: Hans Verkuil > --- > drivers/media/pci/saa7134/saa7134-video.c | 30 > ---

Re: [PATCH 6/7] saa7134: v4l2-compliance: remove V4L2_IN_ST_NO_SYNC from enum_input

2013-01-28 Thread Hans Verkuil
On Sun January 27 2013 20:45:11 Ondrej Zary wrote: > Make saa7134 driver more V4L2 compliant: don't set bogus V4L2_IN_ST_NO_SYNC > flag in enum_input as it's for digital video only I think there is a lot to be said for allowing this particular flag for analog as well. There are two types of flags

Re: [PATCH 6/7] saa7134: v4l2-compliance: remove V4L2_IN_ST_NO_SYNC from enum_input

2013-01-28 Thread Mauro Carvalho Chehab
Em Sun, 27 Jan 2013 20:45:11 +0100 Ondrej Zary escreveu: > Make saa7134 driver more V4L2 compliant: don't set bogus V4L2_IN_ST_NO_SYNC > flag in enum_input as it's for digital video only > > Signed-off-by: Ondrej Zary > --- > drivers/media/pci/saa7134/saa7134-video.c |2 -- > 1 files chang

Re: mb86a20s and cx23885

2013-01-28 Thread Mauro Carvalho Chehab
Em Sun, 27 Jan 2013 18:48:57 -0300 Alfredo Jesús Delaiti escreveu: > Hi > > El 27/01/13 13:16, Mauro Carvalho Chehab escribió: > > Em Sun, 27 Jan 2013 12:27:21 -0300 > > Alfredo Jesús Delaiti escreveu: > > > >> Hi all > >> > >> I'm trying to run the digital part of the card MyGica X8507 and I n

Re: 3.8.0-rc4+ - Oops on removing WinTV-HVR-1400 expresscard TV Tuner

2013-01-28 Thread Chris Clayton
Hi Yijing, On 01/28/13 02:40, Yijing Wang wrote: Hi Chris, Sorry for the delay reply. It seems like my reply last night was missed. From the sysinfo you provide, there are no pcie port devices under /sys/bus/pci_express/devices. Maybe because there are some problems with _OSC in your lapt

Re: [PATCH 5/7] saa7134: v4l2-compliance: fix g_tuner/s_tuner

2013-01-28 Thread Hans Verkuil
On Sun January 27 2013 20:45:10 Ondrej Zary wrote: > Make saa7134 driver more V4L2 compliant: return real frequency range in > g_tuner and fail in s_tuner for non-zero tuner > > Signed-off-by: Ondrej Zary Acked-by: Hans Verkuil > --- > drivers/media/pci/saa7134/saa7134-video.c |5 - >

Re: [PATCH 4/7] saa7134: v4l2-compliance: return real frequency

2013-01-28 Thread Hans Verkuil
On Sun January 27 2013 20:45:09 Ondrej Zary wrote: > Make saa7134 driver more V4L2 compliant: don't cache frequency in > s_frequency/g_frequency but return real one instead > > Signed-off-by: Ondrej Zary Acked-by: Hans Verkuil > --- > drivers/media/pci/saa7134/saa7134-video.c |6 -- >

Re: [PATCH 3/7] saa7134: v4l2-compliance: use v4l2_fh to fix priority handling

2013-01-28 Thread Hans Verkuil
On Sun January 27 2013 20:45:08 Ondrej Zary wrote: > Make saa7134 driver more V4L2 compliant: remove broken priority handling > and use v4l2_fh instead > > Signed-off-by: Ondrej Zary Acked-by: Hans Verkuil Nice cleanup! Regards, Hans > --- > drivers/media/pci/saa7134/saa7134-core.c

Re: [PATCH 2/7] saa7134: v4l2-compliance: don't report invalid audio modes for radio

2013-01-28 Thread Hans Verkuil
On Sun January 27 2013 20:45:07 Ondrej Zary wrote: > Make saa7134 driver more V4L2 compliant: filter audio modes that came from > tuner - keep only MONO/STEREO in radio mode > > Signed-off-by: Ondrej Zary Acked-by: Hans Verkuil > --- > drivers/media/pci/saa7134/saa7134-video.c |1 + > 1 f

Re: [PATCH 1/7] saa7134: v4l2-compliance: implement V4L2_CAP_DEVICE_CAPS

2013-01-28 Thread Hans Verkuil
On Sun January 27 2013 20:45:06 Ondrej Zary wrote: > Make saa7134 driver more V4L2 compliant: implement V4L2_CAP_DEVICE_CAPS > support > and fix all capabilities problems reported by v4l2-compliance. > > Signed-off-by: Ondrej Zary > --- > drivers/media/pci/saa7134/saa7134-video.c | 55 +++

Re: [REVIEW PATCH 00/12] em28xx: ioctl fixes/clean-ups

2013-01-28 Thread Hans Verkuil
On Fri January 25 2013 18:26:50 Frank Schäfer wrote: > This patch series addresses some issues with the ioctl code of the em28xx > driver: > Patches 1-5 and 11 fix and clean up the enabling/disabling of the ioctls > depending on the device type and equipment. > Patches 6-10 remove some obsolete/u

Re: [REVIEW PATCH 11/12] em28xx: make ioctl VIDIOC_DBG_G_CHIP_IDENT available for radio devices

2013-01-28 Thread Hans Verkuil
On Fri January 25 2013 18:27:01 Frank Schäfer wrote: > Signed-off-by: Frank Schäfer > --- > drivers/media/usb/em28xx/em28xx-video.c |1 + > 1 Datei geändert, 1 Zeile hinzugefügt(+) > > diff --git a/drivers/media/usb/em28xx/em28xx-video.c > b/drivers/media/usb/em28xx/em28xx-video.c > index d

Re: partial revert of "uvcvideo: set error_idx properly"

2013-01-28 Thread Hans de Goede
Hi, On 01/27/2013 03:06 PM, Mauro Carvalho Chehab wrote: Em Fri, 25 Jan 2013 11:40:13 +0100 Hans Verkuil escreveu: On Fri January 25 2013 10:51:57 Hans de Goede wrote: Hi, On 12/25/2012 05:56 AM, Mauro Carvalho Chehab wrote: The pwc driver can currently return -ENOENT at VIDIOC_S_FMT io

Re: [PATCH 1/1] v4l: Document timestamp behaviour to correspond to reality

2013-01-28 Thread Hans Verkuil
On Fri January 25 2013 19:03:29 Sakari Ailus wrote: > Document that monotonic timestamps are taken after the corresponding frame > has been received, not when the reception has begun. This corresponds to the > reality of current drivers: the timestamp is naturally taken when the > hardware triggers

Re: [PATCH] video: drm: exynos: Adds display-timing node parsing using video helper function

2013-01-28 Thread Leela Krishna Amudala
Hello Vikas, On Mon, Jan 28, 2013 at 11:15 AM, Vikas Sajjan wrote: > This patch adds display-timing node parsing using video helper function > > Signed-off-by: Leela Krishna Amudala > Signed-off-by: Vikas Sajjan > --- > drivers/gpu/drm/exynos/exynos_drm_fimd.c | 35 > +++