Re: [Workshop-2011] Media Subsystem Workshop 2011

2011-08-10 Thread Hans de Goede
Hi, On 08/10/2011 02:34 AM, Theodore Kilgore wrote: snip but this is the way how the current discussion feels to me. If we agree on aiming for doing it right then with that comes to me doing a software design from scratch, so without taking into account what is already there. Here, a

Re: [Workshop-2011] Media Subsystem Workshop 2011

2011-08-09 Thread Hans de Goede
Hi, On 08/08/2011 07:39 PM, Theodore Kilgore wrote: On Mon, 8 Aug 2011, Mauro Carvalho Chehab wrote: snip Mauro, In fact none of the currently known and supported cameras are using PTP. All of them are proprietary. They have a rather intimidating set of differences in functionality,

Re: [Workshop-2011] Media Subsystem Workshop 2011

2011-08-09 Thread Hans de Goede
Hi, snip OK, another example. The cameras supported in camlibs/jl2005c do not have webcam ability, but someone could at any time design and market a dualmode which has in stillcam mode the same severe limitation. What limitation? Well, the entire memory of the camera must be dumped, or else

Re: USB mini-summit at LinuxCon Vancouver

2011-08-09 Thread Hans de Goede
Hi, On 08/08/2011 07:58 PM, Sarah Sharp wrote: On Fri, Aug 05, 2011 at 09:45:56AM +0200, Hans de Goede wrote: Hi, On 08/05/2011 12:56 AM, Greg KH wrote: On Thu, Aug 04, 2011 at 07:21:47PM -0300, Mauro Carvalho Chehab wrote: I think it is important to separate oranges from apples here

Re: USB mini-summit at LinuxCon Vancouver

2011-08-09 Thread Hans de Goede
Hi, On 08/09/2011 04:19 PM, Alan Stern wrote: On Tue, 9 Aug 2011, Hans de Goede wrote: I would really like to see the dual mode camera and TV tuner discussion separated. They are 2 different issues AFAIK. 1) Dual mode cameras: In the case of the dual mode camera we have 1 single device

Re: [Workshop-2011] Media Subsystem Workshop 2011

2011-08-09 Thread Hans de Goede
Hi, On 08/09/2011 07:10 PM, Theodore Kilgore wrote: On Tue, 9 Aug 2011, Hans de Goede wrote: snip No, but both Adam and I realized, approximately at the same time yesterday afternoon, something which is rather important here. Gphoto is not developed exclusively for Linux. Furthermore

Re: USB mini-summit at LinuxCon Vancouver

2011-08-09 Thread Hans de Goede
Hi, On 08/09/2011 10:31 PM, Adam Baker wrote: On Tuesday 09 August 2011, Hans de Goede wrote: snip It has also just occured to me that it might be possible to solve the issues we are facing just in the kernel. At the moment when the kernel performs a USBDEVFS_DISCONNECT it keeps the kernel

Re: [Workshop-2011] Media Subsystem Workshop 2011

2011-08-08 Thread Hans de Goede
Hi, On 08/08/2011 12:53 AM, Adam Baker wrote: On Friday 05 August 2011, Hans de Goede wrote: This sounds to be a good theme for the Workshop, or even to KS/2011. Agreed, although we don't need to talk about this for very long, the solution is basically: 1) Define a still image retrieval API

Re: [Workshop-2011] Media Subsystem Workshop 2011

2011-08-05 Thread Hans de Goede
Hi all, On 08/04/2011 02:34 PM, Mauro Carvalho Chehab wrote: Em 03-08-2011 20:20, Theodore Kilgore escreveu: snip snip Yes, that kind of thing is an obvious problem. Actually, though, it may be that this had just better not happen. For some of the hardware that I know of, it could be a real

Re: USB mini-summit at LinuxCon Vancouver

2011-08-05 Thread Hans de Goede
Hi, On 08/05/2011 12:56 AM, Greg KH wrote: On Thu, Aug 04, 2011 at 07:21:47PM -0300, Mauro Carvalho Chehab wrote: I know that this problem were somewhat solved for 3G modems, with the usage of the userspace problem usb_modeswitch, and with some quirks for the USB storage driver, but I'm not

Re: USB mini-summit at LinuxCon Vancouveroliver

2011-08-05 Thread Hans de Goede
Hi, On 08/05/2011 09:59 AM, Oliver Neukum wrote: Am Freitag, 5. August 2011, 09:45:56 schrieb Hans de Goede: This is the issue on which I feel a bit stonewalled. Simple putting your fingers in your ears and singing la la la do it in userspace is not going to cut it here. There is no way to do

Re: [Workshop-2011] Media Subsystem Workshop 2011

2011-08-04 Thread Hans de Goede
Hi, On 08/03/2011 10:36 PM, Mauro Carvalho Chehab wrote: Em 03-08-2011 16:53, Theodore Kilgore escreveu: snip snip Mauro, Not saying that you need to change the program for this session to deal with this topic, but an old and vexing problem is dual-mode devices. It is an issue which needs

Re: [GIT PULL for v3.0] media updates for v3.1

2011-08-01 Thread Hans de Goede
Hi, On 07/31/2011 12:38 PM, Mauro Carvalho Chehab wrote: Em 31-07-2011 04:57, Hans de Goede escreveu: Hi, I notice that Hans Verkuil's patches to make poll report what is being polled to drivers, and my corresponding patches for adding event support to pwc are not included, what is the plan

