Re: Proposal for a V4L2 Media Controller mini-summit

2010-03-21 Thread Hans Verkuil
On Friday 12 February 2010 15:50:08 Hans Verkuil wrote: > Hi all, > > During the Linux Plumbers Conference in September 2009 I organized a V4L-DVB > mini-summit. The focus of that summit was on discussing a framework through > which we could support all the functionality that the video hardware of

Re: [PATCH] device_attributes: add sysfs_attr_init() for dynamic attributes

2010-03-21 Thread Dmitry Torokhov
Hi Wolfram, On Mon, Mar 22, 2010 at 07:21:17AM +0100, Wolfram Sang wrote: > Made necessary by 6992f5334995af474c2b58d010d08bc597f0f2fe. > > Found by this semantic patch: > > @ init @ > type T; > identifier A; > @@ > > T { > ... > struct device_attribute A

[PATCH] device_attributes: add sysfs_attr_init() for dynamic attributes

2010-03-21 Thread Wolfram Sang
Made necessary by 6992f5334995af474c2b58d010d08bc597f0f2fe. Found by this semantic patch: @ init @ type T; identifier A; @@ T { ... struct device_attribute A; ... }; @ main extends init @ expression E; statement S; identifier err;

RE: [REPORT] Brainstorm meeting on V4L2 memory handling

2010-03-21 Thread Hiremath, Vaibhav
> -Original Message- > From: linux-media-ow...@vger.kernel.org [mailto:linux-media- > ow...@vger.kernel.org] On Behalf Of Hans Verkuil > Sent: Saturday, March 13, 2010 7:26 PM > To: linux-media@vger.kernel.org > Subject: [REPORT] Brainstorm meeting on V4L2 memory handling > > Introduction

Re: Add AverTV Studio 509UA

2010-03-21 Thread hermann pitton
> > I think best would be to create two patches. > > 1/2 adds the new tuner. (tuner-types.c/tuner.h/tveeprom.c) > 2/2 adds the new Avermedia card. > > Else it looks fine. To add some for the record here for those eventually later on it. We have seen over so many years now, that hundreds of "n

Re: RFC: Phase 1: Proposal to convert V4L1 drivers

2010-03-21 Thread Hans Verkuil
On Sunday 21 March 2010 23:45:04 Hans Verkuil wrote: > On Saturday 20 March 2010 09:58:49 Hans Verkuil wrote: > > These drivers have no hardware to test with: bw-qcam, c-qcam, arv, w9966. > > However, all four should be easy to convert to v4l2, even without hardware. > > Volunteers? > > I've conve

Re: Add AverTV Studio 509UA

2010-03-21 Thread hermann pitton
Hi, Am Samstag, den 20.03.2010, 16:08 +0200 schrieb Евгений Бацман: > 2010/3/20 hermann pitton : > > Hi, Евгений, > > > > is there any reason, why you set the charge pump bit to low and slow > > tuning on a non FM tuner? > > > > Cheers, > > Hermann > > > > > > Am Freitag, den 19.03.2010, 20:32 +02

Re: RFC: Phase 1: Proposal to convert V4L1 drivers

2010-03-21 Thread Hans Verkuil
On Saturday 20 March 2010 09:58:49 Hans Verkuil wrote: > These drivers have no hardware to test with: bw-qcam, c-qcam, arv, w9966. > However, all four should be easy to convert to v4l2, even without hardware. > Volunteers? I've converted these four drivers to V4L2. See my tree: http://www.linuxt

Re: [PATCH] V4L/DVB: saa7146: IRQF_DISABLED causes only trouble

2010-03-21 Thread Andy Walls
On Sun, 2010-03-21 at 21:08 +0100, Bjørn Mork wrote: > As discussed many times, e.g. in http://lkml.org/lkml/2007/7/26/401 > mixing IRQF_DISABLED with IRQF_SHARED just doesn't make sense. > > Remove IRQF_DISABLED to avoid random unexpected behaviour. > > Ever since I started using the saa7146 dri

[PATCH] drivers/media/video: avoid NULL dereference

2010-03-21 Thread Julia Lawall
From: Julia Lawall If ov is NULL, it will not be possible to take the lock in the first place, so move the test up earlier. The semantic match that finds the problem is as follows: (http://coccinelle.lip6.fr/) // @r exists@ expression E, E1; identifier f; statement S1,S3; iterator iter; @@ if

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

2010-03-21 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:Sun Mar 21 19:00:14 CET 2010 path:http://www.linuxtv.org/hg/v4l-dvb changeset: 14494:929298149eba git master:

[PULL] http://linuxtv.org/hg/~endriss/ngene

2010-03-21 Thread Oliver Endriss
Mauro, Please pull from http://linuxtv.org/hg/~endriss/ngene for the following changeset: 01/01: ngene: Workaround for stuck DiSEqC pin http://linuxtv.org/hg/~endriss/ngene?cmd=changeset;node=1dc562463f63 stv090x.c |4 1 file changed, 4 insertions(+) Thanks, Oliver --

[PULL] http://kernellabs.com/hg/~dheitmueller/ngene2

2010-03-21 Thread Devin Heitmueller
Hello, Please PULL from http://kernellabs.com/hg/~dheitmueller/ngene2 for the following: ngene: properly support boards where channel 0 isn't a TS input ngene: add initial support for digital side of Avermedia m780 ngene: split out i2c code into a separate file ngene: split out card specific code

[PATCH] V4L/DVB: saa7146: IRQF_DISABLED causes only trouble

2010-03-21 Thread Bjørn Mork
As discussed many times, e.g. in http://lkml.org/lkml/2007/7/26/401 mixing IRQF_DISABLED with IRQF_SHARED just doesn't make sense. Remove IRQF_DISABLED to avoid random unexpected behaviour. Ever since I started using the saa7146 driver, I've had occasional soft lockups. I do not have any real ev

[PATCH] V4L/DVB: budget-av: wait longer for frontend to power on

2010-03-21 Thread Bjørn Mork
Some devices need much more time than 100ms to power on, leading to a failure to enable the frontend on the first attempt. Instead we get [ 38.194200] saa7146: register extension 'budget_av'. [ 38.253828] budget_av :05:01.0: PCI INT A -> GSI 17 (level, low) -> IRQ 17 [ 38.601572] saa714

[PATCH] V4L2: Replace loops for finding max buffers in VIDIOC_REQBUFS callbacks

2010-03-21 Thread Andreas Bombe
Due to obvious copy and paste coding a number of video capture drivers which implement a limit on the buffer memory decremented the user supplied buffer count in a while loop until it reaches an acceptable value. This is a silly thing to do when the maximum value can be directly computed. Signed-

[PATCH 5/5] Staging: cx25821: fix coding style issues in cx25821-medusa-video.c

2010-03-21 Thread Olimpiu Pascariu
>From 24e5efa163c1fa58f694fd8b44dc3488e0cc92d1 Mon Sep 17 00:00:00 2001 From: Olimpiu Pascariu Date: Sun, 21 Mar 2010 20:46:26 +0200 Subject: [PATCH 5/5] Staging: cx25821: fix coding style issues in cx25821-medusa-video.c This is a patch to cx25821-medusa-video.c file that fixes up warnings and

0004-Staging-cx25821-fix-coding-style-issues-in-cx25821-i.patch

2010-03-21 Thread Olimpiu Pascariu
>From 5f02f6af270ce061174806a039d8d44a35b2ce5e Mon Sep 17 00:00:00 2001 From: Olimpiu Pascariu Date: Sun, 21 Mar 2010 19:52:31 +0200 Subject: [PATCH 4/4] Staging: cx25821: fix coding style issues in cx25821-i2c.c This is a patch to cx25821-i2c.c file that fixes up warnings and errors found by th

0003-Staging-cx25821-fix-coding-style-issues-in-cx25821-g.patch

2010-03-21 Thread Olimpiu Pascariu
>From fae275cf48fd318cbd88e59e90f119c3f75dab72 Mon Sep 17 00:00:00 2001 From: Olimpiu Pascariu Date: Sun, 21 Mar 2010 19:44:01 +0200 Subject: [PATCH 3/3] Staging: cx25821: fix coding style issues in cx25821-gpio.c This is a patch to cx25821-gpio.c file that fixes up warnings and errors found by

0002-Staging-cx25821-fix-coding-style-issues-in-cx25821-c.patch

2010-03-21 Thread Olimpiu Pascariu
>From d0cc598f638e6b1dc8c06c2327e67667d52095ed Mon Sep 17 00:00:00 2001 From: Olimpiu Pascariu Date: Sun, 21 Mar 2010 19:18:19 +0200 Subject: [PATCH 2/2] Staging: cx25821: fix coding style issues in cx25821-core.c This is a patch to cx25821-core.c file that fixes up warnings and errors found by

Re: prof 7300

2010-03-21 Thread joep admiraal
Op zondag 17 januari 2010 16:42:53 schreef Igor M. Liplianin: > On 17 января 2010 16:42:27 joep admiraal wrote: > > I had some troubles with a prof 7300 dvb s-2 card. > > I am running OpenSuse 11.2 with a recent hg copy of the v4l-dvb > > repository. It was detected as a Hauppauge WinTV instead of

Re: RFC: Phase 1: Proposal to convert V4L1 drivers

2010-03-21 Thread Hans Verkuil
On Saturday 20 March 2010 19:38:39 Gerard Klaver wrote: > On Sat, 2010-03-20 at 09:58 +0100, Hans Verkuil wrote: > > Hi all, > > > > > > > - ov511 > > - ovcamchip > > - w9968cf > > - stv680 > > > > > Conclusion: > > > > These drivers have no hardware to test with: bw-qcam, c-qcam, arv, w9966.

Re: [PATCH 12/24] media/video: fix dangling pointers

2010-03-21 Thread Hans Verkuil
On Sunday 21 March 2010 15:14:17 Mark Brown wrote: > On Sun, Mar 21, 2010 at 02:46:55PM +0100, Jean Delvare wrote: > > On Sat, 20 Mar 2010 23:02:49 +0100, Hans Verkuil wrote: > > > > I feel I am missing something here. Why does clientdata have to be set to > > > NULL when we are tearing down the d

Re: [linux-dvb] Problem with decrypting dvb-s channels

2010-03-21 Thread OJ
> "A couple of months" triggers me: for as far as I am aware dvb decoder > cards need regular 'entitlement messages' to continue to work. And > when the card is not in an active system for a long time it misses those > messages and gets shut out. You need to ask the card issuer to re-activate > the

Re: [PATCH] saa7134: add capture boards Hawell HW-404M7 and HW-808M7

2010-03-21 Thread Vladimir Ermakov
Hi Hermann. Sorry, i forgot it. Resend patch with SOB. Signed-off-by: Vladimir Ermakov # HG changeset patch # User Vladimir Ermakov # Date 1269096094 -10800 # Node ID a91db2cf4f5774866c8c5bf906d9ac4faff9173d # Parent 929298149eba4b6d0696124b9880113c25cd0788 saa7134: fix GPIO HW-404M7 diff -r

Re: [PATCH 12/24] media/video: fix dangling pointers

2010-03-21 Thread Mark Brown
On Sun, Mar 21, 2010 at 02:46:55PM +0100, Jean Delvare wrote: > On Sat, 20 Mar 2010 23:02:49 +0100, Hans Verkuil wrote: > > I feel I am missing something here. Why does clientdata have to be set to > > NULL when we are tearing down the device anyway? > We're not tearing down the device, that's th

i2c interface bugs in dvb drivers

2010-03-21 Thread matthieu castet
Hi, some dvb driver that export a i2c interface contains some mistakes because they mainly used by driver (frontend, tuner) wrote for them. But the i2c interface is exposed to everybody. One mistake is expect msg[i].addr with 8 bits address instead of 7 bits address. This make them use eeprom

Re: [PATCH 12/24] media/video: fix dangling pointers

2010-03-21 Thread Jean Delvare
Hi Hans, On Sat, 20 Mar 2010 23:02:49 +0100, Hans Verkuil wrote: > On Saturday 20 March 2010 15:12:53 Wolfram Sang wrote: > > Fix I2C-drivers which missed setting clientdata to NULL before freeing the > > structure it points to. Also fix drivers which do this _after_ the structure > > was freed al

dvb frontend lockup

2010-03-21 Thread matthieu castet
Hi, With my current kernel (2.6.32), if my dvb device is removed while in use, I got [1]. After checking the source code, the problem seems to happen also in master : If there are users (for example users == -2) : - dvb_unregister_frontend : - stop kernel thread with dvb_frontend_stop : - fep

Re: [PATCH 12/24] media/video: fix dangling pointers

2010-03-21 Thread Wolfram Sang
Hello Hans, > > Fix I2C-drivers which missed setting clientdata to NULL before freeing the > > structure it points to. Also fix drivers which do this _after_ the structure > > was freed already. > > I feel I am missing something here. Why does clientdata have to be set to > NULL when we are teari

[PATCH] af9015 : more robust eeprom parsing

2010-03-21 Thread matthieu castet
the af9015 eeprom parsing accept 0x38 as 2nd demodulator. But this is impossible because the first one is already hardcoded to 0x38. This remove a special case for AverMedia AVerTV Volar Black HD. Also in af9015_copy_firmware don't hardcode the 2nd demodulator address to 0x3a. Signed-off-by: M

Re: RFC: Phase 1: Proposal to convert V4L1 drivers

2010-03-21 Thread Nicolas Will
On Sun, 2010-03-21 at 09:19 +0100, Hans Verkuil wrote: > > > - cpia_pp > > > > > > Parallel port webcam driver for the Creative Webcam II. I found > one on eBay, > > > so with luck I can get hold of the hardware and get it to HdG. > > > > > > > Although I appreaciate your effort in getting this ca

Re: RFC: Phase 1: Proposal to convert V4L1 drivers

2010-03-21 Thread Hans Verkuil
On Sunday 21 March 2010 00:09:16 Hans de Goede wrote: > Hi, > > On 03/20/2010 09:58 AM, Hans Verkuil wrote: > > Hi all, > > > > Well, I certainly fired everyone up with my RFC. Based on the replies I got > > I > > do think the time is right to start the removal process. > > > > Phase 1 would be t