Re: Keene

2013-04-14 Thread Hans Verkuil
On Fri April 12 2013 02:11:41 Antti Palosaari wrote: > 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 no

[PATCH] MAINTAINERS: change entry for davinci media driver

2013-04-14 Thread Prabhakar lad
From: Lad, Prabhakar As of now TI has no maintainer/supporter for davinci media drivers, Until it has any I'll be maintaining it. Signed-off-by: Lad, Prabhakar Cc: Sekhar Nori --- MAINTAINERS |5 ++--- 1 files changed, 2 insertions(+), 3 deletions(-) diff --git a/MAINTAINERS b/MAINTAINER

Re: [PATCH 1/2] [media] blackfin: add display support in ppi driver

2013-04-14 Thread Scott Jiang
2013/4/12 Hans Verkuil : > On Sat April 13 2013 01:52:57 Scott Jiang wrote: >> Signed-off-by: Scott Jiang > > Is it OK if I postpone these two patches for 3.11? They don't make sense > AFAICT without the new display driver, and that will definitely not make it > for 3.10. > OK. Scott -- To unsubs

[PATCH] bttv: Add CyberVision CV06

2013-04-14 Thread Ondrej Zary
Add CyberVision CV06 4-camera card (from CyberVision SV card kit): http://www.cybervision.com.tw/products-swcard_kits-sv.html There are some interesting things on the card but they're not supported: 4 LEDs, a connector with 4 IN and 4 OUT pins, RESET IN and RESET OUT connectors, a relay and Cy

Re: [PATCH] bttv: Add noname Bt848 capture card with 14MHz xtal

2013-04-14 Thread Devin Heitmueller
On Sun, Apr 14, 2013 at 5:19 PM, Ondrej Zary wrote: > I wonder why the bttv driver probes for IR chips even when the "has_remote" > flag is not set? Probably so you can add a remote via userland even if there isn't an default remote specified in the in-kernel board profile. That said, it probabl

Re: [PATCH] bttv: Add noname Bt848 capture card with 14MHz xtal

2013-04-14 Thread Ondrej Zary
On Sunday 14 April 2013 18:39:09 Ondrej Zary wrote: > Add support for noname Bt848 capture-only card (3x composite, 1x S-VHS) > with 14MHz crystal: > http://www.rainbow-software.org/images/hardware/bt848_.jpg Noticed that it takes ages to load the bttv module (about 30 seconds). If "disable_ir=1"

Re: [PATCH] dvb-apps: initial scan file de-NetAachen

2013-04-14 Thread Oliver Schinagl
On 04/12/13 18:26, Till Maas wrote: Hi, I created an initial scan file for the German provider Net Aachen (see attachment). Pushed as 495e59e8e0dbe269768aaf2ae720697ff594c1e8 in http://git.linuxtv.org/dtv-scan-tables.git Regards Till Thank you, oliver -- To unsubscribe from this list: se

Re: [PATCH RFC] em28xx: split analog part into a separate module

2013-04-14 Thread Frank Schäfer
Am 14.04.2013 20:32, schrieb Mauro Carvalho Chehab: > Now that dvb-only devices start to happen, it makes sense > to split the analog part on a separate module. > > Signed-off-by: Mauro Carvalho Chehab > --- > > This was compiled-tested only, and it is not ready yet for merging. Yes, please let's

Re: [PATCH 1/3] em28xx: give up GPIO register tracking/caching

2013-04-14 Thread Frank Schäfer
Am 13.04.2013 20:08, schrieb Mauro Carvalho Chehab: > Em Sat, 13 Apr 2013 19:46:20 +0200 > Frank Schäfer escreveu: > >> Am 13.04.2013 19:04, schrieb Mauro Carvalho Chehab: >>> Em Sat, 13 Apr 2013 17:33:28 +0200 >>> Frank Schäfer escreveu: >>> Am 13.04.2013 16:41, schrieb Mauro Carvalho Cheha