Re: [GIT PULL for v3.0] media updates for v3.1

2011-08-01 Thread Hans de Goede
Hi, On 08/01/2011 09:45 AM, Hans Verkuil wrote: Hi, On 07/31/2011 12:38 PM, Mauro Carvalho Chehab wrote: Em 31-07-2011 04:57, Hans de Goede escreveu: Hi, I notice that Hans Verkuil's patches to make poll report what is being polled to drivers, and my corresponding patches for adding event

Re: [GIT PULL for v3.0] media updates for v3.1

2011-07-31 Thread Hans de Goede
Hi, I notice that Hans Verkuil's patches to make poll report what is being polled to drivers, and my corresponding patches for adding event support to pwc are not included, what is the plan with these? Regards, Hans -- To unsubscribe from this list: send the line unsubscribe linux-media in

Re: Some comments on the new autocluster patches

2011-07-26 Thread Hans de Goede
Hi, On 07/26/2011 11:26 AM, Hans Verkuil wrote: OK, I'm back to work after my vacation, so it's time to go through the backlog... Welcome back :) On Tuesday, July 12, 2011 15:25:02 Hans de Goede wrote: Hi, On 07/04/2011 11:43 AM, Hans Verkuil wrote: snip snip It is relevant. Take

Re: Some comments on the new autocluster patches

2011-07-26 Thread Hans de Goede
Hi, On 07/26/2011 04:19 PM, Hans Verkuil wrote: On Tuesday, July 26, 2011 15:51:58 Hans de Goede wrote: snip An open question is whether writing to an inactive and volatile control should return an error or not. I would prefer an error return. I am worried about backwards

[GIT FIXES FOR 3.1] pwc: various fixes

2011-07-24 Thread Hans de Goede
since commit 30178e8623281063c18592a848cdcd71f78f603d: vivi: let vb2_poll handle events. (2011-07-18 13:07:28 +0200) are available in the git repository at: git://linuxtv.org/hgoede/gspca.git media-for_v3.1 Dan Carpenter (1): pwc: precedence bug in pwc_init_controls() Hans de Goede (2

Re: [patch] [media] pwc: precedence bug in pwc_init_controls()

2011-07-24 Thread Hans de Goede
Hi, Thanks I've added your patch to my gspca/pwc tree and send an updated pull request to Mauro including your patch. Regards, Hans On 07/23/2011 08:53 PM, Dan Carpenter wrote: '!' has higher precedence than'' so we need parenthesis here. Signed-off-by: Dan Carpentererro...@gmail.com diff

Re: [PATCH v4 1/1] libv4l: Add plugin support to libv4l

2011-07-24 Thread Hans de Goede
Hi, On 07/22/2011 05:39 PM, Yordan Kamenov wrote: Hi Hans, Hans de Goede wrote: Hi, Sorry it took so long, but I've just merged the plugin support into v4l-utils git. I did make some minor mods / bugfixes before merging, see the commit message in git. Regards, Hans p.s. I think we should

[GIT PATCHES FOR 3.1] pwc: Add support for control events

2011-07-19 Thread Hans de Goede
30178e8623281063c18592a848cdcd71f78f603d: vivi: let vb2_poll handle events. (2011-07-18 13:07:28 +0200) are available in the git repository at: git://linuxtv.org/hgoede/gspca.git media-for_v3.1 Hans de Goede (1): pwc: Add support for control events drivers/media/video/pwc/pwc-if.c | 84

Re: [GIT PATCHES FOR 3.1] pwc: Add support for control events

2011-07-19 Thread Hans de Goede
Hi, On 07/19/2011 12:33 PM, Hans de Goede wrote: Hi Mauro, Please pull from my tree to add support for control events to the pwc driver. Note that this patch depends upon the patches from hverkuils poll tree (and those patches are thus also in my tree, but not part of this pull req

[GIT PATCHES FOR 3.1] pwc: Add support for control events (v2)

2011-07-19 Thread Hans de Goede
30178e8623281063c18592a848cdcd71f78f603d: vivi: let vb2_poll handle events. (2011-07-18 13:07:28 +0200) are available in the git repository at: git://linuxtv.org/hgoede/gspca.git media-for_v3.1 Hans de Goede (2): pwc: Add support for control events pwc: properly mark device_hint as unused in all probe

RFC: removing a bunch of module options from the pwc driver

2011-07-19 Thread Hans de Goede
Hi, In the light of my ongoing pwc driver cleanup I would like to remove a bunch of module options, like the device_hint option which allows one the specify a preferred minor, which does not really fit well in our modern dynamic minor v4l2 config, and there are a bunch of options for things

RFC: removing pwc kconfig options

2011-07-19 Thread Hans de Goede
Hi, The pwc driver currently has 2 kconfig options, one to enable / disable various debugging options, and another for enabling/ disabling input-evdev support. IMHO these both can go away, the debugging can trigger on CONFIG_VIDEO_ADV_DEBUG, and the input on CONFIG_INPUT. Regards, Hans -- To

Announcing v4l-utils-0.8.5

2011-07-17 Thread Hans de Goede
Hi, I'm happy to announce the release of v4l-utils-0.8.5. We're back to our regular boring releases again :) Still this release contains some fixes / things which are good to get out there, hence a new release. Full changelog: v4l-utils-0.8.5 --- * Utils changes *

