[Qemu-devel] [PATCH 0/3] input: add keys and mouse buttons to virtio input

Miika S posted 3 patches 7 years, 10 months ago
Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/qemu tags/patchew/20171222152531.1849-1-miika9764@gmail.com
Test checkpatch passed
Test docker passed
Test ppc passed
Test s390x passed
hw/input/virtio-input-hid.c | 15 +++++++++++++--
qapi/ui.json                |  5 ++++-
ui/keycodemapdb             |  2 +-
3 files changed, 18 insertions(+), 4 deletions(-)
[Qemu-devel] [PATCH 0/3] input: add keys and mouse buttons to virtio input
Posted by Miika S 7 years, 10 months ago
This patchset adds some missing keys and mouse buttons to
hw/input/virtio-input-hid.c.

On a JIS keyboard such as this one (106/109 - JIS)
https://upload.wikimedia.org/wikipedia/commons/b/b2/Physical_keyboard_layouts_comparison_ANSI_ISO_KS_ABNT_JIS.png
the keys are located as follows:
muhenkan: left of spacebar
henkan: right of spacebar
katakanahiragana: right of henkan
compose: the same as menu but the existing one didn't work
ro: left of right shift
    (this produces the same key code as the key right of left shift on
    102/105 ISO keyboard in the linked picture and consequently this
    patch also adds support for those keyboards)
yen: left of backspace

The mouse buttons "side" and "extra" are the ones you use to go to the
previous and next page in modern web browsers such as Firefox and
Chrome.

The last patch fixes mouse wheel event sent for both button up and down
by only forwarding the event when the button state is down.


Miika S (3):
  input: add missing JIS keys to virtio input
  input: add mouse side buttons to virtio input
  input: virtio: don't send mouse wheel event twice

 hw/input/virtio-input-hid.c | 15 +++++++++++++--
 qapi/ui.json                |  5 ++++-
 ui/keycodemapdb             |  2 +-
 3 files changed, 18 insertions(+), 4 deletions(-)

-- 
2.15.1


Re: [Qemu-devel] [PATCH 0/3] input: add keys and mouse buttons to virtio input
Posted by Gerd Hoffmann 7 years, 9 months ago
  Hi,

> Miika S (3):
>   input: add mouse side buttons to virtio input
>   input: virtio: don't send mouse wheel event twice

Cherry-picked these two ad the kbd update depends on the not-yet merged
keycodemapdb update for virtio-keyboard.

thanks,
  Gerd


Re: [Qemu-devel] [PATCH 0/3] input: add keys and mouse buttons to virtio input
Posted by Daniel P. Berrange 7 years, 9 months ago
On Mon, Jan 15, 2018 at 04:18:53PM +0100, Gerd Hoffmann wrote:
>   Hi,
> 
> > Miika S (3):
> >   input: add mouse side buttons to virtio input
> >   input: virtio: don't send mouse wheel event twice
> 
> Cherry-picked these two ad the kbd update depends on the not-yet merged
> keycodemapdb update for virtio-keyboard.

It is merged actually in, so you can include the keycode fix too:

https://git.qemu.org/gitweb.cgi?p=keycodemapdb.git;a=commit;h=05dad417e9d0b37ee1fba33056d91a6b734b3357

Regards,
Daniel
-- 
|: https://berrange.com      -o-    https://www.flickr.com/photos/dberrange :|
|: https://libvirt.org         -o-            https://fstop138.berrange.com :|
|: https://entangle-photo.org    -o-    https://www.instagram.com/dberrange :|

Re: [Qemu-devel] [PATCH 0/3] input: add keys and mouse buttons to virtio input
Posted by Gerd Hoffmann 7 years, 9 months ago
On Mon, Jan 15, 2018 at 03:32:33PM +0000, Daniel P. Berrange wrote:
> On Mon, Jan 15, 2018 at 04:18:53PM +0100, Gerd Hoffmann wrote:
> >   Hi,
> > 
> > > Miika S (3):
> > >   input: add mouse side buttons to virtio input
> > >   input: virtio: don't send mouse wheel event twice
> > 
> > Cherry-picked these two ad the kbd update depends on the not-yet merged
> > keycodemapdb update for virtio-keyboard.
> 
> It is merged actually in, so you can include the keycode fix too:
> 
> https://git.qemu.org/gitweb.cgi?p=keycodemapdb.git;a=commit;h=05dad417e9d0b37ee1fba33056d91a6b734b3357

To be exact: it conflicts (and I think also is obsoleted by) your patch
"hw: convert virtio-input-hid device to keycodemapdb".

cheers,
  Gerd


Re: [Qemu-devel] [PATCH 0/3] input: add keys and mouse buttons to virtio input
Posted by Daniel P. Berrangé 7 years, 9 months ago
On Thu, Jan 25, 2018 at 04:29:23PM +0100, Gerd Hoffmann wrote:
> On Mon, Jan 15, 2018 at 03:32:33PM +0000, Daniel P. Berrange wrote:
> > On Mon, Jan 15, 2018 at 04:18:53PM +0100, Gerd Hoffmann wrote:
> > >   Hi,
> > > 
> > > > Miika S (3):
> > > >   input: add mouse side buttons to virtio input
> > > >   input: virtio: don't send mouse wheel event twice
> > > 
> > > Cherry-picked these two ad the kbd update depends on the not-yet merged
> > > keycodemapdb update for virtio-keyboard.
> > 
> > It is merged actually in, so you can include the keycode fix too:
> > 
> > https://git.qemu.org/gitweb.cgi?p=keycodemapdb.git;a=commit;h=05dad417e9d0b37ee1fba33056d91a6b734b3357
> 
> To be exact: it conflicts (and I think also is obsoleted by) your patch
> "hw: convert virtio-input-hid device to keycodemapdb".

That patch doesn't touch keycodemapdb.

I did obsolete it though in an unrelated series:

  https://lists.gnu.org/archive/html/qemu-devel/2018-01/msg03528.html

You should be fine to just include all 3 of the patches in this series,
if you do a pull request for this series, before that py3 series gets
pulled.

Regards,
Daniel
-- 
|: https://berrange.com      -o-    https://www.flickr.com/photos/dberrange :|
|: https://libvirt.org         -o-            https://fstop138.berrange.com :|
|: https://entangle-photo.org    -o-    https://www.instagram.com/dberrange :|