Re: [Qemu-devel] [PATCH 0/4 v4] Support for LED state extension to Qemu VNC server
Applied. Thanks. Regards, Anthony Liguori
Re: [Qemu-devel] [PATCH 0/4 v4] Support for LED state extension to Qemu VNC server
Lei Li li...@linux.vnet.ibm.com writes: Hi guys, This patch series tries to add support for LED state extension to Qemu VNC server. The proposal has been sent few days ago as link below: http://lists.gnu.org/archive/html/qemu-devel/2013-04/msg01421.html The previous version as link: http://lists.nongnu.org/archive/html/qemu-devel/2013-04/msg04773.html Pleae let me know if there is anything else need be improved. Thanks! Reviewed-by: Anthony Liguori aligu...@us.ibm.com I'll give people some time to take a look before applying but it looks good to me. Regards, Anthony Liguori Changes since v3: - Add adjustment of turning off lock state sync logic in do_key_event() when VNC_FEATURE_LED_STATE supported suggested by Gerd. - Document improvement suggested by Eric. Changes since v2: - Address the comments from Anthony includes: - Just send 1 instead of the actual width and height. - Improve the document by adding the Pseudo-encoding number and simplify the description of the LED state encoding. Changes since v1: - Address the comments from Anthony includes: - Use Pseudo-encoding for led state; - Get rid of send_ext_leds_state_ack; - Add document for the led state Pseudo-encoding. Lei Li (4): vnc: Add SCROLL lock key to kbd_leds vnc: Support for LED state extension vnc: Adjust lock state sync logic with VNC_FEATURE_LED_STATE doc: document the Pseudo-encoding of LED state docs/vnc-ledstate-Pseudo-encoding.txt | 50 + ui/vnc.c | 56 - ui/vnc.h |3 ++ 3 files changed, 108 insertions(+), 1 deletions(-) create mode 100644 docs/vnc-ledstate-Pseudo-encoding.txt
[Qemu-devel] [PATCH 0/4 v4] Support for LED state extension to Qemu VNC server
Hi guys, This patch series tries to add support for LED state extension to Qemu VNC server. The proposal has been sent few days ago as link below: http://lists.gnu.org/archive/html/qemu-devel/2013-04/msg01421.html The previous version as link: http://lists.nongnu.org/archive/html/qemu-devel/2013-04/msg04773.html Pleae let me know if there is anything else need be improved. Thanks! Changes since v3: - Add adjustment of turning off lock state sync logic in do_key_event() when VNC_FEATURE_LED_STATE supported suggested by Gerd. - Document improvement suggested by Eric. Changes since v2: - Address the comments from Anthony includes: - Just send 1 instead of the actual width and height. - Improve the document by adding the Pseudo-encoding number and simplify the description of the LED state encoding. Changes since v1: - Address the comments from Anthony includes: - Use Pseudo-encoding for led state; - Get rid of send_ext_leds_state_ack; - Add document for the led state Pseudo-encoding. Lei Li (4): vnc: Add SCROLL lock key to kbd_leds vnc: Support for LED state extension vnc: Adjust lock state sync logic with VNC_FEATURE_LED_STATE doc: document the Pseudo-encoding of LED state docs/vnc-ledstate-Pseudo-encoding.txt | 50 + ui/vnc.c | 56 - ui/vnc.h |3 ++ 3 files changed, 108 insertions(+), 1 deletions(-) create mode 100644 docs/vnc-ledstate-Pseudo-encoding.txt