Start of v4l-utils-0.9.x devel cycle, break of libv4lconvert ABI

2011-07-17 Thread Hans de Goede
Hi All, With the upcoming support for libv4l2 plugins we need to break the ABI for libv4lconvert, this is a good moment to start a bit more unstable v4l-utils / libv4l tree where we can make some other changes as well. v4l-utils follows the classic odd unstable / even stable release numbering,

Re: [PATCH v4 1/1] libv4l: Add plugin support to libv4l

2011-07-17 Thread Hans de Goede
Hi, Sorry it took so long, but I've just merged the plugin support into v4l-utils git. I did make some minor mods / bugfixes before merging, see the commit message in git. Regards, Hans p.s. I think we should expand the plugin support with support for a output devices, iow add a write()

Re: [PATCH 1/2] libv4l2: add implicit conversion from single- to multi-plane api

2011-07-17 Thread Hans de Goede
Hi, On 07/06/2011 11:24 AM, Tomasz Stanislawski wrote: This patch add implicit conversion of single plane variant of ioctl to multiplane variant. The conversion is performed only in case if a driver implements only mplane api. The conversion is done by substituting SYS_IOCTL with a wrapper that

Re: [ANNOUNCE] Media subsystem workshop 2011 - Prague - Oct 24-26

2011-07-11 Thread Hans de Goede
Hi Mauro, On Wednesday 22 June 2011 20:33:34 Mauro Carvalho Chehab wrote: Media subsystem workshop 2011 - Prague - Oct 24-26 The below website says Oct 23 - 25, I'm confused now and I hope to book a plane ticket soon. Can someone tell me which dates are correct ?

New ctrl framework also enumerates classes

2011-07-04 Thread Hans de Goede
Hi All, One last thing before I really leave on vacation which just popped in my mind as something which I had not mentioned yet. The new ctrl framework also enumerates classes when enumerating ctrls with the next flag. I wonder if this is intentional? IOW if this is a feature or a bug?

[GIT PATCHES FOR 3.1] New SE401 driver + major pwc driver cleanup

2011-07-03 Thread Hans de Goede
-event.h: add overview documentation to the header (2011-07-01 20:54:03 -0300) are available in the git repository at: git://linuxtv.org/hgoede/gspca.git media-for_v3.1 Hans Verkuil (1): [media] v4l2-ctrls.c: add support for V4L2_EVENT_SUB_FL_ALLOW_FEEDBACK Hans de Goede (22

Kudos for the new vtl2 ctrls framework

2011-07-03 Thread Hans de Goede
Hi all, After making some serious use of it in the pwc driver cleanup I would like to thank Hans V. for all his hard work on the new ctrl framework. The clusters bit got a bit getting used to / but once I did it is great. Once you get it, it really makes sense to group certain ctrls into

Vacation

2011-07-03 Thread Hans de Goede
Hi All, This will be one of my last mails as I'm leaving for vacation tomorrow. I'll be back July 12th Regards, Hans -- To unsubscribe from this list: send the line unsubscribe linux-media in the body of a message to majord...@vger.kernel.org More majordomo info at

Re: [ANNOUNCE] Media subsystem workshop 2011 - Prague - Oct 24-26

2011-07-03 Thread Hans de Goede
Hi Mauro, On Wednesday 22 June 2011 20:33:34 Mauro Carvalho Chehab wrote: Media subsystem workshop 2011 - Prague - Oct 24-26 Since 2007, we're doing annual mini-summits for the media subsystem, in order to plan the new features that will be introduced there. Last year, during the Kernel

Re: Some comments on the new autocluster patches

2011-07-02 Thread Hans de Goede
Hi, snip snip snip Ok, thinking about this some more and reading Hans V's comments I think that the current code in Hans V's core8c branch is fine, and should go to 3.1 (rather then be delayed to 3.2). As for the fundamental question what to do with foo controls when autofoo goes from auto to

Re: Some comments on the new autocluster patches

