[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v2 1/2] usb-hid: Add high speed mouse configurat
From: |
Jan Vesely |
Subject: |
Re: [Qemu-devel] [PATCH v2 1/2] usb-hid: Add high speed mouse configuration |
Date: |
Thu, 11 Sep 2014 00:57:28 -0400 |
Hi Gerd,
sorry for taking so long to get back to this. My schedule has been super
busy past couple of months.
I have rebased the patches and moved the usb version desc selection to
hid_initfn as you suggested. I also made the changes follow the tablet
example as closely as possible
I tested the patches on ubuntu 64 and fedora 64 liveCDs and both mouse
and keyboard still work ok.
thanks,
jan
On Wed, 2014-04-16 at 08:27 +0200, Gerd Hoffmann wrote:
> On Di, 2014-04-15 at 19:33 -0400, Ján Veselý wrote:
> > Hi,
> >
> > I found some more time to work on this. However, I'm not sure I
> > understand the purpose of compat property:
> > I noticed that the only place it's set is in PC_COMPAT_1_3 (is this
> > what you meant by older devices?),
>
> Older machine types, yes. This switches the tablet to usb1 by default
> if you start qemu with -M pc-1.3 (or older), because it was usb1 only in
> qemu 1.3 & older.
>
> > I guess same setting for FS mouse and kbd should be added there as well.
>
> They need to go to the (probably not added yet) PC_COMPAT_2_0.
>
> > The switch statement in usb_tablet_initfn selects between two
> > descriptor structures, however, these structures are almost identical
> > with the sole exception of .high field being set in the HS version
> > (desc_tablet2).
> >
> > So changing the switch statement to:
> > if (us->usb_version < 2)
> > dev->usb_desc->high = NULL;
> >
> > should be OK for all usb hid devices, right?
>
> No. That struct is shared between all device instances, so if you muck
> with it all devices are affected, i.e. this ...
>
> qemu -device usb-tablet,usb_ver=1 -device usb_tablet,usb_ver=2
>
> will not work correctly.
>
> cheers,
> Gerd
>
>
signature.asc
Description: This is a digitally signed message part
- Re: [Qemu-devel] [PATCH v2 1/2] usb-hid: Add high speed mouse configuration,
Jan Vesely <=