Re: [RFCv3 PATCH 12/18] vb2_poll: don't start DMA, leave that to the first read().

2011-06-28 Thread Hans Verkuil
On Tuesday, June 28, 2011 15:58:36 Hans Verkuil wrote: > On Tuesday, June 28, 2011 14:43:22 Mauro Carvalho Chehab wrote: > > Em 28-06-2011 09:21, Andy Walls escreveu: > > > Mauro Carvalho Chehab wrote: > > > > >> I'm not very comfortable with vb2 returning unexpected errors there. > > >> Also, >

Re: [RFCv3 PATCH 12/18] vb2_poll: don't start DMA, leave that to the first read().

2011-06-28 Thread Andy Walls
On Tue, 2011-06-28 at 21:00 -0300, Mauro Carvalho Chehab wrote: > Em 28-06-2011 20:14, Andy Walls escreveu: > > On Tue, 2011-06-28 at 09:43 -0300, Mauro Carvalho Chehab wrote: > >> Em 28-06-2011 09:21, Andy Walls escreveu: > > > >>> It is also the case that a driver's poll method should never slee

Re: HVR-1250/CX23885 IR Rx

2011-06-28 Thread Andy Walls
On Tue, 2011-06-28 at 22:17 -0400, Jarod Wilson wrote: > On Jun 28, 2011, at 6:32 PM, Andy Walls wrote: > > > On Tue, 2011-06-28 at 17:39 -0400, Jarod Wilson wrote: > > > >> I'm thinking that in the > >> short-term, we should probably make sure msi doesn't get enabled in > >> the cx23885 driver,

Re: HVR-1250/CX23885 IR Rx

2011-06-28 Thread Jarod Wilson
On Jun 28, 2011, at 6:32 PM, Andy Walls wrote: > On Tue, 2011-06-28 at 17:39 -0400, Jarod Wilson wrote: > >> Up and running on 3.0-rc5 now, and I'm not seeing the panic, but the >> box keeps hard-locking after some number of keypresses. Can't get a >> peep out of it with sysrq, nmi watchdog doesn

[GIT PATCHES FOR 3.0] v4l core and uvcvideo fixes

2011-06-28 Thread Laurent Pinchart
Hi Mauro, The following changes since commit b0af8dfdd67699e25083478c63eedef2e72ebd85: Linux 3.0-rc5 (2011-06-27 19:12:22 -0700) are available in the git repository at: git://linuxtv.org/pinchartl/uvcvideo.git uvcvideo-stable Laurent Pinchart (2): v4l: Don't access media entity after

Re: [RFCv3 PATCH 12/18] vb2_poll: don't start DMA, leave that to the first read().

2011-06-28 Thread Mauro Carvalho Chehab
Em 28-06-2011 20:14, Andy Walls escreveu: > On Tue, 2011-06-28 at 09:43 -0300, Mauro Carvalho Chehab wrote: >> Em 28-06-2011 09:21, Andy Walls escreveu: > >>> It is also the case that a driver's poll method should never sleep. >> >> True. > >>> One issue is how to start streaming with apps that:

Re: [RFCv3 PATCH 12/18] vb2_poll: don't start DMA, leave that to the first read().

2011-06-28 Thread Andy Walls
On Tue, 2011-06-28 at 09:43 -0300, Mauro Carvalho Chehab wrote: > Em 28-06-2011 09:21, Andy Walls escreveu: > > It is also the case that a driver's poll method should never sleep. > > True. > > One issue is how to start streaming with apps that: > > - Open /dev/video/ in a nonblocking mode, and

Re: FW: OMAP 3 ISP

2011-06-28 Thread Laurent Pinchart
Hi Alex, On Tuesday 28 June 2011 09:47:07 Alex Gershgorin wrote: > Hi Laurent, > > I recently got the Zoom OMAP35xx Torpedo and began testing OMAP3ISP. > Currently I have a problem I can't solve. > See Message from booting Kernel: > > Linux media interface: v0.10 > Linux video capture interface:

Re: HVR-1250/CX23885 IR Rx

2011-06-28 Thread Andy Walls
On Tue, 2011-06-28 at 17:39 -0400, Jarod Wilson wrote: > Up and running on 3.0-rc5 now, and I'm not seeing the panic, but the > box keeps hard-locking after some number of keypresses. Can't get a > peep out of it with sysrq, nmi watchdog doesn't seem to fire, etc. > > At the suggestion of "Dark S