2011-07-02 Thread Hans de Goede
Hi, On 07/02/2011 01:10 PM, Hans Verkuil wrote: On Saturday, July 02, 2011 12:28:35 Hans de Goede wrote: Hi, snip snip snip Ok, thinking about this some more and reading Hans V's comments I think that the current code in Hans V's core8c branch is fine, and should go to 3.1 (rather

Some comments on the new autocluster patches

2011-07-01 Thread Hans de Goede
Hi Hans (et all), I've been working on doing a much needed cleanup to the pwc driver, converting it to videobuf2 and using the new ctrl framework. I hope to be able to send a pull request for this, this weekend. I saw your pull request and I'm looking forward to be able to play with ctrl

Re: [git:xawtv3/master] xawtv: reenable its usage with webcam's

2011-06-30 Thread Hans de Goede
Hi, On 06/29/2011 09:27 PM, Mauro Carvalho Chehab wrote: snip Anyway, it is fixed. I also made scantv to force for a TV device at auto mode, as it doesn't sense to scan for TV channels on devices without tuner. Thanks for fixing this, 2 remarks wrt the auto patch for scantv: 1) This bit

Re: [git:xawtv3/master] xawtv: reenable its usage with webcam's

2011-06-29 Thread Hans de Goede
Hi, On 06/28/2011 07:32 PM, Mauro Carvalho Chehab wrote: This is an automatic generated email to let you know that the following patch were queued at the http://git.linuxtv.org/xawtv3.git tree: Subject: xawtv: reenable its usage with webcam's Author: Mauro Carvalho Chehabmche...@redhat.com

Re: RFC: poll behavior

2011-06-29 Thread Hans de Goede
Hi, On 06/29/2011 01:26 PM, Hans Verkuil wrote: Hi all, This RFC is based on recent discussions with regards to how the poll function in a V4L2 driver should behave. Some relevant documents: POSIX: http://pubs.opengroup.org/onlinepubs/009695399/functions/select.html

Re: [git:xawtv3/master] xawtv: reenable its usage with webcam's

2011-06-29 Thread Hans de Goede
Hi, On 06/29/2011 02:01 PM, Mauro Carvalho Chehab wrote: Em 29-06-2011 08:01, Hans de Goede escreveu: Hi, On 06/28/2011 07:32 PM, Mauro Carvalho Chehab wrote: This is an automatic generated email to let you know that the following patch were queued at the http://git.linuxtv.org/xawtv3.git

Re: RFC: poll behavior

2011-06-29 Thread Hans de Goede
Hi, On 06/29/2011 03:43 PM, Hans Verkuil wrote: On Wednesday, June 29, 2011 15:07:14 Hans de Goede wrote: snip Ok, yet more reason to go with my proposal, but then simplified to: When streaming has not started return POLLIN or POLLOUT (or-ed with POLLPRI if events are pending). So would

Re: Some fixes for alsa_stream

2011-06-16 Thread Hans de Goede
Hi, On 06/15/2011 05:45 PM, Mauro Carvalho Chehab wrote: snip 1) try to find a common buffer size that are acceptable by both drivers, as using the same buffer size helps to avoid memcpy's, especially if mmap mode is enabled; This is not needed, using the same buffer size does

Re: [PATCH] usbvision: remove (broken) image format conversion

2011-06-16 Thread Hans de Goede
Hi, On 06/11/2011 02:13 PM, Mauro Carvalho Chehab wrote: Hi Hans de G., Em 26-04-2011 08:54, Hans de Goede escreveu: If you could give it a shot that would be great. I've some hardware to test this with (although I've never actually tested that hardware), so I can hopefully pick things up

Re: Some fixes for alsa_stream

2011-06-16 Thread Hans de Goede
Hi, On 06/16/2011 04:38 PM, Mauro Carvalho Chehab wrote: Em 16-06-2011 09:29, Hans de Goede escreveu: snip Note that I've just pushed a patch set which includes rewritten period / buf size negotiation and a bunch of cleanups in general. This removes over 150 lines of code, while at the same

Re: Some fixes for alsa_stream

2011-06-16 Thread Hans de Goede
Hi, On 06/16/2011 05:35 PM, Mauro Carvalho Chehab wrote: Em 16-06-2011 12:11, Hans de Goede escreveu: snip While developers are not comfortable with pulseaudio, turning it into a default is a bad idea. 1) Some developers are fine with pulseaudio, so please speak on behalf of yourself

Re: Some fixes for alsa_stream

2011-06-15 Thread Hans de Goede
Hi, On 06/14/2011 04:45 PM, Mauro Carvalho Chehab wrote: Em 14-06-2011 11:37, Hans de Goede escreveu: Hi, On 06/14/2011 04:17 PM, Mauro Carvalho Chehab wrote: Em 14-06-2011 10:52, Devin Heitmueller escreveu: snip Yes. The default for capture is the one detected via sysfs. The default

Re: Some fixes for alsa_stream

2011-06-15 Thread Hans de Goede
Hi, On 06/15/2011 04:25 PM, Mauro Carvalho Chehab wrote: Em 15-06-2011 10:43, Hans de Goede escreveu: snip Right, because ConsoleKit ensures that devices like floppydrives, cdroms, audio cards, webcams, etc. are only available to users sitting behind the console, This is a wrong

Re: Some fixes for alsa_stream

2011-06-14 Thread Hans de Goede
Hi, On 06/14/2011 04:01 AM, Mauro Carvalho Chehab wrote: Hi Devin, I've made a few fixes for your alsa_stream.c, used on tvtime. They are at: http://git.linuxtv.org/xawtv3.git In particular, those are the more interesting ones: commit a1bb5ade5c2b09d6d6d624d18025f9e2c4398495

Re: Some fixes for alsa_stream

2011-06-14 Thread Hans de Goede
Hi, On 06/14/2011 03:05 PM, Mauro Carvalho Chehab wrote: Em 14-06-2011 09:48, Hans de Goede escreveu: Hi, On 06/14/2011 04:01 AM, Mauro Carvalho Chehab wrote: Hi Devin, I've made a few fixes for your alsa_stream.c, used on tvtime. They are at: http://git.linuxtv.org/xawtv3.git

Re: Some fixes for alsa_stream

2011-06-14 Thread Hans de Goede
Hi, On 06/14/2011 04:17 PM, Mauro Carvalho Chehab wrote: Em 14-06-2011 10:52, Devin Heitmueller escreveu: snip Yes. The default for capture is the one detected via sysfs. The default for playback is not really hw:0,0. It defaults to the first hw: that it is not associated with a video

