On 01/13/2014 03:33 PM, Romain Naour wrote: > This patch adds the emulation for a serial Genius mouse using > Mouse Systems protocol (5bytes). > This protocol is compatible with most 3-button serial mouse. > > Signed-off-by: Romain Naour <romain.na...@openwide.fr> > --- > backends/Makefile.objs | 2 +- > backends/gnmouse.c | 339 > +++++++++++++++++++++++++++++++++++++++++++++++++ > include/sysemu/char.h | 3 + > qapi-schema.json | 1 + > qemu-char.c | 4 + > qemu-options.hx | 10 ++ > 6 files changed, 358 insertions(+), 1 deletion(-) > create mode 100644 backends/gnmouse.c
> +++ b/backends/gnmouse.c > @@ -0,0 +1,339 @@ > +/* > + * QEMU Genius GM-6 serial mouse emulation > + * > + * Adapted from msmouse > + * > + * Copyright (c) 2012 Romain Naour It is now 2014. > +++ b/qapi-schema.json > @@ -3617,6 +3617,7 @@ > 'null' : 'ChardevDummy', > 'mux' : 'ChardevMux', > 'msmouse': 'ChardevDummy', > + 'gnmouse': 'ChardevDummy', > 'braille': 'ChardevDummy', > 'stdio' : 'ChardevStdio', > 'console': 'ChardevDummy', Ideally, you should also document that this branch was not always available in the union (I was expecting to see a doc line with '(since 2.0)' somewhere in the patch); but looking at the existing schema, you're just copying bad practice of adding to an already under-documented union. -- Eric Blake eblake redhat com +1-919-301-3266 Libvirt virtualization library http://libvirt.org
signature.asc
Description: OpenPGP digital signature