Re: HVR-1250/CX23885 IR Rx

2011-06-28 Thread Jarod Wilson
On Jun 28, 2011, at 6:30 AM, Andy Walls wrote: > Jarod Wilson wrote: > >> On Apr 10, 2011, at 7:08 PM, Andy Walls wrote: >> >>> On Sat, 2011-04-09 at 21:39 -0400, Jarod Wilson wrote: >>> > Jarod, > > The HVR-1850 uses a raw IR receiver in the CX23888 and older HVR-1250s use t

Patch proposition for DVB-T configuration file for Paris area

2011-06-28 Thread Bogdan Cristea
Hi I would like to propose the attached patch for de DVB-T configuration file for Paris area (found in openSUSE 11.4 in this location) /usr/share/dvb/dvb-t/fr-Paris With the current configuration file only 5 channels are found, while in reality there are almost 35 channels. regards -- Bogdan

Genius TVGo C03 TV Card not recognised

2011-06-28 Thread Thomas Spigel
Hello, The em28xx module can't recognise my tvcard on my linux ubuntu 11.04. Here is the dmesg log. Please tell me if there's any way to run this card on my system. If not, i would like to help you to include it in your modules. CARD: Genius TVGo C03 hybrid tvcard DMESG LOG: [13291.629753] em28xx

[cron job] v4l-dvb daily build: ERRORS

2011-06-28 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:Tue Jun 28 19:00:37 CEST 2011 git hash:7023c7dbc3944f42aa1d6910a6098c5f9e23d3f1 gcc version: i686-linux-gcc (GCC) 4.5

Re: [PATCHv2 10/13] [media] gspca: don't include linux/version.h

2011-06-28 Thread Jean-Francois Moine
On Mon, 27 Jun 2011 23:17:34 -0300 Mauro Carvalho Chehab wrote: > Instead of handling a per-driver driver version, use the > per-subsystem one. > > As reviewed by Jean-Francois Moine : > - the 'info' may be simplified: > > Reviewed-by: Jean-Francois Moine > Acked-by: Hans Verkuil > Sign

Re: [PATCH] [media] v4l2 core: return -ENOIOCTLCMD if an ioctl doesn't exist