Re: Improving kernel - userspace (usbfs) usb device hand off

2011-06-12 Thread Hans de Goede
Hi, On 06/11/2011 06:19 PM, Theodore Kilgore wrote: On Sat, 11 Jun 2011, Hans de Goede wrote: Hi, Given the many comments in this thread, I'm just going reply to this one, and try to also answer any other ones in this mail. As far as the dual mode camera is involved, I agree

Re: Improving kernel - userspace (usbfs) usb device hand off

2011-06-11 Thread Hans de Goede
, Hans de Goede wrote: Hi all, The current API for managing kernel - userspace is a bit rough around the edges, so I would like to discuss extending the API. First of all an example use case scenarios where the current API falls short. 1) Redirection of USB devices to a virtual machine, qemu, vbox

Re: USB mini-summit at LinuxCon Vancouver

2011-06-10 Thread Hans de Goede
Hi, On 06/10/2011 02:21 AM, Sarah Sharp wrote: I'm pleased to announce a USB mini-summit at LinuxCon Vancouver. What: USB mini-summit When: Tuesday, August 16th, all day Where: At the conference venue, room TBD pending confirmation from Angela Brown. Proposed topics include USB

Improving kernel - userspace (usbfs) usb device hand off

2011-06-10 Thread Hans de Goede
Hi all, The current API for managing kernel - userspace is a bit rough around the edges, so I would like to discuss extending the API. First of all an example use case scenarios where the current API falls short. 1) Redirection of USB devices to a virtual machine, qemu, vbox, etc. all have the

Re: Improving kernel - userspace (usbfs) usb device hand off

2011-06-10 Thread Hans de Goede
Hi, On 06/10/2011 10:22 AM, Felipe Balbi wrote: Hi, On Fri, Jun 10, 2011 at 09:55:13AM +0200, Hans de Goede wrote: Currently this will cause the usb mass storage driver to see a disconnect, and any possible still pending writes are lost ... This is IMHO unacceptable, but currently

Re: Improving kernel - userspace (usbfs) usb device hand off

2011-06-10 Thread Hans de Goede
Hi, On 06/10/2011 10:42 AM, Felipe Balbi wrote: Hi, On Fri, Jun 10, 2011 at 10:36:47AM +0200, Hans de Goede wrote: On Fri, Jun 10, 2011 at 09:55:13AM +0200, Hans de Goede wrote: snip So what do we need to make this situation better: 1) A usb_driver callback alternative to the disconnect

Crash on unplug with the uvc driver in linuxtv/staging/for_v3.1

2011-06-09 Thread Hans de Goede
Hi, When I unplug a uvc camera *while streaming* I get: Jun 9 13:20:02 shalem kernel: [15824.809741] BUG: unable to handle kernel NULL pointer dereference at (null) Jun 9 13:20:02 shalem kernel: [15824.809816] IP: [a0309eae] media_entity_put+0x12/0x2c [media] Jun 9

Re: Anchor Chips V4L2 driver

2011-06-07 Thread Hans de Goede
Hi, On 06/07/2011 07:24 AM, John McMaster wrote: On 06/06/2011 12:43 AM, Hans de Goede wrote: Hi, On 06/06/2011 07:40 AM, John McMaster wrote: On 06/03/2011 06:22 AM, Hans de Goede wrote: Hi, On 06/03/2011 02:15 PM, Mauro Carvalho Chehab wrote: Em 03-06-2011 02:40, John McMaster escreveu

Re: Anchor Chips V4L2 driver

2011-06-06 Thread Hans de Goede
Hi, On 06/06/2011 07:40 AM, John McMaster wrote: On 06/03/2011 06:22 AM, Hans de Goede wrote: Hi, On 06/03/2011 02:15 PM, Mauro Carvalho Chehab wrote: Em 03-06-2011 02:40, John McMaster escreveu: I'd like to write a driver for an Anchor Chips (seems to be bought by Cypress) USB camera Linux

[PATCH for 3.0] pwc fix oops / reference of free-ed memory on unplug

2011-06-06 Thread Hans de Goede
Hi Mauro et all, I'm working on (much needed) large cleanup + v4l2 compliance series for the pwc driver, more on that later (targetting 3.1). But this particular patch (the first of the series) seems one which we should have in 3.0 and stable releases for older kernels too, since it fixes a real

[PATCH] pwc: better usb disconnect handling

2011-06-06 Thread Hans de Goede
the embedded v4l2 dev structure. Signed-off-by: Hans de Goede hdego...@redhat.com --- drivers/media/video/pwc/pwc-ctrl.c |2 +- drivers/media/video/pwc/pwc-if.c | 152 +++- drivers/media/video/pwc/pwc.h |4 +- 3 files changed, 50 insertions(+), 108

Which error code to return when a usb camera gets unplugged

2011-06-06 Thread Hans de Goede
Hi, While working on my cleanup / v4l2 compliance series for the pwc driver I've noticed that the pwc and gspca drivers are doing different things wrt what error they return to an app is using the device while it gets unplugged. gspca returns -ENODEV, where as pwc returns -EPIPE. Both make some

Re: [PATCH] Increase max exposure value to 255 from 26.