Re: [GIT PULL FOR v3.10] Camera sensors patches

2013-04-14 Thread Mauro Carvalho Chehab
Em Fri, 12 Apr 2013 11:13:06 +0200 Laurent Pinchart escreveu: > Hi Mauro, > > The following changes since commit 81e096c8ac6a064854c2157e0bf802dc4906678c: > > [media] budget: Add support for Philips Semi Sylt PCI ref. design > (2013-04-08 07:28:01 -0300) > > are available in the git reposit

Re: [PATCH 0/3] em28xx: clean up end extend the GPIO port handling

2013-04-14 Thread Antti Palosaari
On 04/14/2013 04:32 AM, Mauro Carvalho Chehab wrote: Em Sat, 13 Apr 2013 17:37:26 +0300 Antti Palosaari escreveu: On 04/13/2013 05:25 PM, Mauro Carvalho Chehab wrote: Em Sat, 13 Apr 2013 16:15:39 +0300 Antti Palosaari escreveu: On 04/13/2013 12:48 PM, Frank Schäfer wrote: Patch 1 removes

[PATCH RFC] em28xx: split analog part into a separate module

2013-04-14 Thread Mauro Carvalho Chehab
Now that dvb-only devices start to happen, it makes sense to split the analog part on a separate module. Signed-off-by: Mauro Carvalho Chehab --- This was compiled-tested only, and it is not ready yet for merging. I'm just pusing it as RFC right now to avoid the risk of losing it, as we'll be s

cron job: media_tree daily build: WARNINGS

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

[PATCH] bttv: Add noname Bt848 capture card with 14MHz xtal

2013-04-14 Thread Ondrej Zary
Add support for noname Bt848 capture-only card (3x composite, 1x S-VHS) with 14MHz crystal: http://www.rainbow-software.org/images/hardware/bt848_.jpg 14MHz PLL was not supported by bttv driver until now. Signed-off-by: Ondrej Zary diff --git a/drivers/media/pci/bt8xx/bttv-cards.c b/drivers/m

[REVIEW PATCH 12/30] cx25821: remove bogus dependencies.

2013-04-14 Thread Hans Verkuil
From: Hans Verkuil This driver doesn't use DVB, RC, cx25840 or tveeprom. Signed-off-by: Hans Verkuil --- drivers/media/pci/cx25821/Kconfig |7 +-- drivers/media/pci/cx25821/Makefile|3 --- drivers/media/pci/cx25821/cx25821-cards.c | 21 - drive

[REVIEW PATCH 26/30] cx25821: group all fmt functions together.

2013-04-14 Thread Hans Verkuil
From: Hans Verkuil No other changes, just function reordering. Signed-off-by: Hans Verkuil --- drivers/media/pci/cx25821/cx25821-video.c | 84 +++-- 1 file changed, 43 insertions(+), 41 deletions(-) diff --git a/drivers/media/pci/cx25821/cx25821-video.c b/drivers/me

[REVIEW PATCH 08/30] cx25821: remove unnecessary global devlist.

2013-04-14 Thread Hans Verkuil
From: Hans Verkuil This device list is not necessary. The kernel already has all that information, so just use that instead. Also remove a bogus refcount and some dead 'private_free' code in the alsa driver. Signed-off-by: Hans Verkuil --- drivers/media/pci/cx25821/cx25821-alsa.c | 79 +++

[REVIEW PATCH 21/30] cx25821: switch to v4l2_fh, add event and prio handling.

2013-04-14 Thread Hans Verkuil
From: Hans Verkuil It is now possible to remove cx25821_fh and replace it with v4l2_fh, which in turn makes event handling and core prio handling possible. Signed-off-by: Hans Verkuil --- drivers/media/pci/cx25821/cx25821-video.c | 193 + drivers/media/pci/cx25821/

[REVIEW PATCH 13/30] cx25821: embed video_device, clean up some kernel log spam

