But that's not much of an argument here, because any solution will require 
patching libng or even its v4l2 driver. 

Anyway, by doing it Youness' way we will avoid an ugly thing like if(driver == 
pwc) { ..... } by adding a real feature. The only disadvantage being all 
reports about 'webcam not released when using v4l2' that we certainly will 
get.

Harry

Op zondag 11 juni 2006 23:44, schreef Philippe Valembois - Phil:
> No without patching libng...
> Phil
>
> Le Sunday 11 June 2006 22:45, Youness Alaoui a écrit :
> > can't we make it use the driver you choose ? for example, the user can
> > choose v4l or v4l2 from the list of devices, so if he chooses v4l, the
> > v4l driver will be used, if he chooses v4l2, then the v4l2 driver will be
> > used... and have a label (we already have one, right ?) for pwc users
> > that would say "use the v4l device, not the v4l2 one"...
> > is it possible ?
> >
> > KKRT
> >
> > On Sun, 11 Jun 2006 16:37:37 -0400, Harry Vennik <[EMAIL PROTECTED]>
> >
> > wrote:
> > > Problem found.
> > >
> > > There are 2 work-arounds:
> > > - If we somehow unset the V4L2_CAP_STREAMING flag for pwc, it will
> > > work. - If we force pwc to v4l instead of v4l2 it will also work.
> > >
> > > The first one seems quite ugly to me, so IMO forcing to v4l is the way
> > > to go.
> > >
> > > Op zondag 11 juni 2006 17:26, schreef Vivia Nikolaidou:
> > >> On Sun, 11 Jun 2006, Max wrote:
> > >> > I see at least one thing missing (imho) in the TODO-list and it is a
> > >>
> > >> bug
> > >>
> > >> > fix for the pwc webcam's drivers. It was working perfectly before
> > >> > but
> > >>
> > >> it
> > >>
> > >> > stopped working after 6511 (i checked). (i'am talking about the "not
> > >> > released webcam").
> > >>
> > >> Thanx!! :)
> > >>
> > >> (BTW, bugfixes can be made between 2 RC releases)
> > >>
> > >> Here's the diff - hope you (or someone else) can find a fix for that:
> > >>
> > >> --- utils/linux/capture/capture.c       (revision 6510)
> > >> +++ utils/linux/capture/capture.c       (revision 6511)
> > >> @@ -480,6 +480,8 @@
> > >>      captureItem->rgb_buffer = ng_malloc_video_buf(&captureItem->dev,
> > >> &captureItem->fmt);
> > >>    }
> > >>
> > >> +  captureItem->dev.v->startvideo(captureItem->dev.handle, 25, 1);
> > >> +
> > >>    Tcl_SetObjResult(interp,
> > >> Tcl_NewStringObj(captureItem->captureName,-1));
> > >>
> > >>    return TCL_OK;
> > >> @@ -506,6 +508,8 @@
> > >>      return TCL_ERROR;
> > >>    }
> > >>
> > >> +  capItem->dev.v->stopvideo(capItem->handle);
> > >> +
> > >>    // If a converter was used, close it and release the rgb_buffer
> > >>    if (capItem->handle) {
> > >>      ng_process_fini(capItem->handle);
> > >> @@ -607,7 +611,7 @@
> > >>    // - High resolution
> > >>    // - Low resolution
> > >>    for (dim_idx = resolution;;) {
> > >> -    if ((capItem->image_data =
> > >> capItem->dev.v->getimage(capItem->dev.handle)) == NULL) {
> > >> +    if ((capItem->image_data =
> > >> capItem->dev.v->nextframe(capItem->dev.handle)) == NULL) {
> > >>  #   ifdef DEBUG
> > >>        fprintf(stderr,"Capturing image failed at %d, %d\n", fmt.width,
> > >> fmt.height);
> > >>  #   endif
> > >>
> > >>
> > >> _______________________________________________
> > >> Amsn-devel mailing list
> > >> Amsn-devel@lists.sourceforge.net
> > >> https://lists.sourceforge.net/lists/listinfo/amsn-devel
> > >
> > > _______________________________________________
> > > Amsn-devel mailing list
> > > Amsn-devel@lists.sourceforge.net
> > > https://lists.sourceforge.net/lists/listinfo/amsn-devel
>
> _______________________________________________
> Amsn-devel mailing list
> Amsn-devel@lists.sourceforge.net
> https://lists.sourceforge.net/lists/listinfo/amsn-devel


_______________________________________________
Amsn-devel mailing list
Amsn-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/amsn-devel

Reply via email to