ShuriZma
/
xqemu
mirror of https://github.com/xqemu/xqemu.git
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

usb hid: move head+n to common struct 

This patch moves the 'head' and 'n' fields from USBMouseState and
USBKeyboardState to the common USBHIDState struct.

Signed-off-by: Gerd Hoffmann <kraxel@redhat.com>
This commit is contained in:
Gerd Hoffmann 2011-01-14 09:23:22 +01:00
parent 5fae51a9c2
commit 42292d4e51
1 changed files with 22 additions and 24 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

46
hw/usb-hid.c
View File

@ -56,16 +56,12 @@ typedef struct USBPointerEvent {
typedef struct USBMouseState { typedef struct USBMouseState {
USBPointerEvent queue[QUEUE_LENGTH]; USBPointerEvent queue[QUEUE_LENGTH];
uint32_t head; /* index into circular queue */
uint32_t n;
int mouse_grabbed; int mouse_grabbed;
QEMUPutMouseEntry *eh_entry; QEMUPutMouseEntry *eh_entry;
} USBMouseState; } USBMouseState;
typedef struct USBKeyboardState { typedef struct USBKeyboardState {
uint32_t keycodes[QUEUE_LENGTH]; uint32_t keycodes[QUEUE_LENGTH];
uint32_t head; /* index into circular queue */
uint32_t n;
uint16_t modifiers; uint16_t modifiers;
uint8_t leds; uint8_t leds;
uint8_t key[16]; uint8_t key[16];
@ -78,6 +74,8 @@ typedef struct USBHIDState {
USBMouseState ptr; USBMouseState ptr;
USBKeyboardState kbd; USBKeyboardState kbd;
}; };
uint32_t head; /* index into circular queue */
uint32_t n;
int kind; int kind;
int protocol; int protocol;
uint8_t idle; uint8_t idle;
@ -468,7 +466,7 @@ static void usb_pointer_event(void *opaque,
{ {
USBHIDState *hs = opaque; USBHIDState *hs = opaque;
USBMouseState *s = &hs->ptr; USBMouseState *s = &hs->ptr;
unsigned use_slot = (s->head + s->n - 1) & QUEUE_MASK; unsigned use_slot = (hs->head + hs->n - 1) & QUEUE_MASK;
unsigned previous_slot = (use_slot - 1) & QUEUE_MASK; unsigned previous_slot = (use_slot - 1) & QUEUE_MASK;
/* We combine events where feasible to keep the queue small. We shouldn't /* We combine events where feasible to keep the queue small. We shouldn't
@ -476,15 +474,15 @@ static void usb_pointer_event(void *opaque,
* that would change the location of the button state change. When the * that would change the location of the button state change. When the
* queue is empty, a second event is needed because we don't know if * queue is empty, a second event is needed because we don't know if
* the first event changed the button state. */ * the first event changed the button state. */
if (s->n == QUEUE_LENGTH) { if (hs->n == QUEUE_LENGTH) {
/* Queue full. Discard old button state, combine motion normally. */ /* Queue full. Discard old button state, combine motion normally. */
s->queue[use_slot].buttons_state = buttons_state; s->queue[use_slot].buttons_state = buttons_state;
} else if (s->n < 2 || } else if (hs->n < 2 ||
s->queue[use_slot].buttons_state != buttons_state || s->queue[use_slot].buttons_state != buttons_state ||
s->queue[previous_slot].buttons_state != s->queue[use_slot].buttons_state) { s->queue[previous_slot].buttons_state != s->queue[use_slot].buttons_state) {
/* Cannot or should not combine, so add an empty item to the queue. */ /* Cannot or should not combine, so add an empty item to the queue. */
QUEUE_INCR(use_slot); QUEUE_INCR(use_slot);
s->n++; hs->n++;
usb_pointer_event_clear(&s->queue[use_slot], buttons_state); usb_pointer_event_clear(&s->queue[use_slot], buttons_state);
} }
usb_pointer_event_combine(&s->queue[use_slot], usb_pointer_event_combine(&s->queue[use_slot],
@ -499,24 +497,25 @@ static void usb_keyboard_event(void *opaque, int keycode)
USBKeyboardState *s = &hs->kbd; USBKeyboardState *s = &hs->kbd;
int slot; int slot;
if (s->n == QUEUE_LENGTH) { if (hs->n == QUEUE_LENGTH) {
fprintf(stderr, "usb-kbd: warning: key event queue full\n"); fprintf(stderr, "usb-kbd: warning: key event queue full\n");
return; return;
} }
slot = (s->head + s->n) & QUEUE_MASK; s->n++; slot = (hs->head + hs->n) & QUEUE_MASK; hs->n++;
s->keycodes[slot] = keycode; s->keycodes[slot] = keycode;
usb_hid_changed(hs); usb_hid_changed(hs);
} }
static void usb_keyboard_process_keycode(USBKeyboardState *s) static void usb_keyboard_process_keycode(USBHIDState *hs)
{ {
USBKeyboardState *s = &hs->kbd;
uint8_t hid_code, key; uint8_t hid_code, key;
int i, keycode, slot; int i, keycode, slot;
if (s->n == 0) { if (hs->n == 0) {
return; return;
} }
slot = s->head & QUEUE_MASK; QUEUE_INCR(s->head); s->n--; slot = hs->head & QUEUE_MASK; QUEUE_INCR(hs->head); hs->n--;
keycode = s->keycodes[slot]; keycode = s->keycodes[slot];
key = keycode & 0x7f; key = keycode & 0x7f;
@ -590,7 +589,7 @@ static int usb_pointer_poll(USBHIDState *hs, uint8_t *buf, int len)
/* When the buffer is empty, return the last event. Relative /* When the buffer is empty, return the last event. Relative
movements will all be zero. */ movements will all be zero. */
index = (s->n ? s->head : s->head - 1); index = (hs->n ? hs->head : hs->head - 1);
e = &s->queue[index & QUEUE_MASK]; e = &s->queue[index & QUEUE_MASK];
if (hs->kind == USB_MOUSE) { if (hs->kind == USB_MOUSE) {
@ -613,12 +612,12 @@ static int usb_pointer_poll(USBHIDState *hs, uint8_t *buf, int len)
if (e->buttons_state & MOUSE_EVENT_MBUTTON) if (e->buttons_state & MOUSE_EVENT_MBUTTON)
b |= 0x04; b |= 0x04;
if (s->n && if (hs->n &&
!e->dz && !e->dz &&
(hs->kind == USB_TABLET || (!e->xdx && !e->ydy))) { (hs->kind == USB_TABLET || (!e->xdx && !e->ydy))) {
/* that deals with this event */ /* that deals with this event */
QUEUE_INCR(s->head); QUEUE_INCR(hs->head);
s->n--; hs->n--;
} }
/* Appears we have to invert the wheel direction */ /* Appears we have to invert the wheel direction */
@ -664,7 +663,7 @@ static int usb_keyboard_poll(USBHIDState *hs, uint8_t *buf, int len)
if (len < 2) if (len < 2)
return 0; return 0;
usb_keyboard_process_keycode(s); usb_keyboard_process_keycode(hs);
buf[0] = s->modifiers & 0xff; buf[0] = s->modifiers & 0xff;
buf[1] = 0; buf[1] = 0;
@ -702,8 +701,8 @@ static void usb_mouse_handle_reset(USBDevice *dev)
USBHIDState *s = (USBHIDState *)dev; USBHIDState *s = (USBHIDState *)dev;
memset(s->ptr.queue, 0, sizeof (s->ptr.queue)); memset(s->ptr.queue, 0, sizeof (s->ptr.queue));
s->ptr.head = 0; s->head = 0;
s->ptr.n = 0; s->n = 0;
s->protocol = 1; s->protocol = 1;
} }
@ -713,8 +712,8 @@ static void usb_keyboard_handle_reset(USBDevice *dev)
qemu_add_kbd_event_handler(usb_keyboard_event, s); qemu_add_kbd_event_handler(usb_keyboard_event, s);
memset(s->kbd.keycodes, 0, sizeof (s->kbd.keycodes)); memset(s->kbd.keycodes, 0, sizeof (s->kbd.keycodes));
s->kbd.head = 0; s->head = 0;
s->kbd.n = 0; s->n = 0;
memset(s->kbd.key, 0, sizeof (s->kbd.key)); memset(s->kbd.key, 0, sizeof (s->kbd.key));
s->kbd.keys = 0; s->kbd.keys = 0;
s->protocol = 1; s->protocol = 1;
@ -822,12 +821,11 @@ static int usb_hid_handle_data(USBDevice *dev, USBPacket *p)
usb_hid_set_next_idle(s, curtime); usb_hid_set_next_idle(s, curtime);
if (s->kind == USB_MOUSE || s->kind == USB_TABLET) { if (s->kind == USB_MOUSE || s->kind == USB_TABLET) {
ret = usb_pointer_poll(s, p->data, p->len); ret = usb_pointer_poll(s, p->data, p->len);
s->changed = s->ptr.n > 0;
} }
else if (s->kind == USB_KEYBOARD) { else if (s->kind == USB_KEYBOARD) {
ret = usb_keyboard_poll(s, p->data, p->len); ret = usb_keyboard_poll(s, p->data, p->len);
s->changed = s->kbd.n > 0;
} }
s->changed = s->n > 0;
} else { } else {
goto fail; goto fail;
} }