2013-04-14 Thread Hans Verkuil
From: Hans Verkuil Embed the video_device struct instead of allocating it. Remove some of the annoying and ugly kernel messages shown during loading and unloading of the module. Signed-off-by: Hans Verkuil --- drivers/media/pci/cx25821/cx25821-core.c | 15 +--- drivers/media/pci/cx2582

[REVIEW PATCH 22/30] cx25821: g/s/try/enum_fmt related fixes and cleanups.

2013-04-14 Thread Hans Verkuil
From: Hans Verkuil - fill in colorspace - zero priv - delete unsupported formats - fix field handling - s_std should update width/height - proper mapping of width/height to valid resolutions Signed-off-by: Hans Verkuil --- drivers/media/pci/cx25821/cx25821-video.c | 134 +++---

[REVIEW PATCH 27/30] cx25821: prepare querycap for output support.

2013-04-14 Thread Hans Verkuil
From: Hans Verkuil Signed-off-by: Hans Verkuil --- drivers/media/pci/cx25821/cx25821-video.c |4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/media/pci/cx25821/cx25821-video.c b/drivers/media/pci/cx25821/cx25821-video.c index d3cf259..dca3391 100644 --- a/driv

[REVIEW PATCH 17/30] cx25821: use core locking.

2013-04-14 Thread Hans Verkuil
From: Hans Verkuil This allows us to replace .ioctl with .unlocked_ioctl. Signed-off-by: Hans Verkuil --- drivers/media/pci/cx25821/cx25821-medusa-video.c | 44 - drivers/media/pci/cx25821/cx25821-video.c| 72 -- drivers/media/pci/cx25821/cx25821-vid

[REVIEW PATCH 11/30] cx25821: remove cropping ioctls.

2013-04-14 Thread Hans Verkuil
From: Hans Verkuil This driver does not implement cropping, so remove the cropping ioctls. Signed-off-by: Hans Verkuil --- drivers/media/pci/cx25821/cx25821-video.c | 44 - 1 file changed, 44 deletions(-) diff --git a/drivers/media/pci/cx25821/cx25821-video.c b/

[REVIEW PATCH 18/30] cx25821: remove 'type' field from cx25821_fh.

2013-04-14 Thread Hans Verkuil
From: Hans Verkuil Signed-off-by: Hans Verkuil --- drivers/media/pci/cx25821/cx25821-video.c | 56 - drivers/media/pci/cx25821/cx25821.h |1 - 2 files changed, 15 insertions(+), 42 deletions(-) diff --git a/drivers/media/pci/cx25821/cx25821-video.c b/dr

[REVIEW PATCH 09/30] cx25821: s_input didn't check for invalid input.

2013-04-14 Thread Hans Verkuil
From: Hans Verkuil The s_input implementation allowed input 1 even if that didn't exist. Signed-off-by: Hans Verkuil --- drivers/media/pci/cx25821/cx25821-video.c |4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) diff --git a/drivers/media/pci/cx25821/cx25821-video.c b/drivers/medi

[REVIEW PATCH 06/30] cx25821: fix log_status, querycap.

2013-04-14 Thread Hans Verkuil
From: Hans Verkuil log_status shouldn't print LOG STATUS lines, the core does that already. Fix querycap version number and add device_caps support. Signed-off-by: Hans Verkuil --- drivers/media/pci/cx25821/cx25821-video.c | 23 +++ 1 file changed, 11 insertions(+), 12 d

[REVIEW PATCH 07/30] cx25821: make cx25821_sram_channels const.

2013-04-14 Thread Hans Verkuil
From: Hans Verkuil And get rid of the channel0-11 external pointers and two more unused fields in cx25821.h. Signed-off-by: Hans Verkuil --- drivers/media/pci/cx25821/cx25821-alsa.c |2 +- drivers/media/pci/cx25821/cx25821-audio-upstream.c | 22 +++ drivers/media/pc

