Re: [RESEND PATCH 1/14] staging/media/as102: initial import from Abilis

2011-10-30 Thread Stefan Richter
On Oct 30 Piotr Chmura wrote: > > + * Note: > > + * - in AS102 SNR=MER > > + * - the SNR will be returned in linear terms, i.e. not in dB > > + * - the accuracy equals ±2dB for a SNR range from 4dB to 30dB > > + * - the accuracy is>2dB for SNR values outside this range > > + */ > > I found

Re: [RESEND PATCH 11/14] staging/media/as102: fix compile warning about unused function

2011-10-30 Thread Stefan Richter
On Oct 30 Sylwester Nawrocki wrote: > On 10/18/2011 10:03 PM, Piotr Chmura wrote: > > Patch taken from http://kernellabs.com/hg/~dheitmueller/v4l-dvb-as102-2/ [...] > > +#if (LINUX_VERSION_CODE< KERNEL_VERSION(2, 6, 19)) > > I was wondering, could such a conditional compilation be simply skipped

Re: Problems cloning the git repostories

2011-10-30 Thread Marek Vasut
> On Sun, Sep 25, 2011 at 8:33 AM, Patrick Dickey wrote: > > Hello there, > > > > I tried to follow the steps for cloning both the "media_tree.git" and > > "media_build.git" repositories, and received errors for both. The > > media_tree repository failed on the first line > > > >> git clone > >

Re: [RESEND PATCH 1/14] staging/media/as102: initial import from Abilis

2011-10-30 Thread Piotr Chmura
W dniu 18.10.2011 21:46, Piotr Chmura pisze: Patch taken from http://kernellabs.com/hg/~dheitmueller/v4l-dvb-as102-2/ Changes made by me: 1. Driver moved from media/dvb to staging/media 2. Removed Makefile/Kconfig - it doesn't compile in current tree (...) + +/* + * Note: + * - in AS102 SNR=ME

Re: [RESEND PATCH 11/14] staging/media/as102: fix compile warning about unused function

2011-10-30 Thread Sylwester Nawrocki
On 10/18/2011 10:03 PM, Piotr Chmura wrote: > Patch taken from http://kernellabs.com/hg/~dheitmueller/v4l-dvb-as102-2/ > > Original source and comment: > # HG changeset patch > # User Devin Heitmueller > # Date 1267319685 18000 > # Node ID 84b93826c0a19efa114a6808165f91390cb86daa > # Parent 22ef1

cron job: media_tree daily build: WARNINGS

2011-10-30 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 Oct 30 19:00:14 CET 2011 git hash:a63366b935456dd0984f237642f6d4001dcf8017 gcc version: i686-linux-gcc (GCC

Re: [PATCH] [RESEND] media: tea5764: reconcile Kconfig symbol and macro

2011-10-30 Thread Paul Bolle
On Sun, 2011-10-30 at 10:24 -0700, Randy Dunlap wrote: > On 10/30/11 10:12, Paul Bolle wrote: > > 2) I'm not sure why things are done that way. Why can't builtin drivers > > and loadable drivers default to identical values? But perhaps I'm just > > misunderstanding the code. > > They could default

Re: [PATCH] [RESEND] media: tea5764: reconcile Kconfig symbol and macro

2011-10-30 Thread Randy Dunlap
On 10/30/11 10:12, Paul Bolle wrote: > On Sun, 2011-10-30 at 09:43 -0700, Randy Dunlap wrote: >> On 10/30/11 05:08, Paul Bolle wrote: >>> The Kconfig symbol RADIO_TEA5764_XTAL is unused. The code does use a >>> RADIO_TEA5764_XTAL macro, but does that rather peculiar. But there seems >>> to be a way

Re: [PATCH] [RESEND] media: tea5764: reconcile Kconfig symbol and macro

