of course that's a good thing to do, but it won't fix it for users with an 
older pwc, and we cannot assume all users will upgrade. some might even not 
know how to do it, or not have root access to the system.

Op maandag 12 juni 2006 18:49, schreef Philippe Valembois - Phil:
> Why not contacting Luc Saillard to inform him of the bug ?
> Phil
>
> Le Monday 12 June 2006 18:40, Harry Vennik a écrit :
> > I think it simply isn't an amsn bug, but a driver bug.
> > The streaming support on pwc seems to be somewhat buggy.
> > That is why clearing the V4L2_CAP_STREAMING capability flag will make it
> > work. On v4l the streaming support is not available at all, so it will
> > always work.
> >
> > And if indeed it is a driver bug, we can only provide a work-around, not
> > a real fix.
> >
> > Harry
> >
> > Op maandag 12 juni 2006 18:26, schreef Philippe Valembois - Phil:
> > > Hi,
> > > for me the better solution is to find the bug and eliminate it....
> > > Phil
> > > P.S. I don't have marks anymore only curses, so I consider I am almost
> > > in holidays !
> > >
> > > Le Monday 12 June 2006 18:17, Harry Vennik a écrit :
> > > > 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
> > >
> > > _______________________________________________
> > > 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