[REVIEW PATCH 16/30] cx25821: remove unnecessary debug messages.

2013-04-14 Thread Hans Verkuil
From: Hans Verkuil The v4l2 core already has support for debugging ioctls/file operations. No need to do that again. Signed-off-by: Hans Verkuil --- drivers/media/pci/cx25821/cx25821-video.c | 12 1 file changed, 12 deletions(-) diff --git a/drivers/media/pci/cx25821/cx25821-v

[REVIEW PATCH 23/30] cx25821: remove custom ioctls that duplicate v4l2 ioctls.

2013-04-14 Thread Hans Verkuil
From: Hans Verkuil No idea why these custom ioctls exist: they have perfectly normal v4l2 counterparts which are already implemented. Signed-off-by: Hans Verkuil --- drivers/media/pci/cx25821/cx25821-video.c | 128 ++--- drivers/media/pci/cx25821/cx25821-video.h |8

[REVIEW PATCH 20/30] cx25821: replace resource management functions with fh ownership.

2013-04-14 Thread Hans Verkuil
From: Hans Verkuil Just remember which filehandle is streaming instead of using complicated resource masks. After this patch we can replace cx25821_fh with v4l2_fh. Signed-off-by: Hans Verkuil --- drivers/media/pci/cx25821/cx25821-video.c | 130 - drivers/media/pc

[REVIEW PATCH 19/30] cx25821: move vidq from cx25821_fh to cx25821_channel.

2013-04-14 Thread Hans Verkuil
From: Hans Verkuil This is not a per-filehandle object, it's a per-channel object. Signed-off-by: Hans Verkuil --- drivers/media/pci/cx25821/cx25821-video.c | 235 ++--- drivers/media/pci/cx25821/cx25821.h |8 +- 2 files changed, 115 insertions(+), 128 deleti

[REVIEW PATCH 30/30] cx25821: replace custom ioctls with write()

2013-04-14 Thread Hans Verkuil
From: Hans Verkuil Ideally this should be implemented with vb2, but it'll do for now. Signed-off-by: Hans Verkuil --- drivers/media/pci/cx25821/cx25821-core.c |3 +- drivers/media/pci/cx25821/cx25821-video-upstream.c | 248 +++- drivers/media/pci/cx25821/cx25821-

[REVIEW PATCH 15/30] cx25821: remove TRUE/FALSE/STATUS_(UN)SUCCESSFUL defines.

2013-04-14 Thread Hans Verkuil
From: Hans Verkuil Signed-off-by: Hans Verkuil --- drivers/media/pci/cx25821/cx25821-core.c |2 +- drivers/media/pci/cx25821/cx25821-medusa-video.c |2 +- drivers/media/pci/cx25821/cx25821.h |5 - 3 files changed, 2 insertions(+), 7 deletions(-) diff --git

[REVIEW PATCH 28/30] cx25821: add output format ioctls.

2013-04-14 Thread Hans Verkuil
From: Hans Verkuil Signed-off-by: Hans Verkuil --- drivers/media/pci/cx25821/cx25821-video.c | 46 + 1 file changed, 46 insertions(+) diff --git a/drivers/media/pci/cx25821/cx25821-video.c b/drivers/media/pci/cx25821/cx25821-video.c index dca3391..70e33b1 100644

[REVIEW PATCH 25/30] cx25821: setup output nodes correctly.

2013-04-14 Thread Hans Verkuil
From: Hans Verkuil Drop the custom ioctls and enable the video output nodes again, this time using standard ioctls. The next step will be to provide a proper write() interface. Signed-off-by: Hans Verkuil --- drivers/media/pci/cx25821/cx25821-core.c | 30 drivers/media/pci/cx25821/cx2

[REVIEW PATCH 24/30] cx25821: remove references to subdevices that aren't there.

