Re: [Qemu-devel] Large USB-Patch Documentation and todays CVS patch

2007-05-23 Thread Per Åstrand
Hello! I just wanted to hear about the Large USB patch that was discussed quite alot here a couple of months ago. As I understood it the patch was put on hold until the 0.90 release, but I've heard nothing more about it... Is anybody (Tino, Lonnie?) working on getting the patch into CVS? Would

Re: [Qemu-devel] Large USB-Patch Documentation and todays CVS patch

2006-08-20 Thread Joseph Miller
-BEGIN PGP SIGNED MESSAGE- Hash: SHA1 With all due respect, Tino Seifert, I must comment on your USB patch. I do not normally post to this list, but I follow it very closely as the project is of great interest to me. I must admit that I have rarely even looked at qemu code, and that I

Re: [Qemu-devel] Large USB-Patch Documentation and todays CVS patch

2006-04-27 Thread nix . wie . weg
Hello Lonnie. Lonnie Mendez wrote: > Johannes Schindelin wrote: > >> > Seeing as there is a release coming up this is most definitely not a > good thing. Initial testing yielded lots of this. > I'd like to see my all-in-one patch stripped out. Then simply > modifying the linux redirector to

Re: [Qemu-devel] Large USB-Patch Documentation and todays CVS patch

2006-04-27 Thread nix . wie . weg
Hello Johannes, I have to admit that your post did piss me off a little bit. But on the other hand it would not help anybody, if I would shoot back, so I will try to be as polite as possible. My first note is on your first message, which I really have not received. It is probably my fault, but I

Re: [Qemu-devel] Large USB-Patch Documentation and todays CVS patch

2006-04-27 Thread Lonnie Mendez
Johannes Schindelin wrote: Also, you rename some things when it has no apparent use to rename them, such as usb_device_add, or product_name. Sorry but on that point you say things which are in no case provable by any study. Also kindly look into line 1417 and 1419 of your patch.

Re: [Qemu-devel] Large USB-Patch Documentation and todays CVS patch

2006-04-27 Thread Johannes Schindelin
Hi, nix.wie.weg wrote: > Johannes Schindelin wrote: > > I am quite sure you put a lot of work into this patch, but you sure make > > it hard to appreciate, too. > > > > First note that applying such a huge patch is bad. Let me help you (a > > little more than last time) > Sorry I dont know why,

Re: [Qemu-devel] Large USB-Patch Documentation and todays CVS patch

2006-04-27 Thread andrzej zaborowski
Hi, On 27/04/06, [EMAIL PROTECTED] <[EMAIL PROTECTED]> wrote: > Hello Johannes, > > Thanks for your comments. > > Johannes Schindelin wrote: > > I am quite sure you put a lot of work into this patch, but you sure make it > > hard to appreciate, too. > > > > First note that applying such a huge pat

Re: [Qemu-devel] Large USB-Patch Documentation and todays CVS patch

2006-04-26 Thread Lonnie Mendez
Johannes Schindelin wrote: I am quite sure you put a lot of work into this patch, but you sure make it hard to appreciate, too. First note that applying such a huge patch is bad. Let me help you (a little more than last time) to understand that: You are almost guaranteed to introduce bugs, and

Re: [Qemu-devel] Large USB-Patch Documentation and todays CVS patch

2006-04-26 Thread nix . wie . weg
Hello Johannes, Thanks for your comments. Johannes Schindelin wrote: > I am quite sure you put a lot of work into this patch, but you sure make it > hard to appreciate, too. > > First note that applying such a huge patch is bad. Let me help you (a little > more than last time) Sorry I dont know

Re: [Qemu-devel] Large USB-Patch Documentation and todays CVS patch

2006-04-26 Thread Johannes Schindelin
> As Fabrice pointed out to me yesterday, it takes time to understand the > new usb api. To make this process easier I have assembled a small > documentation. > You will find it here: > http://217.20.126.200/tino/usb_api.pdf > http://217.20.126.200/tino/usb_api.odg That is a nice description. > t

[Qemu-devel] Large USB-Patch Documentation and todays CVS patch

2006-04-26 Thread nix . wie . weg
Hello, As Fabrice pointed out to me yesterday, it takes time to understand the new usb api. To make this process easier I have assembled a small documentation. You will find it here: http://217.20.126.200/tino/usb_api.pdf http://217.20.126.200/tino/usb_api.odg the patch which applies cleanly agai