From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([140.186.70.92]:38418) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1QozZo-0005j3-78 for qemu-devel@nongnu.org; Thu, 04 Aug 2011 11:11:19 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1QozZd-0007th-8P for qemu-devel@nongnu.org; Thu, 04 Aug 2011 11:11:06 -0400 Received: from mx1.redhat.com ([209.132.183.28]:15331) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1QozZd-0007sA-1H for qemu-devel@nongnu.org; Thu, 04 Aug 2011 11:11:01 -0400 Received: from int-mx02.intmail.prod.int.phx2.redhat.com (int-mx02.intmail.prod.int.phx2.redhat.com [10.5.11.12]) by mx1.redhat.com (8.14.4/8.14.4) with ESMTP id p74FAo0v017249 (version=TLSv1/SSLv3 cipher=DHE-RSA-AES256-SHA bits=256 verify=OK) for ; Thu, 4 Aug 2011 11:10:50 -0400 From: Gerd Hoffmann Date: Thu, 4 Aug 2011 17:10:23 +0200 Message-Id: <1312470626-25872-14-git-send-email-kraxel@redhat.com> In-Reply-To: <1312470626-25872-1-git-send-email-kraxel@redhat.com> References: <1312470626-25872-1-git-send-email-kraxel@redhat.com> Subject: [Qemu-devel] [PATCH 13/16] usb-hid: add hid_has_events() List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: qemu-devel@nongnu.org Cc: Gerd Hoffmann Add hid_has_events function, use it to figure whenever there are pending events instead of checking and updating USBHIDState->changed. Setting ->changed to 1 on init is removed, that should have absolutely no effect as the initial state of ->idle is 0 so we report hid state anyway until the guest configures some idle time. Also should clear ->idle on reset. Signed-off-by: Gerd Hoffmann --- hw/usb-hid.c | 16 +++++++--------- 1 files changed, 7 insertions(+), 9 deletions(-) diff --git a/hw/usb-hid.c b/hw/usb-hid.c index 870cc66..b730692 100644 --- a/hw/usb-hid.c +++ b/hw/usb-hid.c @@ -88,7 +88,6 @@ typedef struct USBHIDState { int32_t protocol; uint8_t idle; int64_t next_idle_clock; - int changed; void *datain_opaque; void (*datain)(void *); } USBHIDState; @@ -444,12 +443,15 @@ static const uint8_t usb_hid_usage_keys[0x100] = { 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, }; +static bool hid_has_events(HIDState *hs) +{ + return hs->n > 0; +} + static void usb_hid_changed(HIDState *hs) { USBHIDState *us = container_of(hs, USBHIDState, hid); - us->changed = 1; - if (us->datain) { us->datain(us->datain_opaque); } @@ -742,6 +744,7 @@ static void usb_hid_handle_reset(USBDevice *dev) hid_handle_reset(&us->hid); us->protocol = 1; + us->idle = 0; } static void usb_hid_set_next_idle(USBHIDState *s, int64_t curtime) @@ -798,7 +801,6 @@ static int usb_hid_handle_control(USBDevice *dev, USBPacket *p, } else if (hs->kind == HID_KEYBOARD) { ret = hid_keyboard_poll(hs, data, length); } - us->changed = hs->n > 0; break; case SET_REPORT: if (hs->kind == HID_KEYBOARD) { @@ -849,7 +851,7 @@ static int usb_hid_handle_data(USBDevice *dev, USBPacket *p) case USB_TOKEN_IN: if (p->devep == 1) { int64_t curtime = qemu_get_clock_ns(vm_clock); - if (!us->changed && + if (!hid_has_events(hs) && (!us->idle || us->next_idle_clock - curtime > 0)) { return USB_RET_NAK; } @@ -860,7 +862,6 @@ static int usb_hid_handle_data(USBDevice *dev, USBPacket *p) ret = hid_keyboard_poll(hs, buf, p->iov.size); } usb_packet_copy(p, buf, ret); - us->changed = hs->n > 0; } else { goto fail; } @@ -914,9 +915,6 @@ static int usb_hid_initfn(USBDevice *dev, int kind) usb_desc_init(dev); hid_init(&us->hid, kind, usb_hid_changed); - - /* Force poll routine to be run and grab input the first time. */ - us->changed = 1; return 0; } -- 1.7.1