[RFC/PATCH] v4l2-compliance: Reject invalid ioctl error codes

2012-12-23 Thread Laurent Pinchart
The recent uvcvideo regression that broke pulseaudio/KDE (see commit 9c016d61097cc39427a2f5025bdd97ac633d26a6 in the mainline kernel) was caused by the uvcvideo driver returning a -ENOENT error code to userspace by mistake. To make sure such regressions will be caught before reaching users, test i

cron job: media_tree daily build: WARNINGS

2012-12-23 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 Dec 23 19:00:21 CET 2012 git hash:a6bad040dacb7c0c59173feba98001ae1d4811e4 gcc version: i686-linux-gcc (GCC

Re: [v3.8-rc1] Multimedia regression, ioctl(17,..)-API changed ?

2012-12-23 Thread Mauro Carvalho Chehab
Hi Jörg, Em Sun, 23 Dec 2012 17:46:07 +0100 Jörg Otte escreveu: > With kernel v3.8 all multimedia programs under KDE4 don't work (Kubuntu > 12.04). > They alltogether ( at least Dragonplayer (Mediaplayer), Knotify4 > (system-sound), > System-Settings-Multimedia,..) are looping forever producing

Re: [v3.8-rc1] Multimedia regression, ioctl(17,..)-API changed ?

2012-12-23 Thread Linus Torvalds
Jörg - does current git work for you? It has a patch from Rafael that just reverts the insane error code, and fixed something very similar for him. (I just pushed out, so it might take a few minutes to mirror out to the public sites). Linus On Sun, Dec 23, 2012 at 8:46 AM, Jörg Ott

Re: cron job: media_tree daily build: ERRORS

2012-12-23 Thread Hans Verkuil
On Sun December 23 2012 00:40:59 Mauro Carvalho Chehab wrote: > Em Sat, 22 Dec 2012 21:46:17 +0100 (CET) > "Hans Verkuil" escreveu: > > > 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 bui

[v3.8-rc1] Multimedia regression, ioctl(17,..)-API changed ?

2012-12-23 Thread Jörg Otte
With kernel v3.8 all multimedia programs under KDE4 don't work (Kubuntu 12.04). They alltogether ( at least Dragonplayer (Mediaplayer), Knotify4 (system-sound), System-Settings-Multimedia,..) are looping forever producing 100% CPU-usage and must be killed. With kernel 3.7 there are no problems. I

Re: Patch update notification: 37 patches updated

2012-12-23 Thread Mauro Carvalho Chehab
Em Sun, 23 Dec 2012 14:45:31 +0100 Frank Schäfer escreveu: > Hi Mauro, > > Am 23.12.2012 01:08, schrieb Patchwork: > > Hello, > > > > The following patches (submitted by you) have been updated in patchwork: > > > > * [3/6] em28xx: fix capture type setting in em28xx_urb_data_copy_vbi() > >

Re: [PATCH v2 2/5] em28xx: respect the message size constraints for i2c transfers

2012-12-23 Thread Mauro Carvalho Chehab
Em Sun, 23 Dec 2012 14:58:12 +0100 Frank Schäfer escreveu: > Am 23.12.2012 01:07, schrieb Mauro Carvalho Chehab: > > Em Sun, 16 Dec 2012 19:23:28 +0100 > > Frank Schäfer escreveu: > > > >> The em2800 can transfer up to 4 bytes per i2c message. > >> All other em25xx/em27xx/28xx chips can transfer

Re: [PATCH v2 21/21] em28xx: add module parameter for selection of the preferred USB transfer type

2012-12-23 Thread Mauro Carvalho Chehab
Em Sun, 23 Dec 2012 15:01:26 +0100 Frank Schäfer escreveu: > Am 23.12.2012 14:44, schrieb Mauro Carvalho Chehab: > > Hi Frank, > > > > Em Thu, 8 Nov 2012 20:11:53 +0200 > > Frank Schäfer escreveu: > > > >> By default, isoc transfers are used if possible. > >> With the new module parameter, bulk

Re: [PATCH v2 21/21] em28xx: add module parameter for selection of the preferred USB transfer type

2012-12-23 Thread Frank Schäfer
Am 23.12.2012 14:44, schrieb Mauro Carvalho Chehab: > Hi Frank, > > Em Thu, 8 Nov 2012 20:11:53 +0200 > Frank Schäfer escreveu: > >> By default, isoc transfers are used if possible. >> With the new module parameter, bulk can be selected as the >> preferred USB transfer type. > I did some tests ye

Re: [PATCH v2 2/5] em28xx: respect the message size constraints for i2c transfers

2012-12-23 Thread Frank Schäfer
Am 23.12.2012 01:07, schrieb Mauro Carvalho Chehab: > Em Sun, 16 Dec 2012 19:23:28 +0100 > Frank Schäfer escreveu: > >> The em2800 can transfer up to 4 bytes per i2c message. >> All other em25xx/em27xx/28xx chips can transfer at least 64 bytes per >> message. >> >> I2C adapters should never split

Re: [PATCH v2 16/21] em28xx: rename usb debugging module parameter and macro

2012-12-23 Thread Mauro Carvalho Chehab
Em Sun, 23 Dec 2012 14:34:28 +0100 Frank Schäfer escreveu: > Am 22.12.2012 21:10, schrieb Mauro Carvalho Chehab: > > Em Thu, 8 Nov 2012 20:11:48 +0200 > > Frank Schäfer escreveu: > > > >> Rename module parameter isoc_debug to usb_debug and macro > >> em28xx_isocdbg to em28xx_usb dbg to reflect

Re: Patch update notification: 37 patches updated

2012-12-23 Thread Frank Schäfer
Hi Mauro, Am 23.12.2012 01:08, schrieb Patchwork: > Hello, > > The following patches (submitted by you) have been updated in patchwork: > > * [3/6] em28xx: fix capture type setting in em28xx_urb_data_copy_vbi() > - http://patchwork.linuxtv.org/patch/15651/ > was: New > now: Accepted

Re: [PATCH v2 21/21] em28xx: add module parameter for selection of the preferred USB transfer type

2012-12-23 Thread Mauro Carvalho Chehab
Hi Frank, Em Thu, 8 Nov 2012 20:11:53 +0200 Frank Schäfer escreveu: > By default, isoc transfers are used if possible. > With the new module parameter, bulk can be selected as the > preferred USB transfer type. I did some tests yesterday with prefer_bulk. IMHO, webcams should select bulk mode

Re: [PATCH v2 16/21] em28xx: rename usb debugging module parameter and macro

2012-12-23 Thread Frank Schäfer
Am 22.12.2012 21:10, schrieb Mauro Carvalho Chehab: > Em Thu, 8 Nov 2012 20:11:48 +0200 > Frank Schäfer escreveu: > >> Rename module parameter isoc_debug to usb_debug and macro >> em28xx_isocdbg to em28xx_usb dbg to reflect that they are >> used for isoc and bulk USB transfers. >> >> Signed-off-b