2011-06-04 Thread Hans de Goede
Hi, Interesting. I'll go and test this with my 6 or so pac207 cameras, but first I need to wait till this evening as atm it is too light to test high exposure settings :) Regards, Hans On 06/04/2011 09:38 AM, Marco Diego Aurélio Mesquita wrote: The inline patch increases maximum exposure

Re: [RFCv2] Add a library to retrieve associated media devices - was: Re: [ANNOUNCE] experimental alsa stream support at xawtv3

2011-05-29 Thread Hans de Goede
Hi, On 05/29/2011 01:19 PM, Hans Verkuil wrote: Hi Mauro, Thanks for the RFC! Some initial comments below. I'll hope to do some more testing and reviewing in the coming week. Snip c) get_not_associated_device: Returns the next device not associated with an

Re: [RFCv2] Add a library to retrieve associated media devices - was: Re: [ANNOUNCE] experimental alsa stream support at xawtv3

2011-05-29 Thread Hans de Goede
Hi, On 05/29/2011 03:08 PM, Mauro Carvalho Chehab wrote: Em 29-05-2011 08:54, Hans de Goede escreveu: Hi, On 05/29/2011 01:19 PM, Hans Verkuil wrote: Hi Mauro, Thanks for the RFC! Some initial comments below. I'll hope to do some more testing and reviewing in the coming week. Snip c

Re: [ANNOUNCE] experimental alsa stream support at xawtv3

2011-05-28 Thread Hans de Goede
Hi, On 05/28/2011 02:17 PM, Mauro Carvalho Chehab wrote: Em 26-05-2011 03:53, Hans Verkuil escreveu: On Tuesday, May 24, 2011 16:57:22 Devin Heitmueller wrote: On Tue, May 24, 2011 at 2:50 AM, Hans Verkuilhverk...@xs4all.nl wrote: On Monday, May 23, 2011 22:17:06 Mauro Carvalho Chehab

Re: [ANNOUNCE] experimental alsa stream support at xawtv3

2011-05-24 Thread Hans de Goede
Hi, On 05/24/2011 08:50 AM, Hans Verkuil wrote: On Monday, May 23, 2011 22:17:06 Mauro Carvalho Chehab wrote: Due to the alsa detection code that I've added at libv4l2util (at v4l2-utils) during the weekend, I decided to add alsa support also on xawtv3, basically to provide a real usecase

Re: [ANNOUNCE] experimental alsa stream support at xawtv3

2011-05-24 Thread Hans de Goede
Hi, On 05/24/2011 04:09 PM, Mauro Carvalho Chehab wrote: Em 24-05-2011 04:21, Hans de Goede escreveu: Hi, My I suggest that we instead just copy over the single get_media_devices.c file to xawtv, and not install the not so much a lib lib ? If we do that, then all other places where

Re: [git:v4l-utils/master] Add an install target to libv4l2util

2011-05-23 Thread Hans de Goede
Hi, On 05/23/2011 12:00 PM, Mauro Carvalho Chehab wrote: This is an automatic generated email to let you know that the following patch were queued at the http://git.linuxtv.org/v4l-utils.git tree: Subject: Add an install target to libv4l2util Author: Mauro Carvalho Chehabmche...@redhat.com

Re: [libv4l-mcplugin PATCH 0/3] Media controller plugin for libv4l2

2011-05-20 Thread Hans de Goede
On 05/19/2011 02:36 PM, Yordan Kamenov wrote: Hi, This is the Media Controller plugin for libv4l. It uses libv4l2 plugin support which is accepted by Hans De Goede, but not yet included in mainline libv4l2: http://www.spinics.net/lists/linux-media/msg32017.html The plugin allows a traditional v4l2

Re: [libv4l] [PATCH] Webcam image upside down on Asus Eee PC T101MT (13d3:5122)

2011-05-08 Thread Hans de Goede
Hi, Thanks, applied. Regards, Hans On 05/06/2011 08:04 AM, Stefan Löffler wrote: The caption pretty much says it all. Owing to different IDs, the flags for similar Asus products don't cut in. Originally reported for Ubuntu at https://bugs.launchpad.net/ubuntu/+source/libv4l/+bug/774123.

Re: [PATCH v4 1/1] libv4l: Add plugin support to libv4l

2011-05-05 Thread Hans de Goede
Hi, Thanks, looks good now! I'm going to keep this in my inbox before applying for a while though, as this involves an abi change (of libv4lconvert, which is almost not used directly but still). I've a number of important fixes / changes planned which I hope to be able to do soon, and then I

Re: [PATCH 1/1 v3] libv4l: Add plugin support to libv4l

2011-04-29 Thread Hans de Goede
Hi, On 04/29/2011 10:55 AM, Yordan Kamenov wrote: Hans de Goede wrote: Hi, Hi Hans, Thanks for your comments. First of all my apologies for taking so long to get around to reviewing this. Over all it looks good, I've put some small remarks inline, if you fix these I can merge this. I

Re: [PATCH 1/1 v3] libv4l: Add plugin support to libv4l

2011-04-28 Thread Hans de Goede
Hi, First of all my apologies for taking so long to get around to reviewing this. Over all it looks good, I've put some small remarks inline, if you fix these I can merge this. I wonder though, given the recent limbo around Nokia's change of focus, if there are any plans to actually move

Re: [RFC, PATCH] libv4lconvert: Add support for Y10B grey format (V4L2_PIX_FMT_Y10BPACK)

