Re: [RFC,PATCH] VIDIOC_G_EXT_CTRLS does not handle NULL pointer correctly

2009-05-27 Thread Németh Márton
Laurent Pinchart worte: > Could a very large number of control requests be used as a DoS attack vector > ? > A userspace application could kmalloc large amounts of memory without any > restriction. Memory would be reclaimed eventually, but after performing a > large number of USB requests, whic

Re: [PATCH] vino: replace dma_sync_single with dma_sync_single_for_cpu

2009-05-27 Thread Jiri Slaby
On 05/28/2009 03:10 AM, FUJITA Tomonori wrote: > This replaces dma_sync_single() with dma_sync_single_for_cpu() because > dma_sync_single() is an obsolete API; include/linux/dma-mapping.h says: > > /* Backwards compat, remove in 2.7.x */ > #define dma_sync_single dma_sync_single_for_

[RESEND PATCH 3/3 v2] big rework of TS for saa7134

2009-05-27 Thread Dmitri Belimov
Hi all. 1. Add start/stop TS function. 2. Move setup DMA of TS to DMA function. 3. Write support cupture via MMAP 4. Rework startup and finish process, remove simple FSM. diff -r 8d37e8505664 linux/drivers/media/video/saa7134/saa7134-cards.c --- a/linux/drivers/media/video/saa7134/saa7134-cards.c

Re: [PATCH 3/3 ] big rework of TS for saa7134

2009-05-27 Thread Dmitri Belimov
Hi > Hi, > > Am Montag, den 11.05.2009, 21:01 +1000 schrieb Dmitri Belimov: > > Hi > > > > > Em Mon, 11 May 2009 20:21:23 +1000 > > > Dmitri Belimov escreveu: > > > > > > > > > Cheers, > > > > > > Mauro > > > > > > > > > > Did you check it is still OK for DVB-T and DVB-S also? > > > > > >

"Unknown symbol __udivdi3" with rev >= 11873

2009-05-27 Thread David Ward
Revision 11873 (committed earlier today) has broken the cx18 driver for me, with the line "cx18: Unknown symbol __udivdi3" appearing in dmesg when the module tries to load. I'm using Ubuntu 8.04.2 which uses kernel 2.6.24 and gcc 4.2.4. I also wanted to express my appreciation to Mauro for fi

[PULL] http://kernellabs.com/hg/~dheitmueller/misc-fixes-2

2009-05-27 Thread Devin Heitmueller
Hello Mauro, Please pull from http://kernellabs.com/hg/~dheitmueller/misc-fixes-2 for the following: em28xx: Don't let device work unless connected to a high speed USB port au0828: Don't let device work unless connected to a high speed USB port em28xx: Add support for the K-World 2800d tuner-core

Re: PULL request - http://linuxtv.org/hg/~pb/v4l-dvb/

2009-05-27 Thread Trent Piepho
On Tue, 26 May 2009, Patrick Boettcher wrote: > > MHz to 161 Mhz as welll as 439-447 MHz. Probably wrong. My guess is the > > data sheet says, "low band 48 to 154 MHz, mid band 161 MHz to 439 MHz, > > etc.," where 154 is the frequency of the last channel in the low band and > > 161 is the first c

[PATCH] vino: replace dma_sync_single with dma_sync_single_for_cpu

2009-05-27 Thread FUJITA Tomonori
This replaces dma_sync_single() with dma_sync_single_for_cpu() because dma_sync_single() is an obsolete API; include/linux/dma-mapping.h says: /* Backwards compat, remove in 2.7.x */ #define dma_sync_single dma_sync_single_for_cpu #define dma_sync_sg dma_sync_sg_for_cpu Signed

Re: [PULL] http://kernellabs.com/hg/~stoth/tda10048-merge/

2009-05-27 Thread Steven Toth
Mauro Carvalho Chehab wrote: Em Wed, 20 May 2009 15:42:34 -0400 Steven Toth escreveu: Mauro, Please pull from http://kernellabs.com/hg/~stoth/tda10048-merge/ Didn't work: $ ./hgimport http://kernellabs.com/hg/~stoth/tda10048-merge/ Pushing from remote site http://kernellabs.com/hg/~stoth/t

Re: USB/DVB - Old Technotrend TT-connect S-2400 regression tracked down

2009-05-27 Thread David
Alan Stern wrote: > On Wed, 27 May 2009, David wrote: > > >> Sorry for the delay, your patch reached me just after I turned in last >> night. >> >> It looks good to me. dmesg is how I'd expect, and I've attached the usb >> trace which looks pretty similar to when the original patch was reverted.

