Well, Youness had been maybe a little harsh ;) What is clear is : 1) bug was entirely in aMSN and the only error uvcvideo had was corrected when I reported it to Laurent Pinchart. And, I am sure too that uvcvideo respect V4L2 specs perfectly 2) The bug introduced in aMSN was introduced because of an attempt to make PWC work EVEN if it doesn't comply with V4L2 standard... Luc Saillard provided us a patch but I didn't find it good so I decided to make my patch. Unfortunately, it was a done a Sunday afternoon before a homework I had to do and... I was a little in a hurry. So that's why I accuse PWC : if it would have complied with V4L2 spec, nothing of that would have appeared. 3) Now, the cause of the late fix. Logitech released 2 generations of 5000Pro webcams : one which support YUV for every resolutions and one which only support MJEPG for little resolutions... I forgot a little these details and was sure I used MJPEG compression. Yesterday, I wanted to test a little to see how you could help us with gdb and discovered that I used YUV. I immediately understood why it worked for me and not for you. So, I did some hacks to force MJPEG mode and discovered the bug. Some thinking about the ugly workaround I put yesterday and that's all : committed. 4) I really hope that next PWC release (candidate) will fully comply with V4L2 and that I will be able to remove this workaround... Don't stress too much Luc Saillard as its last version published was released before our discussion on tracker...
Phil David Girault a écrit : > Hello Youness, > > I fully agree with you. > It mean nothing to just say "it is not a bug in driver because it work > with X or Y". > One of the other thing that mean nothing: "it works here, problems come > from your drivers and/or hardware". > Each time i heard this at work, I want to "eat" the person telling > that!!! ;-) > > But, anyway, when I say 3 weeks ago that it STOP to work with r8464, I > think it was a good indication that the problem is in aMsn. > Perhaps I don't give enough information!! :-( > > By reading also the uvcvideo dev mailing list, I'm pretty sure it > conform to V4L2 api strictly. > And ekiga is one of the existing V4L2 application that work great. It is > used by a lot of people. > > So, just want to say to finish: Thanks for your great works, all aMsn devs. > > Regards, > David > > Youness Alaoui a écrit : >> I just can't stand it anymore when someone says "it's not a bug with the >> driver because it works with X or Y"... >> it means NOTHING if it works with ekiga or uvcviewwhatever... uvcviewer >> works for uvc webcams and it does some >> things to make it work with uvc webcams... but it might be that the bug is >> still coming from the driver, it's >> just that uvcviewer has a workaround, or that it uses something different... >> for example, pwc driver, our code worked perfect with it when we did a >> "open, capture an image, close", but when >> we did a "open, start streaming, get image, get image, get image, ...., >> close" we got the bug... >> aMSN tried to implement the specifications for v4l as much as possible so >> that it works with ALL webcams, so >> your uvcviewer/ekiga may not work with pwc drivers, but works with uvc >> drivers, or gnomemeeting works with pwc >> drivers and uvc, but it's not optimized at all (because it never streams the >> webcam), etc... >> >> in short.. if it works with X or Y, it doesn't mean that the problem is from >> amsn!!!! >> >> p.s.: I'm not saying the bug (or workaround) that Phil just committed was >> because of a bug in the driver or in >> amsn, I have no idea, I'm just saying that I'm fed up and that it means >> nothing if it works with some other >> program... >> >> KKRT >> >> On Tue, May 08, 2007 at 12:12:59AM +0200, giopas wrote: >> >>> Philippe Valembois - Phil ha scritto: >>> >>>> This bug is fixed in r8571... >>>> Please test... >>>> David you can test too as it should fix yours too... >>>> These IO errors are so silly : they lead me to think the fault was with >>>> the driver.... >>>> Anyway, I repeat it again and again, PWC is a pure shit as it made me >>>> introduce this nasty bug :@ >>>> Without PWC all worked perfectly before my attemps to fix for PWC... >>>> If any PWC user read here please remind to Luc Saillard that he still >>>> has to fix his driver. pwc-10.0.12-rc1.tar.bz2 still contains the bug... >>>> Phil >>>> >>>> Philippe Valembois - Phil a écrit : >>>> >>>> >>>>> Arfffffffffff !!! >>>>> Just played a little with the extension ! >>>>> I was sure I used MJPEG but I used YUV... >>>>> Well I am happy to say you that you use an old generation webcam : you >>>>> surely bought it in a lost place because I bought the same webcam at the >>>>> same date at Paris and I got a new gen 5000Pro (niark :d)... >>>>> I did a little mod to capture extension and now I get the bug... >>>>> So now I can test it ! >>>>> Expect a fix soon... >>>>> Phil >>>>> >>>>> >>>>> >>> Cool guy, >>> >>> now it works again. Thanks a bounch!! >>> It was too strange to be a driver/kernel/hardware problem, since it >>> perfectly worked with ekiga or luvcview... >>> >>> giopas >>> >>> ps: my gf gave me this camera last Christmas, but I don't know where she >>> bought it. Merci encore! >>> >>> ------------------------------------------------------------------------- >>> This SF.net email is sponsored by DB2 Express >>> Download DB2 Express C - the FREE version of DB2 express and take >>> control of your XML. No limits. Just data. Click to get it now. >>> http://sourceforge.net/powerbar/db2/ >>> _______________________________________________ >>> Amsn-devel mailing list >>> Amsn-devel@lists.sourceforge.net >>> https://lists.sourceforge.net/lists/listinfo/amsn-devel >>> >> ------------------------------------------------------------------------- >> This SF.net email is sponsored by DB2 Express >> Download DB2 Express C - the FREE version of DB2 express and take >> control of your XML. No limits. Just data. Click to get it now. >> http://sourceforge.net/powerbar/db2/ >> _______________________________________________ >> Amsn-devel mailing list >> Amsn-devel@lists.sourceforge.net >> https://lists.sourceforge.net/lists/listinfo/amsn-devel >> > > > ------------------------------------------------------------------------- > This SF.net email is sponsored by DB2 Express > Download DB2 Express C - the FREE version of DB2 express and take > control of your XML. No limits. Just data. Click to get it now. > http://sourceforge.net/powerbar/db2/ > _______________________________________________ > Amsn-devel mailing list > Amsn-devel@lists.sourceforge.net > https://lists.sourceforge.net/lists/listinfo/amsn-devel > ------------------------------------------------------------------------- This SF.net email is sponsored by DB2 Express Download DB2 Express C - the FREE version of DB2 express and take control of your XML. No limits. Just data. Click to get it now. http://sourceforge.net/powerbar/db2/ _______________________________________________ Amsn-devel mailing list Amsn-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/amsn-devel