2011-04-27 Thread Hans de Goede
Hi, sorry for being a bit slow ... On 04/18/2011 12:25 PM, Antonio Ospite wrote: On Mon, 11 Apr 2011 23:07:36 +0200 Hans de Goedehdego...@redhat.com wrote: [...] I don't know libv4l yet, so I am asking for advice providing some code to discuss on; looking at the last hunk of the patch: can

Re: [PATCH] usbvision: remove (broken) image format conversion

2011-04-26 Thread Hans de Goede
Hi, On 04/26/2011 08:32 AM, Hans Verkuil wrote: On Monday, April 25, 2011 23:23:17 Ondrej Zary wrote: The YVU420 and YUV422P formats are broken and cause kernel panic on use. (YVU420 does not work and sometimes causes unable to handle paging request panic, YUV422P always causes NULL pointer

Re: [PATCH] usbvision: remove (broken) image format conversion

2011-04-26 Thread Hans de Goede
Hi, On 04/26/2011 10:30 AM, Ondrej Zary wrote: On Tuesday 26 April 2011, you wrote: On Monday, April 25, 2011 23:23:17 Ondrej Zary wrote: The YVU420 and YUV422P formats are broken and cause kernel panic on use. (YVU420 does not work and sometimes causes unable to handle paging request panic,

Re: Driver for r5u870 USB webcams

2011-04-21 Thread Hans de Goede
Hi, On 04/21/2011 03:08 PM, Jon Mason wrote: My laptop has the Ricoh Co., Ltd Visual Communication Camera VGP-VCC7 [R5U870] webcam. A quick scan of the kernel does not show the USB ID listed. `lsusb` has it listed as: Bus 001 Device 005: ID 05ca:183a Ricoh Co., Ltd Visual Communication

Re: [PATCH] libv4lconvert-priv.h: indent with tabs, not spaces

2011-04-11 Thread Hans de Goede
Thanks, applied. On 04/07/2011 05:57 PM, Antonio Ospite wrote: Indent wrapped lines with tabs, just like it is done for the other functions in the same file. Signed-off-by: Antonio Ospiteosp...@studenti.unina.it --- lib/libv4lconvert/libv4lconvert-priv.h |2 +- 1 files changed, 1

Re: [RFC, PATCH] libv4lconvert: Add support for Y10B grey format (V4L2_PIX_FMT_Y10BPACK)

2011-04-11 Thread Hans de Goede
Hi, On 04/07/2011 06:16 PM, Antonio Ospite wrote: Y10B is a 10 bits per pixel greyscale format in a bit-packed array representation. Such pixel format is supplied for instance by the Kinect sensor device. Signed-off-by: Antonio Ospiteosp...@studenti.unina.it --- Hi, this is a very first

[GIT PATCHES FOR 2.6.39] Add support for cpia1 camera button

2011-03-14 Thread Hans de Goede
repository at: git://linuxtv.org/hgoede/gspca.git gspca-for_v2.6.39 Hans de Goede (1): gspca_cpia1: Add support for button drivers/media/video/gspca/cpia1.c | 31 ++- 1 files changed, 26 insertions(+), 5 deletions(-) Thanks, Hans -- To unsubscribe from this list

Re: Missing V4L2_PIX_FMT_JPGL

2011-03-14 Thread Hans de Goede
Hi, On 03/14/2011 11:16 AM, Hans Verkuil wrote: Hi Hans, I just copied the latest videobuf2.h to v4l-utils and tried to compile, but it fails with: make[2]: Entering directory `/home/hve/work/src/v4l/v4l-utils/lib/libv4lconvert' cc -Wp,-MMD,libv4lconvert.d,-MQ,libv4lconvert.o,-MP -c

Re: What is the driver for 0c45:6413 Microdia webcam?

2011-03-11 Thread Hans de Goede
Hi, On 03/11/2011 07:04 PM, Chen, Xianwen wrote: Hi there, I'm having a hard time locating the proper driver for 0c45:6413 Microdia webcam. I consulted Documentation/video4linux/gspca.txt, but didn't find such a device. The 6413 is a UVC compatible camera / controller from Microdia, as such

Re: [PATCH 0/1 v3] libv4l: Add plugin support

2011-03-10 Thread Hans de Goede
Hi, On 03/10/2011 02:46 PM, Yordan Kamenov wrote: Hi Hans, any comments on that? I looked at globally immediately after you send it, summary of that look: looks good, but I still spotted some minor issues. I'm still trying to make some time for a more detailed look. I hope to do that real

Re: v4l-utils-0.8.3 and KVDR

2011-02-20 Thread Hans de Goede
Hi, On 02/20/2011 12:48 AM, Mike Booth wrote: My understanding of the wrapperscontained in this library is that v4l applications should work with kernels from 2.6.36 onwards if the compat.so is preloaded. I use KVDR for watching and controlling VDR on my TV. Xine and Xineliboutput or not

[GIT PATCHES FOR 2.6.39] gspca_sn9c20x fixes

2011-02-17 Thread Hans de Goede
repository at: git://linuxtv.org/hgoede/gspca.git gspca-for_v2.6.39 Hans de Goede (5): gspca_sn9c20x: Fix colored borders with ov7660 sensor gspca_sn9c20x: Add hflip and vflip controls for the ov7660 sensor gspca_sn9c20x: Add LED_REVERSE flag for 0c45:62bb gspca_sn9c20x: Make

Re: [PATCH] v4l-utils: Add the JPEG Lite decoding function

2011-02-15 Thread Hans de Goede
Hi, On 02/15/2011 10:26 AM, Jean-Francois Moine wrote: Hi Hans, I got the permission to relicense the JPEG Lite decoding to the LGPL. Ah, good, patch applied and pushed to git. If you want to test the nw80x driver, get the gspca tarball from my web page (2.12.12). I added your webcam

Re: [PATCH] v4l-utils: Add the JPEG Lite decoding function

2011-02-14 Thread Hans de Goede
Hi, On 02/14/2011 01:36 PM, Jean-Francois Moine wrote: Hi Hans, JPEG Lite images are created by the DivIO nw80x chips. The gspca subdriver nw80x will be ready soon. Decoding to RGB24 and BGR24 are ok. Decoding to YUV420 and YVU420 are not tested. You're working on nw80x support? Cool!

What to do with videodev.h

2011-01-26 Thread Hans de Goede
Hi All, With v4l1 support going completely away, the question is raised what to do with linux/videodev.h . Since v4l1 apps can still use the old API through libv4l1, these apps will still need linux/videodev.h to compile. So I see 3 options: 1) Keep videodev.h in the kernel tree even after

Announcing v4l-utils-0.8.2

2011-01-25 Thread Hans de Goede
Hi, I'm happy to announce the release of v4l-utils-0.8.2 This release features many small fixes, esp. to the ir-keytable util (which now comes with udev rules and a manpage), qv4l2, v4l2-ctl and v4l2-compliance. v4l-utils-0.8.2 --- * Utils changes: * Various ir-keytable

Re: [RFC PATCH 2/3] v4l2-ctrls: add v4l2_ctrl_auto_cluster to simplify autogain/gain scenarios

2011-01-25 Thread Hans de Goede
Hi, On 01/23/2011 05:13 PM, Hans Verkuil wrote: On Sunday, January 23, 2011 16:15:03 Hans de Goede wrote: snip This is what the UVC spec for example mandates and what the current UVC driver does. Combining this with an app which honors the update and the read only flag (try gtk-v4l

Re: [RFC PATCH 2/3] v4l2-ctrls: add v4l2_ctrl_auto_cluster to simplify autogain/gain scenarios

2011-01-23 Thread Hans de Goede
Hi, On 01/22/2011 12:06 PM, Hans Verkuil wrote: It is a bit tricky to handle autogain/gain type scenerios correctly. Such controls need to be clustered and the V4L2_CTRL_FLAG_UPDATE should be set on the non-auto controls. If you set a non-auto control without setting the auto control at the

Re: Upstreaming syntek driver

2011-01-20 Thread Hans de Goede
Hi, On 01/20/2011 12:35 PM, Luca Tettamanti wrote: On Tue, Jan 18, 2011 at 11:45 PM, Hans Verkuilhverk...@xs4all.nl wrote: [...] After a quick scan through the sources in svn I found the following (in no particular order): - Supports easycap model with ID 05e1:0408: a driver for this model

Re: [PATCH 1/1 v2] Add plugin support to libv4l

2011-01-16 Thread Hans de Goede
Hi, Thanks for the new patch, it looks much better. Unfortunately I found what I can only describe as a bug in the plugin rfc (which as you probasbly know I wrote). The problem is 2 fold: 1) The existence of v4l2_fd_open (and its active use by multiple applications) means that we cannot let

Re: [PATCH][rfc] media, video, stv06xx, pb0100: Don't potentially deref NULL in pb0100_start().

2011-01-14 Thread Hans de Goede
Hi, On 01/13/2011 11:05 PM, Jesper Juhl wrote: usb_altnum_to_altsetting() may return NULL. If it does we'll dereference a NULL pointer in drivers/media/video/gspca/stv06xx/stv06xx_pb0100.c::pb0100_start(). As far as I can tell there's not really anything more sensible than -ENODEV that we can

Re: RFC: Move the deprecated et61x251 and sn9c102 to staging

2011-01-10 Thread Hans de Goede
Hi, On 01/10/2011 02:33 AM, Mauro Carvalho Chehab wrote: Em 09-01-2011 10:02, Hans de Goede escreveu: snip I've managed to make some time to also sort out the sn9c1xx usb ids situation. I've just send a pull request which includes patches cleaning things up. After this there are only 5 usb

Re: RFC: Move the deprecated et61x251 and sn9c102 to staging

2011-01-09 Thread Hans de Goede
Hi, On 01/02/2011 09:13 PM, Hans Verkuil wrote: Hi Hans, On Sunday, January 02, 2011 19:33:31 Hans de Goede wrote: snip So only 3 raw bayer + custom compression models supported by sn9c102 are not supported by gspca_sonixb, and all jpeg models are supported by gspca_sonixj. Porting the 3

Re: [PATCH 1/1] Add plugin support to libv4l

2011-01-08 Thread Hans de Goede
Hi, First of all many thanks for working on this! I've several remarks which I would like to see addressed before merging this. Since most remarks are rather high level remarks I've opted to just make a bulleted list of them rather then inserting them inline. * The biggest problem with your

<    2   3   4   5   6   7   8   9   10   >