| Message ID | 20180115170243.24578-11-berrange@redhat.com |
|---|---|
| State | New |
| Headers | show |
| Series | Support building with py2 or py3 | expand |
On 01/15/2018 11:02 AM, Daniel P. Berrange wrote: > From: Miika S <miika9764@gmail.com> > > keycodemapdb updated to add the QKeyCodes muhenkan and katakanahiragana > > Signed-off-by: Miika S <miika9764@gmail.com> > --- > hw/input/virtio-input-hid.c | 7 +++++++ > qapi/ui.json | 5 ++++- > ui/keycodemapdb | 2 +- > 3 files changed, 12 insertions(+), 2 deletions(-) Why are you bumping the submodule here and again in 11/13? Can those two patches be squashed together? > +++ b/ui/keycodemapdb > @@ -1 +1 @@ > -Subproject commit 10739aa26051a5d49d88132604539d3ed085e72e > +Subproject commit 05dad417e9d0b37ee1fba33056d91a6b734b3357 >
On Mon, Jan 15, 2018 at 11:17:15AM -0600, Eric Blake wrote: > On 01/15/2018 11:02 AM, Daniel P. Berrange wrote: > > From: Miika S <miika9764@gmail.com> > > > > keycodemapdb updated to add the QKeyCodes muhenkan and katakanahiragana > > > > Signed-off-by: Miika S <miika9764@gmail.com> > > --- > > hw/input/virtio-input-hid.c | 7 +++++++ > > qapi/ui.json | 5 ++++- > > ui/keycodemapdb | 2 +- > > 3 files changed, 12 insertions(+), 2 deletions(-) > > Why are you bumping the submodule here and again in 11/13? Can those two > patches be squashed together? This patch is just temporarily pulled from another patch series, which I expect to get merged before this one does. > > +++ b/ui/keycodemapdb > > @@ -1 +1 @@ > > -Subproject commit 10739aa26051a5d49d88132604539d3ed085e72e > > +Subproject commit 05dad417e9d0b37ee1fba33056d91a6b734b3357 Regards, Daniel
diff --git a/hw/input/virtio-input-hid.c b/hw/input/virtio-input-hid.c index e78faec0b1..9628d289f9 100644 --- a/hw/input/virtio-input-hid.c +++ b/hw/input/virtio-input-hid.c @@ -139,6 +139,13 @@ static const unsigned int keymap_qcode[Q_KEY_CODE__MAX] = { [Q_KEY_CODE_META_L] = KEY_LEFTMETA, [Q_KEY_CODE_META_R] = KEY_RIGHTMETA, [Q_KEY_CODE_MENU] = KEY_MENU, + + [Q_KEY_CODE_MUHENKAN] = KEY_MUHENKAN, + [Q_KEY_CODE_HENKAN] = KEY_HENKAN, + [Q_KEY_CODE_KATAKANAHIRAGANA] = KEY_KATAKANAHIRAGANA, + [Q_KEY_CODE_COMPOSE] = KEY_COMPOSE, + [Q_KEY_CODE_RO] = KEY_RO, + [Q_KEY_CODE_YEN] = KEY_YEN, }; static const unsigned int keymap_button[INPUT_BUTTON__MAX] = { diff --git a/qapi/ui.json b/qapi/ui.json index 07b468f625..d6679aa8f5 100644 --- a/qapi/ui.json +++ b/qapi/ui.json @@ -748,6 +748,9 @@ # @ac_bookmarks: since 2.10 # altgr, altgr_r: dropped in 2.10 # +# @muhenkan: since 2.12 +# @katakanahiragana: since 2.12 +# # 'sysrq' was mistakenly added to hack around the fact that # the ps2 driver was not generating correct scancodes sequences # when 'alt+print' was pressed. This flaw is now fixed and the @@ -775,7 +778,7 @@ 'left', 'up', 'down', 'right', 'insert', 'delete', 'stop', 'again', 'props', 'undo', 'front', 'copy', 'open', 'paste', 'find', 'cut', 'lf', 'help', 'meta_l', 'meta_r', 'compose', 'pause', - 'ro', 'hiragana', 'henkan', 'yen', + 'ro', 'hiragana', 'henkan', 'yen', 'muhenkan', 'katakanahiragana', 'kp_comma', 'kp_equals', 'power', 'sleep', 'wake', 'audionext', 'audioprev', 'audiostop', 'audioplay', 'audiomute', 'volumeup', 'volumedown', 'mediaselect', diff --git a/ui/keycodemapdb b/ui/keycodemapdb index 10739aa260..05dad417e9 160000 --- a/ui/keycodemapdb +++ b/ui/keycodemapdb @@ -1 +1 @@ -Subproject commit 10739aa26051a5d49d88132604539d3ed085e72e +Subproject commit 05dad417e9d0b37ee1fba33056d91a6b734b3357