Re: Licensing question regarding SN9C2028 decompression (fwd)

2009-05-27 Thread Theodore Kilgore
Hans, Here is the answer which I got about the question of GPL->LGPL licensing in regard to the sn9c2028 decompression code. Theodore Kilgore -- Forwarded message -- Date: Wed, 27 May 2009 13:19:46 -0400 From: Harald To: Theodore Kilgore Subject: Re: Licensing question rega

Re: USB/DVB - Old Technotrend TT-connect S-2400 regression tracked down

2009-05-27 Thread Alan Stern
On Wed, 27 May 2009, David wrote: > Sorry for the delay, your patch reached me just after I turned in last > night. > > It looks good to me. dmesg is how I'd expect, and I've attached the usb > trace which looks pretty similar to when the original patch was reverted. > > I'll test some more with

Re: [PATCH][RESEND] Use correct sampling rate for TV/FM radio

2009-05-27 Thread Mauro Carvalho Chehab
Em Sat, 23 May 2009 19:09:12 +0200 Oldrich Jedlicka escreveu: > On Tuesday 19 of May 2009 at 19:54:18, Oldrich Jedlicka wrote: > > On Tuesday 14 of April 2009 at 20:48:14, Oldrich Jedlicka wrote: > > > Here is the fix for using the 32kHz sampling rate for TV and FM radio > > > (ALSA). The TV uses

[PATCH v2] Re: videodev: Unknown symbol i2c_unregister_device (in kernels older than 2.6.26)

2009-05-27 Thread Randy Dunlap
ideo. Patch below applies on top of the previous one. --- From: Randy Dunlap Fix v4l2-device usage of i2c_unregister_device() and handle the case of CONFIG_I2C=m & CONFIG_MEDIA_VIDEO=y. Signed-off-by: Randy Dunlap --- drivers/media/video/v4l2-device.c |2 +- 1 file changed, 1 ins

Re: USB/DVB - Old Technotrend TT-connect S-2400 regression tracked down