2011-10-30 Thread Paul Bolle
On Sun, 2011-10-30 at 09:43 -0700, Randy Dunlap wrote: > On 10/30/11 05:08, Paul Bolle wrote: > > The Kconfig symbol RADIO_TEA5764_XTAL is unused. The code does use a > > RADIO_TEA5764_XTAL macro, but does that rather peculiar. But there seems > > to be a way to keep both. (The easiest way out woul

Re: [PATCH] [RESEND] media: tea5764: reconcile Kconfig symbol and macro

2011-10-30 Thread Randy Dunlap
On 10/30/11 05:08, Paul Bolle wrote: > The Kconfig symbol RADIO_TEA5764_XTAL is unused. The code does use a > RADIO_TEA5764_XTAL macro, but does that rather peculiar. But there seems > to be a way to keep both. (The easiest way out would be to rip out both > the Kconfig symbol and the macro.) > >

Re: femon patch for dB

2011-10-30 Thread VDR User
On Sun, Oct 30, 2011 at 6:52 AM, Michael Krufky wrote: > I will push the work for the ATSC snr conversions to my git repository > and issue a pull request to Mauro by the end of the day.  This issue > is larger than a simple userspace unit conversion.  Please send in the > patch inline anyway, as

Re: femon patch for dB

2011-10-30 Thread Antti Palosaari
On 10/29/2011 02:01 AM, James wrote: I added a switch to femon so it displays signal and snr in dB. The cx23885 driver for my Hauppauge WinTV-HVR1250 reports signal and snr in dB. http://lockie.ca/test/femon.patch.bz2 from patch: human readable output: (signal: 0-65335, snr: 1/256 increments)

budget-av problem with inserted CAM (KNC1 DVB-C TDA10024)

2011-10-30 Thread Roger Mårtensson
Hello! (again) Am I the only one with a KNC1 DVB-C TDA10024(clone) card that is trying to use a CAM(smit)? Is there anyone else that have got this to work? In the code it says it's compatible with TDA10023 som the KNC1 DVB-C MK3 card should be identical? (It's identical in code anyway) Symto

Re: femon patch for dB

2011-10-30 Thread Michael Krufky
On Sun, Oct 30, 2011 at 9:52 AM, Michael Krufky wrote: > On Sat, Oct 29, 2011 at 2:37 AM, Mauro Carvalho Chehab > wrote: >> Em 29-10-2011 07:39, Randy Dunlap escreveu: >>> On 10/28/11 22:12, James wrote: diff -r d4e8bf5658ce util/femon/femon.c --- a/util/femon/femon.c    Fri Oct 07 01:2

[PATCH v3 5/14] staging/media/as102: checkpatch fixes

2011-10-30 Thread Piotr Chmura
From: Devin Heitmueller Reformats as10x_cmd_stream.c to take it closer to linux format code. Signed-off-by: Devin Heitmueller Signed-off-by: Piotr Chmura --- v3: added missing empty lines on end of patch. diff --git linux/drivers/staging/media/as102/as10x_cmd_stream.c linuxb/drivers/stagin

Re: femon patch for dB

2011-10-30 Thread Michael Krufky
On Sat, Oct 29, 2011 at 2:37 AM, Mauro Carvalho Chehab wrote: > Em 29-10-2011 07:39, Randy Dunlap escreveu: >> On 10/28/11 22:12, James wrote: >>> diff -r d4e8bf5658ce util/femon/femon.c >>> --- a/util/femon/femon.c    Fri Oct 07 01:26:04 2011 +0530 >>> +++ b/util/femon/femon.c    Fri Oct 28 18:52

Initial tuning file for DVB-C network of Delta in the netherlands

2011-10-30 Thread Hein Rigolo
Here is an initial tuning file for the DVB-C network of Delta in the netherlands: # Initial Tuning file for nl-DELTA # This file only lists the main # frequency. You still need to do # a network scan to find other # transponders. # # C 40200 6875000 NONE QAM64 # Main Frequency Could this be

[PATCH] [RESEND] media: tea5764: reconcile Kconfig symbol and macro

2011-10-30 Thread Paul Bolle
The Kconfig symbol RADIO_TEA5764_XTAL is unused. The code does use a RADIO_TEA5764_XTAL macro, but does that rather peculiar. But there seems to be a way to keep both. (The easiest way out would be to rip out both the Kconfig symbol and the macro.) Note there's also a module parameter 'use_xtal' t

Re: [PATCH 5/6] v4l2-event: Add v4l2_subscribed_event_ops