2011-06-28 Thread Arnd Bergmann
On Tuesday 28 June 2011, Alan Cox wrote: > > (In fact, the correct thing to do would probably be to just do > > > >#define ENOIOCTLCMD ENOTTY > > > > and get rid of any translation - just giving ENOTTY a more appropriate > > name and less chance for confusion) > > Some code uses the two to s

Re: FM RDS receiver

2011-06-28 Thread Markus Rechberger
On Tue, Jun 28, 2011 at 12:02 AM, Helmut Auer wrote: > Hello List, > > Can you recommend any FM receiver card or usb stick currently available in > Germany with RDS support ? We do have some FM Receiver/Transmitters which support RDS, however they are still only available upon request at this tim

Re: [PATCH] [media] v4l2 core: return -ENOIOCTLCMD if an ioctl doesn't exist

2011-06-28 Thread Alan Cox
> (In fact, the _correct_ thing to do would probably be to just do > >#define ENOIOCTLCMD ENOTTY > > and get rid of any translation - just giving ENOTTY a more appropriate > name and less chance for confusion) Some code uses the two to separate 'the driver specific helper code doesn't handle

[PATCHv2 00/13] Remove linux/version.h from most drivers/media/

2011-06-28 Thread Mauro Carvalho Chehab
At the V4L2 API, one of the fields of VIDIOC_QUERYCAP requires the usage of KERNEL_VERSION macro, in order to provide the driver version. However, this is not handled consistently across subsystems. There are very few drivers that take it seriously. So, instead of the current way, let's replace it

[PATCHv2 10/13] [media] gspca: don't include linux/version.h

2011-06-28 Thread Mauro Carvalho Chehab
Instead of handling a per-driver driver version, use the per-subsystem one. As reviewed by Jean-Francois Moine : - the 'info' may be simplified: Reviewed-by: Jean-Francois Moine Acked-by: Hans Verkuil Signed-off-by: Mauro Carvalho Chehab diff --git a/drivers/media/video/gspca/gspca.c

[PATCHv2 13/13] [media] DocBook/v4l: Document the new system-wide version behavior

2011-06-28 Thread Mauro Carvalho Chehab
Acked-by: Hans Verkuil Signed-off-by: Mauro Carvalho Chehab diff --git a/Documentation/DocBook/media/v4l/common.xml b/Documentation/DocBook/media/v4l/common.xml index 9028721..a86f7a0 100644 --- a/Documentation/DocBook/media/v4l/common.xml +++ b/Documentation/DocBook/media/v4l/common.xml @@ -23

[PATCHv2 09/13] [media] uvcvideo: Use LINUX_VERSION_CODE for VIDIOC_QUERYCAP

2011-06-28 Thread Mauro Carvalho Chehab
uvcvideo doesn't use vidioc_ioctl2. As the API is changing to use a common version for all drivers, we need to expliticly fix this driver. Acked-by: Hans Verkuil Signed-off-by: Mauro Carvalho Chehab diff --git a/drivers/media/video/uvc/uvc_driver.c b/drivers/media/video/uvc/uvc_driver.c index

[PATCHv2 07/13] [media] pvrusb2: Use LINUX_VERSION_CODE for VIDIOC_QUERYCAP

2011-06-28 Thread Mauro Carvalho Chehab
pvrusb2 doesn't use vidioc_ioctl2. As the API is changing to use a common version for all drivers, we need to expliticly fix this driver. Acked-by: Hans Verkuil Signed-off-by: Mauro Carvalho Chehab diff --git a/drivers/media/video/pvrusb2/pvrusb2-main.c b/drivers/media/video/pvrusb2/pvrusb2-ma

[PATCHv2 08/13] [media] sn9c102: Use LINUX_VERSION_CODE for VIDIOC_QUERYCAP

2011-06-28 Thread Mauro Carvalho Chehab
sn9c102 doesn't use vidioc_ioctl2. As the API is changing to use a common version for all drivers, we need to expliticly fix this driver. Acked-by: Hans Verkuil Signed-off-by: Mauro Carvalho Chehab diff --git a/drivers/media/video/sn9c102/sn9c102.h b/drivers/media/video/sn9c102/sn9c102.h index

[PATCHv2 11/13] [media] Stop using linux/version.h the remaining video drivers

2011-06-28 Thread Mauro Carvalho Chehab
Standardize the remaining video drivers to return the API version for the VIDIOC_QUERYCAP version, instead of a per-driver version. Those drivers had the version updated more recently or are SoC drivers. Even so, it doesn't sound a good idea to keep a per-driver version control, so, let's use the

[PATCHv2 06/13] [media] et61x251: Use LINUX_VERSION_CODE for VIDIOC_QUERYCAP

2011-06-28 Thread Mauro Carvalho Chehab
et61x251 doesn't use vidioc_ioctl2. As the API is changing to use a common version for all drivers, we need to expliticly fix this driver. Acked-by: Hans Verkuil Signed-off-by: Mauro Carvalho Chehab diff --git a/drivers/media/video/et61x251/et61x251.h b/drivers/media/video/et61x251/et61x251.h

[PATCHv2 05/13] [media] ivtv,cx18: Use default version control for VIDIOC_QUERYCAP

2011-06-28 Thread Mauro Carvalho Chehab
After discussing with Andy Walls on irc, we've agreeded that this is the best thing to do. No regressions will be introduced, as 3.x.y is greater then the current versions for cx18 and ivtv. Acked-by: Hans Verkuil Signed-off-by: Mauro Carvalho Chehab diff --git a/drivers/media/video/cx18/cx18-d

[PATCHv2 02/13] [media] drxd, siano: Remove unused include linux/version.h

2011-06-28 Thread Mauro Carvalho Chehab
Both drxd and siano drivers were including linux/version.h without any reason. Probably, this is due to some compatibility code that used to exist before having their support added into the Linux Kernel. Acked-by: Hans Verkuil Signed-off-by: Mauro Carvalho Chehab diff --git a/drivers/media/dvb/

[PATCHv2 12/13] [media] radio: Use the subsystem version control for VIDIOC_QUERYCAP

2011-06-28 Thread Mauro Carvalho Chehab
Just like the video drivers, the right thing to do is to use the per-subsystem version control. Acked-by: Hans Verkuil Signed-off-by: Mauro Carvalho Chehab diff --git a/drivers/media/radio/dsbr100.c b/drivers/media/radio/dsbr100.c index 3d8cc42..25e58cb 100644 --- a/drivers/media/radio/dsbr100.

[PATCHv2 04/13] [media] pwc: Use the default version for VIDIOC_QUERYCAP

2011-06-28 Thread Mauro Carvalho Chehab
After discussing with Hans, change pwc to use the default version control. The only version ever used for pwc driver is 10.0.12, due to commit 2b455db6d456ef2d44808a8377fd3bc832e08317. Changing it to 3.x.y won't conflict with the old version. There's no namespace conflicts in any predictable futu

[PATCHv2 03/13] [media] Stop using linux/version.h on most video drivers

2011-06-28 Thread Mauro Carvalho Chehab
All the modified drivers didn't have any version increment since Jan, 1 2011. Several of them didn't have any version increment for a long time, even having new features and important bug fixes happening. As we're now filling the QUERYCAP version with the current Kernel Release, we don't need to m

[PATCHv2 01/13] [media] v4l2-ioctl: Add a default value for kernel version

2011-06-28 Thread Mauro Carvalho Chehab
Most drivers don't increase kernel versions as newer features are added or bug fixes are solved. So, vidioc_querycap returned value for cap->version is meaningless. Instead of keeping this situation forever, let's add a default value matching the current Linux version. Drivers that want to keep th

[PATCH] Fix locking problem in vivi

2011-06-28 Thread Hans Verkuil
Fix sleep-in-atomic-context bug in vivi (you need to run with lock debugging on): Jun 28 18:14:39 tschai kernel: [ 80.970478] BUG: sleeping function called from invalid context at kernel/mutex.c:271 Jun 28 18:14:39 tschai kernel: [ 80.970483] in_atomic(): 0, irqs_disabled(): 1, pid: 2854, n

Re: [PATCH] [media] v4l2 core: return -ENOIOCTLCMD if an ioctl doesn't exist

2011-06-28 Thread Linus Torvalds
On Mon, Jun 27, 2011 at 11:04 PM, Hans Verkuil wrote: > > Of course, since vfs_ioctl does the same thing this is clearly isn't some > isolated V4L thing. Pot. Kettle. Oh, absolutely. V4L is not at all the only confused user around. The EINVAL thing goes way back, and is a disaster. It predates L

[RFC PATCH] Add support for V4L2_EVENT_SUB_FL_NO_FEEDBACK

2011-06-28 Thread Hans Verkuil
Originally no control events would go to the filehandle that called the VIDIOC_S_CTRL/VIDIOC_S_EXT_CTRLS ioctls. This was to prevent a feedback loop. This is now only done if the new V4L2_EVENT_SUB_FL_NO_FEEDBACK flag is set. Based on a suggestion from Mauro Carvalho Chehab . This goes on top of

[RFC PATCH] capture-example: allow V4L2_PIX_FMT_GREY with USERPTR

2011-06-28 Thread Michael Jones
There is an assumption that the format coming from the device needs 2 bytes per pixel, which is not the case when the device delivers e.g. V4L2_PIX_FMT_GREY. This doesn't manifest itself with IO_METHOD_MMAP because init_mmap() (the default) doesn't take sizeimage as an argument. Signed-off-by: Mic

[GIT PATCHES FOR 3.1] Add Control Event and autofoo/foo support

2011-06-28 Thread Hans Verkuil
Hi Mauro, This is the same as the first pull request, except for: - the poll patch has been removed (to be discussed further) - the corresponding poll changes in ivtv were removed (pending same discussion). - a new patch has been added that fixes your manual_mode_value comments. Note that I ha

Re: [RFCv3 PATCH 12/18] vb2_poll: don't start DMA, leave that to the first read().

2011-06-28 Thread Hans Verkuil
On Tuesday, June 28, 2011 14:43:22 Mauro Carvalho Chehab wrote: > Em 28-06-2011 09:21, Andy Walls escreveu: > > Mauro Carvalho Chehab wrote: > > >> I'm not very comfortable with vb2 returning unexpected errors there. > >> Also, > >> for me it is clear that, if read will fail, POLLERR should be ri

Re: [PATCH] Fix suspend/resume of pxa_camera driver

2011-06-28 Thread Guennadi Liakhovetski
Hi Robert Hope you don't mind me resuming an almost 3 year old mail thread;) I'm referring to your patches to soc-camera core and pxa-camera, adding PM support to them. Below is your message again, explaining, why the standard pm hooks cannot be used to suspend and resume the camera host and th

Help getting TechniSat SkyStar HD2 working in Ubuntu 10.10

2011-06-28 Thread John Taylor
I'm having problems with this card- I can get it to scan DVB-S and DVB-S2 using scan-s2, and can get it to tune to DVB-S channels, but I cannot szap to DVB-S2 channels szap2 -p -t 2 "BBC HD" reports error ioctl DVBFE_SET_DELSYS failed: Operation not supported I have tried the latest s2-liplianin

Re: [GIT PATCHES FOR 3.1] Add Control Event and autofoo/foo support

2011-06-28 Thread Hans Verkuil
On Tuesday, June 28, 2011 14:53:22 Hans Verkuil wrote: > Hi Mauro, > > This is the same as the first pull request, except for: > > - the poll patch has been removed (to be discussed further) > - a new patch has been added that fixes your manual_mode_value comments. I've removed this branch: with

[GIT PATCHES FOR 3.1] Add Control Event and autofoo/foo support

2011-06-28 Thread Hans Verkuil
Hi Mauro, This is the same as the first pull request, except for: - the poll patch has been removed (to be discussed further) - a new patch has been added that fixes your manual_mode_value comments. Note that I have left out the 'event feedback' fix in this patch series. I'm going to make a patc

Re: [RFCv3 PATCH 12/18] vb2_poll: don't start DMA, leave that to the first read().

2011-06-28 Thread Mauro Carvalho Chehab
Em 28-06-2011 09:21, Andy Walls escreveu: > Mauro Carvalho Chehab wrote: >> I'm not very comfortable with vb2 returning unexpected errors there. >> Also, >> for me it is clear that, if read will fail, POLLERR should be rised. >> >> Mauro. >> -- >> To unsubscribe from this list: send the line "un

RE: [RFC] Media Controller Capture driver for DM365

2011-06-28 Thread Hadli, Manjunath
Laurent, Sorry for getting back late on this. I was also tied up. I am following it up with the patches. -Manju On Mon, May 02, 2011 at 03:36:23, Laurent Pinchart wrote: > Hi Manjunath, > > On Wednesday 27 April 2011 16:14:03 Hadli, Manjunath wrote: > > Introduction > > > > This

Re: [RFCv3 PATCH 12/18] vb2_poll: don't start DMA, leave that to the first read().

2011-06-28 Thread Andy Walls
Mauro Carvalho Chehab wrote: >Em 28-06-2011 04:33, Hans Verkuil escreveu: >> On Monday, June 27, 2011 23:52:37 Mauro Carvalho Chehab wrote: >>> Em 07-06-2011 12:05, Hans Verkuil escreveu: From: Hans Verkuil The vb2_poll function would start read-DMA if called without any >streamin

[RFCv2 PATCH 13/13] v4l2-event.h: add overview documentation to the header.

2011-06-28 Thread Hans Verkuil
From: Hans Verkuil It's getting confusing who is linking to what, so add an overview at the start of the header. Signed-off-by: Hans Verkuil --- include/media/v4l2-event.h | 34 ++ 1 files changed, 34 insertions(+), 0 deletions(-) diff --git a/include/media/v

[RFCv2 PATCH 12/13] v4l2-ctrls/v4l2-events: small coding style cleanups

2011-06-28 Thread Hans Verkuil
From: Hans Verkuil Thanks to Laurent Pinchart . Signed-off-by: Hans Verkuil --- drivers/media/video/v4l2-ctrls.c |2 +- drivers/media/video/v4l2-event.c |6 ++ include/media/v4l2-ctrls.h |1 - include/media/v4l2-event.h | 34 +++--- 4 f

[RFCv2 PATCH 11/13] DocBook: update V4L Event Interface section.

2011-06-28 Thread Hans Verkuil
From: Hans Verkuil Starting with v3.1 the V4L2 API provides certain guarantees with respect to events. Document these. Signed-off-by: Hans Verkuil --- Documentation/DocBook/media/v4l/dev-event.xml | 30 --- .../DocBook/media/v4l/vidioc-subscribe-event.xml | 15 ++

[RFCv2 PATCH 10/13] v4l2-framework.txt: update v4l2_event section.

2011-06-28 Thread Hans Verkuil
From: Hans Verkuil Signed-off-by: Hans Verkuil --- Documentation/video4linux/v4l2-framework.txt | 51 ++ 1 files changed, 35 insertions(+), 16 deletions(-) diff --git a/Documentation/video4linux/v4l2-framework.txt b/Documentation/video4linux/v4l2-framework.txt index

[RFCv2 PATCH 09/13] v4l2-framework.txt: updated v4l2_fh_init documentation.

2011-06-28 Thread Hans Verkuil
From: Hans Verkuil v4l2_fh_init now returns void instead of int, updated the doc. Signed-off-by: Hans Verkuil --- Documentation/video4linux/v4l2-framework.txt |8 ++-- 1 files changed, 2 insertions(+), 6 deletions(-) diff --git a/Documentation/video4linux/v4l2-framework.txt b/Documen

[RFCv2 PATCH 08/13] v4l2-ctrls: v4l2_ctrl_handler_setup code simplification

2011-06-28 Thread Hans Verkuil
From: Hans Verkuil Signed-off-by: Hans Verkuil --- drivers/media/video/v4l2-ctrls.c | 13 - 1 files changed, 4 insertions(+), 9 deletions(-) diff --git a/drivers/media/video/v4l2-ctrls.c b/drivers/media/video/v4l2-ctrls.c index 040d5c9..627a1e4 100644 --- a/drivers/media/video/v4

[RFCv2 PATCH 07/13] v4l2-ctrls: split try_or_set_ext_ctrls()

2011-06-28 Thread Hans Verkuil
From: Hans Verkuil Split try_or_set_ext_ctrls() into a validate_ctrls() part ('Phase 1') and merge the second part ('Phase 2') into try_set_ext_ctrls(). This makes a lot more sense and it also does the validation before trying to try/set the controls. Signed-off-by: Hans Verkuil --- drivers/m

[RFCv2 PATCH 05/13] v4l2-ctrls: don't initially set CH_VALUE for write-only controls

2011-06-28 Thread Hans Verkuil
From: Hans Verkuil When sending the SEND_INITIAL event for write-only controls the V4L2_EVENT_CTRL_CH_VALUE flag should not be set. It's meaningless. Signed-off-by: Hans Verkuil --- drivers/media/video/v4l2-ctrls.c |6 -- 1 files changed, 4 insertions(+), 2 deletions(-) diff --git a/d

[RFCv2 PATCH 06/13] v4l2-ctrls: improve discovery of controls of the same cluster

2011-06-28 Thread Hans Verkuil
From: Hans Verkuil The implementation of VIDIOC_G/S/TRY_EXT_CTRLS in the control framework has to figure out which controls in the control list belong to the same cluster. Since controls belonging to the same cluster need to be handled as a unit, this is important information. It did that by goi

[RFCv2 PATCH 04/13] v4l2-event: add optional merge and replace callbacks

2011-06-28 Thread Hans Verkuil
From: Hans Verkuil When the event queue for a subscribed event is full, then the oldest event is dropped. It would be nice if the contents of that oldest event could be merged with the next-oldest. That way no information is lost, only intermediate steps are lost. This patch adds optional replac

[RFCv2 PATCH 03/13] v4l2-event/ctrls/fh: allocate events per fh and per type instead of just per-fh

2011-06-28 Thread Hans Verkuil
From: Hans Verkuil The driver had to decide how many events to allocate when the v4l2_fh struct was created. It was possible to add more events afterwards, but there was no way to ensure that you wouldn't miss important events if the event queue would fill up for that filehandle. In addition, on

[RFCv2 PATCH 02/13] v4l2-ctrls/event: remove struct v4l2_ctrl_fh, instead use v4l2_subscribed_event

2011-06-28 Thread Hans Verkuil
From: Hans Verkuil The v4l2_ctrl_fh struct connected v4l2_ctrl with v4l2_fh so the control would know which filehandles subscribed to it. However, it is much easier to use struct v4l2_subscribed_event directly for that and get rid of that intermediate struct. Signed-off-by: Hans Verkuil --- dr

[RFCv2 PATCH 01/13] v4l2-events/fh: merge v4l2_events into v4l2_fh

2011-06-28 Thread Hans Verkuil
From: Hans Verkuil Drivers that supported events used to be rare, but now that controls can also raise events this will become much more common since almost all drivers have controls. This means that keeping struct v4l2_events as a separate struct make no more sense. Merging it into struct v4l2_

[RFCv2 PATCH 0/13] Allocate events per-event-type, v4l2-ctrls cleanup

2011-06-28 Thread Hans Verkuil
This is the second version of this patch series. The only real change is in patch 4/13: the original merge callback has been replaced with separate replace and merge callbacks. The original version was hard to understand and had const issues as well. The other changes are all documentation update

Re: [RFCv3 PATCH 12/18] vb2_poll: don't start DMA, leave that to the first read().

2011-06-28 Thread Mauro Carvalho Chehab
Em 28-06-2011 04:33, Hans Verkuil escreveu: > On Monday, June 27, 2011 23:52:37 Mauro Carvalho Chehab wrote: >> Em 07-06-2011 12:05, Hans Verkuil escreveu: >>> From: Hans Verkuil >>> >>> The vb2_poll function would start read-DMA if called without any streaming >>> in progress. This unfortunately

Re: HVR-1250/CX23885 IR Rx

2011-06-28 Thread Andy Walls
Jarod Wilson wrote: >On Apr 10, 2011, at 7:08 PM, Andy Walls wrote: > >> On Sat, 2011-04-09 at 21:39 -0400, Jarod Wilson wrote: >> Jarod, The HVR-1850 uses a raw IR receiver in the CX23888 and older >>> HVR-1250s use the raw IR receiver in the CX23885. They both work >for >>> Rx

Re: [RFCv3 PATCH 11/18] v4l2-ctrls: add v4l2_fh pointer to the set control functions.

2011-06-28 Thread Mauro Carvalho Chehab
Em 28-06-2011 03:22, Hans Verkuil escreveu: > On Monday, June 27, 2011 23:20:07 Mauro Carvalho Chehab wrote: >> Em 07-06-2011 12:05, Hans Verkuil escreveu: >>> From: Hans Verkuil >>> >>> When an application changes a control you want to generate an event. >>> However, you want to avoid sending suc

Re: [RFCv3 PATCH 08/18] v4l2-ctrls: add v4l2_ctrl_auto_cluster to simplify autogain/gain scenarios

2011-06-28 Thread Mauro Carvalho Chehab
Em 28-06-2011 03:08, Hans Verkuil escreveu: > On Monday, June 27, 2011 22:57:34 Mauro Carvalho Chehab wrote: >> Em 07-06-2011 12:05, Hans Verkuil escreveu: >>> From: Hans Verkuil >>> >>> It is a bit tricky to handle autogain/gain type scenerios correctly. Such >>> controls need to be clustered and

Re: [PATCH] media: initial driver for ov5642 CMOS sensor

2011-06-28 Thread Bastian Hecht
2011/6/27 Laurent Pinchart : > Hi Bastian, > > Thanks for the patch. > > On Friday 24 June 2011 12:57:36 Bastian Hecht wrote: >> This is an initial driver release for the Omnivision 5642 CMOS sensor. >> >> Signed-off-by: Bastian Hecht >> --- >> >> diff --git a/drivers/media/video/ov5642.c b/driver

Re: [RFCv3 PATCH 12/18] vb2_poll: don't start DMA, leave that to the first read().

2011-06-28 Thread Hans Verkuil
On Tuesday, June 28, 2011 09:33:57 Hans Verkuil wrote: > On Monday, June 27, 2011 23:52:37 Mauro Carvalho Chehab wrote: > > Em 07-06-2011 12:05, Hans Verkuil escreveu: > > > From: Hans Verkuil > > > > > > The vb2_poll function would start read-DMA if called without any streaming > > > in progress

RE: FW: OMAP 3 ISP

2011-06-28 Thread Alex Gershgorin
Hi Laurent, I recently got the Zoom OMAP35xx Torpedo and began testing OMAP3ISP. Currently I have a problem I can't solve. See Message from booting Kernel: Linux media interface: v0.10 Linux video capture interface: v2.00 omap3isp omap3isp: Revision 2.0 found omap-iommu omap-iommu.0: isp: version

Re: [RFCv3 PATCH 12/18] vb2_poll: don't start DMA, leave that to the first read().

2011-06-28 Thread Hans Verkuil
On Monday, June 27, 2011 23:52:37 Mauro Carvalho Chehab wrote: > Em 07-06-2011 12:05, Hans Verkuil escreveu: > > From: Hans Verkuil > > > > The vb2_poll function would start read-DMA if called without any streaming > > in progress. This unfortunately does not work if the application just wants >