2013-04-14 Thread Hans Verkuil
From: Hans Verkuil This driver does not have subdevices, so why call subdev ops? After removing that it became apparent that only Composite is supported as input, so remove also any reference to other inputs. Signed-off-by: Hans Verkuil --- drivers/media/pci/cx25821/cx25821-cards.c |1 - d

[REVIEW PATCH 14/30] cx25821: convert to the control framework.

2013-04-14 Thread Hans Verkuil
From: Hans Verkuil Signed-off-by: Hans Verkuil --- drivers/media/pci/cx25821/cx25821-core.c |5 +- drivers/media/pci/cx25821/cx25821-video.c | 210 + drivers/media/pci/cx25821/cx25821.h | 10 +- 3 files changed, 41 insertions(+), 184 deletions(-) diff

[REVIEW PATCH 02/30] cx25821: the audio channel was registered as a video node.

2013-04-14 Thread Hans Verkuil
From: Hans Verkuil Skip the audio channel when registering the video nodes. This fixes a bug where that incorrectly registered 'video' node was never unregistered. Note: this bug only surfaces if the video output nodes are enabled again after the previous patch disabled them. Signed-off-by: Han

[REVIEW PATCH 10/30] cx25821: make lots of externals static.

2013-04-14 Thread Hans Verkuil
From: Hans Verkuil A lot of functions and variables were external when they really can be declared as static. Signed-off-by: Hans Verkuil --- drivers/media/pci/cx25821/cx25821-video.c | 313 ++--- drivers/media/pci/cx25821/cx25821-video.h | 74 --- drivers/media/

[REVIEW PATCH 04/30] cx25821: remove bogus radio/vbi/'video-ioctl' support.

2013-04-14 Thread Hans Verkuil
From: Hans Verkuil This device does not support radio or vbi, so remove anything referring to that. In addition, the driver created an 'video ioctl' node, which was unused and was effectively identical to the first video node. This bogus video node is now removed, leaving us with 8 video captur

[REVIEW PATCH 05/30] cx25821: remove unused fields, ioctls.

2013-04-14 Thread Hans Verkuil
From: Hans Verkuil Do some spring cleaning: - there are no board defines with tuners, so remove bogus tuner support. - tv standard handling has nothing to do with tuners, so keep that. - replace the deprecated current_norm by g_std. - querystd isn't implemented, so remove the ioctl. - remove a b

[REVIEW PATCH 01/30] cx25821: do not expose broken video output streams.

2013-04-14 Thread Hans Verkuil
From: Hans Verkuil The cx25821 driver has support for one audio output channel and two video output channels. This is implemented in a very ugly and very evil way through a custom ioctl that passes the filename of a file containing the video data, which is then read by the driver itself using vf

[REVIEW PATCH 00/30] cx25821: driver overhaul

2013-04-14 Thread Hans Verkuil
Hi all, This patch series cleans up and overhauls the cx25821 driver. The incentive for doing this are the broken video output nodes that pose a security risk (thanks to Al Viro for pointing this out). This is fixed in the first patch. Mauro, this patch should go to 3.9 and stable. All other pat

[REVIEW PATCH 03/30] cx25821: fix compiler warning

2013-04-14 Thread Hans Verkuil
From: Hans Verkuil drivers/media/pci/cx25821/cx25821-video.c: In function ‘cx25821_video_register’: drivers/media/pci/cx25821/cx25821-video.c:518:1: warning: the frame size of 1600 bytes is larger than 1024 bytes [-Wframe-larger-than=] Fixed by just making the struct video_device template stati

[PATCH] media: dmxdev: remove dvb_ringbuffer_flush() on writer side

2013-04-14 Thread Soeren Moch
In dvb_ringbuffer lock-less synchronizationof reader and writer threads is done with separateread and write pointers. Sincedvb_ringbuffer_flush() modifies the read pointer, this function must not be called from the writer thread. This patch removes the dvb_ringbuffer_flush() calls in the dmxdev r