2011-10-30 Thread Hans Verkuil
On Sunday, October 30, 2011 12:17:28 Hans Verkuil wrote: > On Thursday, October 27, 2011 13:18:02 Hans de Goede wrote: > > Just like with ctrl events, drivers may want to get called back on > > listener add / remove for other event types too. Rather then special > > casing all of this in subscribe

Re: [PATCH 6/6] v4l2-ctrls: Use v4l2_subscribed_event_ops

2011-10-30 Thread Hans Verkuil
On Thursday, October 27, 2011 13:18:03 Hans de Goede wrote: > Signed-off-by: Hans de Goede Acked-by: Hans Verkuil > --- > drivers/media/video/ivtv/ivtv-ioctl.c |3 +- > drivers/media/video/pwc/pwc-v4l.c |2 +- > drivers/media/video/v4l2-ctrls.c | 56 > ++

Re: [PATCH 5/6] v4l2-event: Add v4l2_subscribed_event_ops

2011-10-30 Thread Hans Verkuil
On Thursday, October 27, 2011 13:18:02 Hans de Goede wrote: > Just like with ctrl events, drivers may want to get called back on > listener add / remove for other event types too. Rather then special > casing all of this in subscribe / unsubscribe event it is better to > use ops for this. > > Sign

Re: [PATCH v3 4/14] staging/media/as102: checkpatch fixes

2011-10-30 Thread Stefan Richter
On Oct 30 Piotr Chmura wrote: > Patch taken from http://kernellabs.com/hg/~dheitmueller/v4l-dvb-as102-2/ > > Original source and comment: > # HG changeset patch > # User Devin Heitmueller > # Date 1267318701 18000 > # Node ID 69c8f5172790784738bcc18f8301919ef3d5373f > # Parent b91e96a07bee27c1d4

Re: [PATCH 4/6] v4l2-event: Don't set sev->fh to NULL on unsubcribe

2011-10-30 Thread Hans Verkuil
On Thursday, October 27, 2011 13:18:01 Hans de Goede wrote: > 1: There is no reason for this after v4l2_event_unsubscribe releases the > spinlock nothing is holding a reference to the sev anymore except for the > local reference in the v4l2_event_unsubscribe function. Not true. v4l2-ctrls.c may st

Re: [PATCH 3/6] v4l2-event: Remove pending events from fh event queue when unsubscribing

2011-10-30 Thread Hans Verkuil
On Thursday, October 27, 2011 13:18:00 Hans de Goede wrote: > The kev pointers inside the pending events queue (the available queue) of the > fh point to data inside the sev, unsubscribing frees the sev, thus making > these > pointers point to freed memory! > > This patch fixes these dangling poi

Re: [PATCH 2/6] v4l2-event: Deny subscribing with a type of V4L2_EVENT_ALL

2011-10-30 Thread Hans Verkuil
Acked-by: Hans Verkuil Regards, Hans On Thursday, October 27, 2011 13:17:59 Hans de Goede wrote: > Signed-off-by: Hans de Goede > --- > drivers/media/video/v4l2-event.c |3 +++ > 1 files changed, 3 insertions(+), 0 deletions(-) > > diff --git a/drivers/media/video/v4l2-event.c >

Re: [PATCH 1/6] v4l2-ctrl: Send change events to all fh for auto cluster slave controls

2011-10-30 Thread Hans Verkuil
This is also part of a pull request from me (and so has my Signed-off-by already): http://www.mail-archive.com/linux-media@vger.kernel.org/msg38018.html Regards, Hans On Thursday, October 27, 2011 13:17:58 Hans de Goede wrote: > Otherwise the fh changing the master control won't get th

[PATCH v3 4/14] staging/media/as102: checkpatch fixes

2011-10-30 Thread Piotr Chmura
Patch taken from http://kernellabs.com/hg/~dheitmueller/v4l-dvb-as102-2/ Original source and comment: # HG changeset patch # User Devin Heitmueller # Date 1267318701 18000 # Node ID 69c8f5172790784738bcc18f8301919ef3d5373f # Parent b91e96a07bee27c1d421b4c3702e33ee8075de83 as102: checkpatch fixes