qemu-devel
[Top][All Lists]
Advanced

[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
> 
> 


Attachment: signature.asc
Description: This is a digitally signed message part


reply via email to

[Prev in Thread] Current Thread [Next in Thread]