| Message ID | 1311274344-8720-1-git-send-email-michael@walle.cc |
|---|---|
| State | New |
| Headers | show |
Hi Anthony, could you please apply this patch to the 0.15 (and master) branch. Am Donnerstag 21 Juli 2011, 20:52:24 schrieb Michael Walle: > The QEMU keyboard and mouse reports themselves as full speed devices, > though they are actually low speed devices. Until this is fixed, claim that > we are supporting full speed devices. > > Signed-off-by: Michael Walle <michael@walle.cc> > --- > hw/milkymist-softusb.c | 6 ++++-- > 1 files changed, 4 insertions(+), 2 deletions(-) > > diff --git a/hw/milkymist-softusb.c b/hw/milkymist-softusb.c > index ce2bfc6..abf7b59 100644 > --- a/hw/milkymist-softusb.c > +++ b/hw/milkymist-softusb.c > @@ -310,10 +310,12 @@ static int milkymist_softusb_init(SysBusDevice *dev) > usb_bus_new(&s->usbbus, &softusb_bus_ops, NULL); > > /* our two ports */ > + /* FIXME: claim to support full speed devices. qemu mouse and keyboard > + * report themselves as full speed devices. */ > usb_register_port(&s->usbbus, &s->usbport[0], NULL, 0, &softusb_ops, > - USB_SPEED_MASK_LOW); > + USB_SPEED_MASK_LOW | USB_SPEED_MASK_FULL); > usb_register_port(&s->usbbus, &s->usbport[1], NULL, 1, &softusb_ops, > - USB_SPEED_MASK_LOW); > + USB_SPEED_MASK_LOW | USB_SPEED_MASK_FULL); > > /* and finally create an usb keyboard */ > s->usbdev = usb_create_simple(&s->usbbus, "usb-kbd");
Am Donnerstag 21 Juli 2011, 20:52:24 schrieb Michael Walle: > The QEMU keyboard and mouse reports themselves as full speed devices, > though they are actually low speed devices. Until this is fixed, claim that > we are supporting full speed devices. > > Signed-off-by: Michael Walle <michael@walle.cc> > --- > hw/milkymist-softusb.c | 6 ++++-- > 1 files changed, 4 insertions(+), 2 deletions(-) > > diff --git a/hw/milkymist-softusb.c b/hw/milkymist-softusb.c > index ce2bfc6..abf7b59 100644 > --- a/hw/milkymist-softusb.c > +++ b/hw/milkymist-softusb.c > @@ -310,10 +310,12 @@ static int milkymist_softusb_init(SysBusDevice *dev) > usb_bus_new(&s->usbbus, &softusb_bus_ops, NULL); > > /* our two ports */ > + /* FIXME: claim to support full speed devices. qemu mouse and keyboard > + * report themselves as full speed devices. */ > usb_register_port(&s->usbbus, &s->usbport[0], NULL, 0, &softusb_ops, > - USB_SPEED_MASK_LOW); > + USB_SPEED_MASK_LOW | USB_SPEED_MASK_FULL); > usb_register_port(&s->usbbus, &s->usbport[1], NULL, 1, &softusb_ops, > - USB_SPEED_MASK_LOW); > + USB_SPEED_MASK_LOW | USB_SPEED_MASK_FULL); > > /* and finally create an usb keyboard */ > s->usbdev = usb_create_simple(&s->usbbus, "usb-kbd"); Ping. Without this patch the whole milkymist target is non-functional in the 0.15 release.
On 07/21/11 20:52, Michael Walle wrote: > The QEMU keyboard and mouse reports themselves as full speed devices, > though they are actually low speed devices. Until this is fixed, claim that > we are supporting full speed devices. > > Signed-off-by: Michael Walle<michael@walle.cc> For both master+stable: Acked-by: Gerd Hoffmann <kraxel@redhat.com> cheers, Gerd
On Wed, Aug 03, 2011 at 03:17:07PM +0200, Gerd Hoffmann wrote: > On 07/21/11 20:52, Michael Walle wrote: > >The QEMU keyboard and mouse reports themselves as full speed devices, > >though they are actually low speed devices. Until this is fixed, claim that > >we are supporting full speed devices. > > > >Signed-off-by: Michael Walle<michael@walle.cc> > > For both master+stable: > > Acked-by: Gerd Hoffmann <kraxel@redhat.com> Thanks, I've applied it to both master and stable-0.15. Cheers
diff --git a/hw/milkymist-softusb.c b/hw/milkymist-softusb.c index ce2bfc6..abf7b59 100644 --- a/hw/milkymist-softusb.c +++ b/hw/milkymist-softusb.c @@ -310,10 +310,12 @@ static int milkymist_softusb_init(SysBusDevice *dev) usb_bus_new(&s->usbbus, &softusb_bus_ops, NULL); /* our two ports */ + /* FIXME: claim to support full speed devices. qemu mouse and keyboard + * report themselves as full speed devices. */ usb_register_port(&s->usbbus, &s->usbport[0], NULL, 0, &softusb_ops, - USB_SPEED_MASK_LOW); + USB_SPEED_MASK_LOW | USB_SPEED_MASK_FULL); usb_register_port(&s->usbbus, &s->usbport[1], NULL, 1, &softusb_ops, - USB_SPEED_MASK_LOW); + USB_SPEED_MASK_LOW | USB_SPEED_MASK_FULL); /* and finally create an usb keyboard */ s->usbdev = usb_create_simple(&s->usbbus, "usb-kbd");
The QEMU keyboard and mouse reports themselves as full speed devices, though they are actually low speed devices. Until this is fixed, claim that we are supporting full speed devices. Signed-off-by: Michael Walle <michael@walle.cc> --- hw/milkymist-softusb.c | 6 ++++-- 1 files changed, 4 insertions(+), 2 deletions(-)