From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mailman by lists.gnu.org with tmda-scanned (Exim 4.43) id 1FZ8gJ-0007Hv-Dl for qemu-devel@nongnu.org; Thu, 27 Apr 2006 11:45:27 -0400 Received: from exim by lists.gnu.org with spam-scanned (Exim 4.43) id 1FZ8gI-0007HV-Kw for qemu-devel@nongnu.org; Thu, 27 Apr 2006 11:45:26 -0400 Received: from [199.232.76.173] (helo=monty-python.gnu.org) by lists.gnu.org with esmtp (Exim 4.43) id 1FZ8gI-0007HP-HF for qemu-devel@nongnu.org; Thu, 27 Apr 2006 11:45:26 -0400 Received: from [213.165.64.20] (helo=mail.gmx.net) by monty-python.gnu.org with smtp (Exim 4.52) id 1FZ8jE-0001Qy-JK for qemu-devel@nongnu.org; Thu, 27 Apr 2006 11:48:28 -0400 Message-ID: <4450E71B.8000003@gmx.de> Date: Thu, 27 Apr 2006 17:45:31 +0200 From: nix.wie.weg@gmx.de MIME-Version: 1.0 Subject: Re: [Qemu-devel] Large USB-Patch Documentation and todays CVS patch References: <444F9D90.9040306@gmx.de> <13040.1146091241@www072.gmx.net> <445061CA.7070605@austin.rr.com> In-Reply-To: <445061CA.7070605@austin.rr.com> Content-Type: text/plain; charset=ISO-8859-1 Content-Transfer-Encoding: 7bit Reply-To: qemu-devel@nongnu.org List-Id: qemu-devel.nongnu.org List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: qemu-devel@nongnu.org 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 support the improved error handling > (have it clear endpoint halt/etc) and other improvements. Later, the > new redirectors can be merged in and modified as necessary. Nobody said anything about a short to come release. I have no problem to suspend the patch until after this release. But then usb should stay as untouched as possible. > > The purpose of modifying the user interface to the usb layer also > confuses me. What was the reasoning behind changing host:busaddr.addr > to host:busaddr:addr and host:VID:PID to host:VIDxPID? This is > something that should be abstracted in the layer and not handed down > to the user. Why display the bcdUSB revision and not the connected > speed to the user (as is already done)? The reason was a simple thougth of mine: a linux user could probably better understand bus:device (it is something which he gets displayed sometimes from libusb: libusb:001:002) but I m really open to change it to whatever string you want. A other possibility is that we introduce something like hostid:VID:PID. I can only tell you that I do not completely like the old notation. (But as I said before, no hard feelings about that, it is changed in less than a minute) The reason for the bcdUSB revision is even simpler. A person who has only used USB has no idea what full speed or lowspeed does mean. If you look through the code you will notice that the speed setting was determined by bcdUSB and so I thougth it may be better to display it. As most people have a idea what USB 1.0, 1.1 or 2.0 means. With kind regards, Tino H. Seifert