Re: [PATCH] Add support for OmniVision OV534 based USB cameras.

[Jean-Francois Moine <moinejf@free.fr>]

On Sat, 2008-08-16 at 08:47 -0300, Mauro Carvalho Chehab wrote:
> On Sat, 16 Aug 2008 10:03:13 +0200
> > > I would definitely be willing to merge the code into an existing driver,
> > > though I was under the impression that the gspca core was for ISOC based
> > > USB devices.  The ov534's imagine end-point is bulk transfer, with the
> > > audio endpoints being isoc.
> > > 
> > 
> > Ah yes it is I didn't know non isoc cams existed, so thats why your driver is 
> > so small I already was sorta missing the isoc setup stufff :)

Hello Mauro and Hans,

The code for bulk transfer existed in the first versions, but, as no
webcam used it, I removed. It should be easy to put it back.

> The usage of videobuf simplifies a lot the driver logic. That's why it is so small :)
> 
> I think it would be good if gspca could also use videobuf in the future.

I think that videobuf is rather complex for handling usually less than 8
buffers (max = 16 in gspca) and it asks for a mutex at irq level. My
buffer handling also offers a very small memory overhead...

Cheers.

-- 
Ken ar c'hentañ |             ** Breizh ha Linux atav! **
Jef             |               http://moinejf.free.fr/


--
video4linux-list mailing list
Unsubscribe mailto:video4linux-list-request@redhat.com?subject=unsubscribe
https://www.redhat.com/mailman/listinfo/video4linux-list