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
