From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mailman by lists.gnu.org with tmda-scanned (Exim 4.43) id 1F9xUz-0005f6-20 for qemu-devel@nongnu.org; Thu, 16 Feb 2006 23:45:41 -0500 Received: from exim by lists.gnu.org with spam-scanned (Exim 4.43) id 1F9xUc-0005BM-N1 for qemu-devel@nongnu.org; Thu, 16 Feb 2006 23:45:19 -0500 Received: from [199.232.76.173] (helo=monty-python.gnu.org) by lists.gnu.org with esmtp (Exim 4.43) id 1F9xRJ-0000rU-O9 for qemu-devel@nongnu.org; Thu, 16 Feb 2006 23:41:54 -0500 Received: from [24.93.47.43] (helo=ms-smtp-04.texas.rr.com) by monty-python.gnu.org with esmtp (Exim 4.52) id 1F9xWa-0001LV-Qx for qemu-devel@nongnu.org; Thu, 16 Feb 2006 23:47:21 -0500 Received: from [192.168.0.11] (cpe-67-9-160-120.austin.res.rr.com [67.9.160.120]) by ms-smtp-04.texas.rr.com (8.12.10/8.12.7) with ESMTP id k1H4fo9C028138 for ; Thu, 16 Feb 2006 22:41:50 -0600 (CST) Message-ID: <43F553F5.9040406@austin.rr.com> Date: Thu, 16 Feb 2006 22:41:25 -0600 From: Lonnie Mendez MIME-Version: 1.0 Subject: Re: [Qemu-devel] patch for USB issues References: In-Reply-To: Content-Type: text/plain; charset=ISO-8859-1; format=flowed 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 Krister Walfridsson wrote: > I'm playing with the USB code, but I cannot get the USB mouse emulation > to work with the guests I have tried (NetBSD and Win2k). Attached are > two patches that improves the situation somewhat. > > * hw/usb-uhci.c > The PCI configuration registers are set up with an incorrect > offset for USBBASE and uninitialized SBRN. This makes NetBSD > complain that this is not a valid UHCI controller. > * hw/usb.c > The length field of the returned descriptor strings are > incorrect, which trunkates the returned strings. > > There are some additional problems with the hub emulation that I hope > to get the time to fix during the weekend. > > /Krister > > > > Index: usb-uhci.c > =================================================================== > RCS file: /sources/qemu/qemu/hw/usb-uhci.c,v > retrieving revision 1.4 > diff -b -u -p -r1.4 usb-uhci.c > --- usb-uhci.c 19 Nov 2005 17:43:37 -0000 1.4 > +++ usb-uhci.c 16 Feb 2006 20:02:52 -0000 > @@ -654,6 +654,7 @@ void usb_uhci_init(PCIBus *bus, USBPort > pci_conf[0x0b] = 0x0c; > pci_conf[0x0e] = 0x00; // header_type > pci_conf[0x3d] = 4; // interrupt pin 3 > + pci_conf[0x60] = 0x11; // USB 1.1 > > for(i = 0; i < NB_PORTS; i++) { > port = &s->ports[i]; > @@ -666,6 +667,6 @@ void usb_uhci_init(PCIBus *bus, USBPort > > uhci_reset(s); > > - pci_register_io_region(&s->dev, 0, 0x20, + > pci_register_io_region(&s->dev, 4, 0x20, > PCI_ADDRESS_SPACE_IO, uhci_map); > } > Index: usb.c > =================================================================== > RCS file: /sources/qemu/qemu/hw/usb.c,v > retrieving revision 1.3 > diff -b -u -p -r1.3 usb.c > --- usb.c 6 Nov 2005 16:13:29 -0000 1.3 > +++ usb.c 16 Feb 2006 20:02:55 -0000 > @@ -183,7 +183,7 @@ int set_usb_string(uint8_t *buf, const c > > q = buf; > len = strlen(str); > - *q++ = 2 * len + 1; > + *q++ = 2 * len + 2; > *q++ = 3; > for(i = 0; i < len; i++) { > *q++ = str[i]; > > > _______________________________________________ > Qemu-devel mailing list > Qemu-devel@nongnu.org > http://lists.nongnu.org/mailman/listinfo/qemu-devel > Hello. That change to pci_register_io_region works over here with FreeBSD 6.0. However the revision for register 0x60 is incorrect. It should read: pci_conf[0x60] = 0x10; // USB Release 1.0 At least according to the uhci document I found that is the appropriate value. Changing it to 0x11 causes FreeBSD's uhci layer to complain of an invalid version number and kill the controller and it is indeed not listed in the document.