On 24 March 2016 at 14:09, Programmingkid <programmingk...@gmail.com> wrote: > The old pc scancode translation is replaced with QEMU's QKeyCode. > > Signed-off-by: John Arbuckle <programmingk...@gmail.com> > --- > *v2 changes > Changed order of this patch.
I wrote a quick test program to check that this patch doesn't change what key we generate for any particular qcode (you can find the test program at http://people.linaro.org/~peter.maydell/adb-test.c), but it shows up a handful of changes: qcode 5 pckey 0x64 oldadb 0x0 newadb 0x7c qcode 82 pckey 0x54 oldadb 0x0 newadb 0x69 qcode 93 pckey 0x56 oldadb 0xa newadb 0x0 qcode 96 pckey 0xb7 oldadb 0x0 newadb 0x69 qcode 118 pckey 0x0 oldadb 0x0 newadb 0x72 qcode 120 pckey 0xdc oldadb 0x7e newadb 0x37 qcode 122 pckey 0x0 oldadb 0x0 newadb 0x71 qcode 125 pckey 0x0 oldadb 0x0 newadb 0x51 That's Q_KEY_CODE_ALTGR, Q_KEY_CODE_SYSRQ, Q_KEY_CODE_LESS, Q_KEY_CODE_PRINT, Q_KEY_CODE_HELP, Q_KEY_CODE_META_R, Q_KEY_CODE_PAUSE, Q_KEY_CODE_KP_EQUALS. I suspect these are all bugfixes to the layout, but they should be done in a separate patch, so that this patch doesn't change behaviour at all. If you fix the qcode_to_adb_keycode[] array so that it gives the same results for those entries as the existing code then this patch is good. thanks -- PMM