On Mar 23, 2016, at 10:37 AM, Peter Maydell wrote: > On 15 March 2016 at 17:13, Programmingkid <programmingk...@gmail.com> wrote: >> This patchset adds QKeyCode support to the adb and cocoa code. >> >> Note: you do not need to be on a Mac to test out the adb.c, qapi-schema.json, >> and adb-keys.h files. Only the cocoa.m file changes are Mac specific. >> >> If you are using Linux as a guest, then the xev command is what you could >> use to >> test out these patches. For a Mac OS guest the Key Caps application would >> help >> with testing out these patches. >> >> John Arbuckle (4): >> hw/input/adb.c: implement QKeyCode support >> ui/cocoa.m: switch to QKeyCode >> adb-keys.h: initial commit >> qapi-schema.json: Add power and keypad equal keys > > I've put the qapi-schema.json and the ui/cocoa.m patches into the > cocoa pull I've just sent. For the ADB keyboard part, I definitely > think that the best approach is the suggestion I made on an earlier > series: split the changes up into "move to using QKeyCode and the > adb-keys.h enum values, but don't change any behaviour of what keys > get sent", and "fix bugs, add missing keys, etc" as separate changes. > This will be much easier to review.
Would it be ok to send the adb-keys.h and adb.c patches separately instead of using a series? It would make things a lot easier for me. You want the adb.c patch so that it does not have the keypad equals and power keys in it yet? You want those keys added in a separate patch?