2009-05-27 Thread David
Alan Stern wrote: > On Mon, 25 May 2009, David wrote: > > > I think the idea of the patch was good, but the endpoint direction > information got lost (because the information was taken from the dummy > qTD which is always marked as OUT -- I don't see how this could ever > have worked properly).

Re: videodev: Unknown symbol i2c_unregister_device (in kernels older than 2.6.26)

2009-05-27 Thread Mauro Carvalho Chehab
Em Mon, 25 May 2009 09:10:17 +1000 Matt Doran escreveu: > Hi there, > > I tried using the latest v4l code on an Mythtv box running 2.6.20, but > the v4l videodev module fails to load with the following warnings: > > videodev: Unknown symbol i2c_unregister_device > v4l2_common: Unknown

Re: Fwd: [Linux-uvc-devel] Compilation error

2009-05-27 Thread Mauro Carvalho Chehab
Em Wed, 27 May 2009 01:57:55 +0200 Laurent Pinchart escreveu: > Hi everybody, > > -- Forwarded Message -- > > Subject: [Linux-uvc-devel] Compilation error > Date: Tuesday 26 May 2009 > From: Phil Lemelin > To: linux-uvc-de...@lists.berlios.de > > Hi uvc-devel list, > > I'm t

Re: v4l: Compile ADV7343 and THS7303 drivers for kernels >= 2.6.26

2009-05-27 Thread Mauro Carvalho Chehab
Em Wed, 27 May 2009 01:42:44 -0400 Chaithrika U S escreveu: > Signed-off-by: Chaithrika U S > --- > v4l/versions.txt |2 ++ > 1 files changed, 2 insertions(+), 0 deletions(-) > > diff --git a/v4l/versions.txt b/v4l/versions.txt > index eb206b5..3c57c14 100644 > --- a/v4l/versions.txt > +++

Re: v4l-dvb and old kernels

2009-05-27 Thread Mauro Carvalho Chehab
Em Tue, 26 May 2009 15:03:56 +0200 Antonio Beamud Montero escreveu: > It would compile today's snapshot of v4l-dvb with an old kernel version > (for example 2.6.16)? (or is better to upgrade the kernel?) > > Trying to compile today's mercurial snapshot in a SuSE 10.1 (2.6.16-21), > give the ne

[cron job] v4l-dvb daily build 2.6.22 and up: ERRORS, 2.6.16-2.6.21: ERRORS

2009-05-27 Thread Hans Verkuil
This message is generated daily by a cron job that builds v4l-dvb for the kernels and architectures in the list below. Results of the daily build of v4l-dvb: date:Wed May 27 19:00:06 CEST 2009 path:http://www.linuxtv.org/hg/v4l-dvb changeset: 11873:22fcdb8c1c36 gcc version: gcc

Probably strange bug with usb radio-mr800

2009-05-27 Thread Alexey Klimov
Hello, Doing some improvements in media/radio/radio-mr800.c i see very strange situation. I made simple patch that removed usb_amradio_open function. There is no need in this function (no need to start/set frequency radio in open procedure, and also removed lock/unlock_kernel() calls). Here is thi

Re: [RFC, PATCH] VIDIOC_G_EXT_CTRLS does not handle NULL pointer correctly

2009-05-27 Thread Hans Verkuil
> On Monday 25 May 2009 21:22:06 Trent Piepho wrote: >> On Mon, 25 May 2009, Laurent Pinchart wrote: >> > diff -r e0d881b21bc9 linux/drivers/media/video/v4l2-ioctl.c >> > --- a/linux/drivers/media/video/v4l2-ioctl.c Tue May 19 15:12:17 2009 >> > +0200 +++ b/linux/drivers/media/video/v4l2-ioc

Re: [RFC,PATCH] VIDIOC_G_EXT_CTRLS does not handle NULL pointer correctly

2009-05-27 Thread Laurent Pinchart
On Monday 25 May 2009 21:22:06 Trent Piepho wrote: > On Mon, 25 May 2009, Laurent Pinchart wrote: > > diff -r e0d881b21bc9 linux/drivers/media/video/v4l2-ioctl.c > > --- a/linux/drivers/media/video/v4l2-ioctl.cTue May 19 15:12:17 2009 > > +0200 +++ b/linux/drivers/media/video/v4l2-ioctl.c

Re: [linux-dvb] EPG (Electronic Program Guide) Tools

2009-05-27 Thread Steven Toth
Where does the EPG come from? Is it incorporated into the output stream through PID's some how or is it read from one of the other devices under adapter0? Are there simple command line tools to read it or do you have to write a custom program to interpret it somehow? Could someone please point

Re: [linux-dvb] EPG (Electronic Program Guide) Tools

2009-05-27 Thread Devin Heitmueller
On Wed, May 27, 2009 at 10:56 AM, Steven Toth wrote: > Chris, > > You'd expect to be able to get 3-4 days of advanced data to populate a guide > with, in reality you can often get as little a six hours and that's it. The > mandatory side of the spec is weak in this area. six hours is fine for Now

Re: [linux-dvb] fe status values

2009-05-27 Thread Devin Heitmueller
On Wed, May 27, 2009 at 6:59 AM, Markus Oliver Hahn wrote: > Hi there, > I was just going throug the dvbapi 5.0 > but I couldn`t find ow to interpret the > values which I get by > > > > int ioctl( int fd, int request = FE_READ_SIGNAL_STRENGTH, int16_t *strength); > > and > > int ioctl(int fd, int

Re: [PULL] http://kernellabs.com/hg/~dheitmueller/misc-fixes

2009-05-27 Thread Devin Heitmueller
On Wed, May 27, 2009 at 9:00 AM, Mauro Carvalho Chehab wrote: > Em Thu, 21 May 2009 22:26:35 -0400 > Devin Heitmueller escreveu: > >> Hello Mauro, >> >> Please pull from http://kernellabs.com/hg/~dheitmueller/misc-fixes for >> the following: >> >> em28xx: don't create the audio device if no onboa

Re: [PULL] http://kernellabs.com/hg/~dheitmueller/misc-fixes

2009-05-27 Thread Devin Heitmueller
On Wed, May 27, 2009 at 8:54 AM, Mauro Carvalho Chehab wrote: > Em Sat, 23 May 2009 03:48:21 +0200 (CEST) > BOUWSMA Barry escreveu: > >> On Thu, 21 May 2009, Devin Heitmueller wrote: >> >> > em28xx: Don't let device work unless connected to a high speed USB port >> >> (grrr, dyndns addresses that

Re: [PULL] http://kernellabs.com/hg/~dheitmueller/misc-fixes

2009-05-27 Thread Mauro Carvalho Chehab
Em Sat, 23 May 2009 03:48:21 +0200 (CEST) BOUWSMA Barry escreveu: > On Thu, 21 May 2009, Devin Heitmueller wrote: > > > em28xx: Don't let device work unless connected to a high speed USB port > > (grrr, dyndns addresses that are actually dynamically changing) > > |The em28xx basically just

Re: [PULL] http://kernellabs.com/hg/~dheitmueller/misc-fixes

2009-05-27 Thread Mauro Carvalho Chehab
Em Thu, 21 May 2009 22:26:35 -0400 Devin Heitmueller escreveu: > Hello Mauro, > > Please pull from http://kernellabs.com/hg/~dheitmueller/misc-fixes for > the following: > > em28xx: don't create the audio device if no onboard audio processor I don't think that this patch is right. There are s

Re: [PULL] http://kernellabs.com/hg/~stoth/tda10048-merge/

2009-05-27 Thread Mauro Carvalho Chehab
Em Thu, 21 May 2009 22:26:29 -0500 (CDT) Mike Isely escreveu: > > I see no issues here with the pvrusb2 part of it... > > Acked-by: Mike Isely Due to the way -hg works, I can't add your ack at -hg (otherwise, I would break Steven and mkrufky's trees). So, I'll add it at -git only Cheers, M

Re: [PATCH] i2c: Simplified CONFIG_I2C=n interface.

2009-05-27 Thread Mark Brown
On Wed, May 27, 2009 at 09:18:31AM +0200, Jean Delvare wrote: > Violent nack. Drivers which optionally use I2C are a minority. It's extremely common for devices like the CODECs and PMICs used in embedded systems to have both I2C and SPI interfaces, selectable via a pin strap at power on. It's le

i2c-compat.h and old kernel support

2009-05-27 Thread David Wong
I found that i2c-compat.h is not referred by any linux/media code and it seems code for Linux 2.4/2.5 support. What is the minimum supporting kernel of the current development tree? It looks like i2c-compat.h could be dropped. David -- To unsubscribe from this list: send the line "unsubscribe linu

OV7670 Soc-camera driver (anyone else want to merge versions?)

2009-05-27 Thread Jonathan Cameron
Hi All, The email is mainly in the interests of avoiding wasting time. I'm currently updating the driver I posted alongside Guennadi's soc-camera changes with the intention to also merge features from the 3 drivers that are out there. Unsurprisingly they all have different feature sets at the mo

Re: Image filter controls

2009-05-27 Thread Dongsoo, Nathaniel Kim
On Wed, May 27, 2009 at 8:28 PM, Guennadi Liakhovetski wrote: > On Wed, 27 May 2009, Dongsoo, Nathaniel Kim wrote: > >> Hi Guennadi, >> >> In my opinion (in totally user aspect) those CIDs you mentioned do not >> fit in the purpose. Only low-pass filtering control could be >> considered as a color

Re: Image filter controls

2009-05-27 Thread Guennadi Liakhovetski
On Wed, 27 May 2009, Dongsoo, Nathaniel Kim wrote: > Hi Guennadi, > > In my opinion (in totally user aspect) those CIDs you mentioned do not > fit in the purpose. Only low-pass filtering control could be > considered as a colorfx, even though low-pass filtering is not a color > effect it could be

Re: Image filter controls

2009-05-27 Thread Dongsoo, Nathaniel Kim
Hi Guennadi, In my opinion (in totally user aspect) those CIDs you mentioned do not fit in the purpose. Only low-pass filtering control could be considered as a colorfx, even though low-pass filtering is not a color effect it could be possible to sense the meaning of effect on it. How about making

Image filter controls

2009-05-27 Thread Guennadi Liakhovetski
Hi, I have to implement 2 filter controls: a boolean colour filter control to switch a fluorescent light band-stop filter on and off, and an also boolean control to switch a low-pass (border denoising, I think) filter. I found the following two filters in the existing API: case V4L2_CI

[PATCHv4 6 of 8] FMTx: si4713: Add files to add radio interface for si4713

2009-05-27 Thread Eduardo Valentin
# HG changeset patch # User Eduardo Valentin # Date 1243414606 -10800 # Branch export # Node ID 1abb96fdce05e1449faac2223e93056bacf389bd # Parent 94b5043b692a6218a667326536d3d76a0c591307 This patch adds files which creates the radio interface for si4713 FM transmitter devices. Signed-off-by: Edu

[PATCHv4 3 of 8] v4l2: video device: Add FMTX controls default configurations

2009-05-27 Thread Eduardo Valentin
# HG changeset patch # User Eduardo Valentin # Date 1243414606 -10800 # Branch export # Node ID 85b64a6cc67c0c0d6a795388f8ca02dad2ff9da7 # Parent ebb409d7a258df2bc7a6dcd72113584b4c0e7ce2 Signed-off-by: Eduardo Valentin --- drivers/media/linux/video/v4l2-common.c | 46

[PATCHv4 4 of 8] Add documentation description for FM Transmitter Extended Control Class

2009-05-27 Thread Eduardo Valentin
# HG changeset patch # User Eduardo Valentin # Date 1242209424 -10800 # Branch export # Node ID 2dd45d3736ae1952dfa98e65b072eb08157fb19d # Parent fadf1cddf504609cdb4889f4aa3305ca8d15323a From: Eduardo Valentin This single patch adds documentation description for FM Transmitter (FMTX) Extended

[PATCHv4 2 of 8] v4l2: video device: Add V4L2_CTRL_CLASS_FMTX controls

2009-05-27 Thread Eduardo Valentin
# HG changeset patch # User Eduardo Valentin # Date 1243414606 -10800 # Branch export # Node ID ebb409d7a258df2bc7a6dcd72113584b4c0e7ce2 # Parent 4fb354645426f8b187c2c90cd8528b2518461005 This patch adds a new class of extended controls. This class is intended to support Radio Modulators propertie

[PATCHv4 7 of 8] FMTx: si4713: Add Kconfig and Makefile entries

2009-05-27 Thread Eduardo Valentin
# HG changeset patch # User Eduardo Valentin # Date 1243414607 -10800 # Branch export # Node ID b1d98e675a3c4e9e6d247701c9ac18239e3dcc1c # Parent 1abb96fdce05e1449faac2223e93056bacf389bd Signed-off-by: Eduardo Valentin --- drivers/media/linux/radio/Kconfig | 22 ++ driver

[PATCHv4 1 of 8] v4l2_subdev i2c: Add v4l2_i2c_new_subdev_board i2c helper function

2009-05-27 Thread Eduardo Valentin
# HG changeset patch # User Eduardo Valentin # Date 1243414605 -10800 # Branch export # Node ID 4fb354645426f8b187c2c90cd8528b2518461005 # Parent 142fd6020df3b4d543068155e49a2618140efa49 Device drivers of v4l2_subdev devices may want to have board specific data. This patch adds an helper function

[PATCHv4 0 of 8] FM Transmitter (si4713) and another changes

2009-05-27 Thread Eduardo Valentin
Hello all, I'm resending the FM transmitter driver and the proposed changes in v4l2 api files in order to cover the fmtx extended controls class. It is basically the same series of version #3. However I rewrote it to add the following comments: * Check kernel version for i2c helper functio

[PATCHv4 8 of 8] FMTx: si4713: Add document file

2009-05-27 Thread Eduardo Valentin
# HG changeset patch # User Eduardo Valentin # Date 1243414607 -10800 # Branch export # Node ID fadf1cddf504609cdb4889f4aa3305ca8d15323a # Parent b1d98e675a3c4e9e6d247701c9ac18239e3dcc1c Signed-off-by: Eduardo Valentin --- Documentation/video4linux/si4713.txt | 133

Re: [linux-dvb] how to get the SNR DB value for DVICO DVB-T Dual express card

2009-05-27 Thread Antti Palosaari
On 05/26/2009 09:37 PM, Tu-Tu Yu wrote: Hello: this is the output of the status... I would like to know how to calculate the the SNR DB value. Thank you so much! The frontend they used for DVB-T DUAL express is "Zarlink zl10353 DVB-T" status 1e | signal b788 | snr 0f1f1 | ber | unc

Re: [PATCH] i2c: Simplified CONFIG_I2C=n interface.

2009-05-27 Thread Paul Mundt
On Wed, May 27, 2009 at 09:18:31AM +0200, Jean Delvare wrote: > On Wed, 27 May 2009 16:08:50 +0900, Paul Mundt wrote: > > Another day, another module-related failure due to the i2c interface > > being used in code that optionally uses it: > > > > ERROR: "i2c_new_device" [drivers/media/video/soc_ca

Re: [PATCH] i2c: Simplified CONFIG_I2C=n interface.

2009-05-27 Thread Jean Delvare
On Wed, 27 May 2009 16:08:50 +0900, Paul Mundt wrote: > Another day, another module-related failure due to the i2c interface > being used in code that optionally uses it: > > ERROR: "i2c_new_device" [drivers/media/video/soc_camera.ko] undefined! > ERROR: "i2c_get_adapter" [drivers/media/video/soc_

[PATCH] i2c: Simplified CONFIG_I2C=n interface.

2009-05-27 Thread Paul Mundt
Another day, another module-related failure due to the i2c interface being used in code that optionally uses it: ERROR: "i2c_new_device" [drivers/media/video/soc_camera.ko] undefined! ERROR: "i2c_get_adapter" [drivers/media/video/soc_camera.ko] undefined! ERROR: "i2c_put_adapter" [drivers/media/vi