but we can still say: go update or don't bother us. Anyway it is important to let upstream know about this
On Mon, 2006-06-12 at 19:16 +0200, Harry Vennik wrote: > 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
signature.asc
Description: This is a digitally signed message part
_______________________________________________ Amsn-devel mailing list Amsn-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/amsn-devel