From: Gerd Hoffmann <kraxel@redhat.com>
To: qemu-devel@nongnu.org
Cc: avg.tolik@gmail.com, Gerd Hoffmann <kraxel@redhat.com>
Subject: [Qemu-devel] [PATCH 12/17] wctablet: drop timer, hook into chr->accept_input instead
Date: Fri, 6 Jan 2017 09:55:40 +0100 [thread overview]
Message-ID: <1483692945-9866-13-git-send-email-kraxel@redhat.com> (raw)
In-Reply-To: <1483692945-9866-1-git-send-email-kraxel@redhat.com>
Signed-off-by: Gerd Hoffmann <kraxel@redhat.com>
---
backends/wctablet.c | 26 +++++++-------------------
1 file changed, 7 insertions(+), 19 deletions(-)
diff --git a/backends/wctablet.c b/backends/wctablet.c
index f21b1f4..28daf26 100644
--- a/backends/wctablet.c
+++ b/backends/wctablet.c
@@ -76,10 +76,7 @@ int COMMON_SPEAD = 900 * 1000;
// This structure is used to save private info for Wacom Tablet.
typedef struct {
- struct QEMUTimer *transmit_timer;
- /* QEMU timer */
- uint64_t transmit_time;
- /* time to transmit a char in ticks */
+ CharDriverState *chr;
uint8_t query[100];
int query_index;
/* Query string from serial */
@@ -89,6 +86,8 @@ typedef struct {
int line_speed;
} TabletState;
+static void wctablet_chr_accept_input(CharDriverState *chr);
+
static void wctablet_shift_input(TabletState *tablet, int count)
{
tablet->query_index -= count;
@@ -104,6 +103,7 @@ static void wctablet_queue_output(TabletState *tablet, uint8_t *buf, int count)
memcpy(tablet->outbuf + tablet->outlen, buf, count);
tablet->outlen += count;
+ wctablet_chr_accept_input(tablet->chr);
}
static void wctablet_reset(TabletState *tablet)
@@ -146,9 +146,8 @@ static void wctablet_event(void *opaque, int x,
wctablet_queue_output(tablet, codes, 7);
}
-static void wctablet_handler(void *opaque)
+static void wctablet_chr_accept_input(CharDriverState *chr)
{
- CharDriverState *chr = (CharDriverState *) opaque;
TabletState *tablet = (TabletState *) chr->opaque;
int len, canWrite; // , i;
@@ -165,9 +164,6 @@ static void wctablet_handler(void *opaque)
memmove(tablet->outbuf, tablet->outbuf + len, tablet->outlen);
}
}
-
- timer_mod(tablet->transmit_timer,
- qemu_clock_get_ns(QEMU_CLOCK_VIRTUAL) + tablet->transmit_time);
}
static int wctablet_chr_write(struct CharDriverState *s,
@@ -285,24 +281,16 @@ static CharDriverState *qemu_chr_open_wctablet(const char *id,
chr->chr_write = wctablet_chr_write;
chr->chr_ioctl = wctablet_chr_ioctl;
chr->chr_free = wctablet_chr_free;
+ chr->chr_accept_input = wctablet_chr_accept_input;
*be_opened = true;
- /* create a new QEMU's timer with wctablet_handler() as timeout handler. */
- tablet->transmit_timer = timer_new_ns(QEMU_CLOCK_VIRTUAL,
- (QEMUTimerCB *) wctablet_handler, chr);
-
- tablet->transmit_time = COMMON_SPEAD;
-
- timer_mod(tablet->transmit_timer,
- qemu_clock_get_ns(QEMU_CLOCK_VIRTUAL) + tablet->transmit_time);
-
-
/* init state machine */
memcpy(tablet->outbuf, WC_FULL_CONFIG_STRING, WC_FULL_CONFIG_STRING_LENGTH);
tablet->outlen = WC_FULL_CONFIG_STRING_LENGTH;
tablet->query_index = 0;
chr->opaque = tablet;
+ tablet->chr = chr;
qemu_add_mouse_event_handler(wctablet_event, chr, 1,
"QEMU Wacome Pen Tablet");
--
1.8.3.1
next prev parent reply other threads:[~2017-01-06 8:56 UTC|newest]
Thread overview: 25+ messages / expand[flat|nested] mbox.gz Atom feed top
2017-01-06 8:55 [Qemu-devel] [PATCH 00/17] add serial wacom tablet emulation (gsoc 2016) Gerd Hoffmann
2017-01-06 8:55 ` [Qemu-devel] [PATCH 01/17] Add wctablet device Gerd Hoffmann
2017-01-06 13:15 ` Eric Blake
2017-01-06 8:55 ` [Qemu-devel] [PATCH 02/17] wctablet: add wctablet_queue_output helper Gerd Hoffmann
2017-01-06 8:55 ` [Qemu-devel] [PATCH 03/17] wctablet: save all chars in the query buffer Gerd Hoffmann
2017-01-06 8:55 ` [Qemu-devel] [PATCH 04/17] wctablet: drop wctablet_commands_names Gerd Hoffmann
2017-01-06 8:55 ` [Qemu-devel] [PATCH 05/17] wctablet: strip leading \r + \n from buffer Gerd Hoffmann
2017-01-06 8:55 ` [Qemu-devel] [PATCH 06/17] wctablet: track line speed, reset on speed changes Gerd Hoffmann
2017-01-06 8:55 ` [Qemu-devel] [PATCH 07/17] wctablet: operate on line speed 9600 Gerd Hoffmann
2017-01-06 8:55 ` [Qemu-devel] [PATCH 08/17] wctablet: drop debug code from wctablet_handler Gerd Hoffmann
2017-01-06 13:17 ` Eric Blake
2017-01-09 7:50 ` Gerd Hoffmann
2017-01-06 8:55 ` [Qemu-devel] [PATCH 09/17] wctablet: add wctablet_shift_input Gerd Hoffmann
2017-01-06 8:55 ` [Qemu-devel] [PATCH 10/17] wctablet: move init/detect sequence Gerd Hoffmann
2017-01-06 8:55 ` [Qemu-devel] [PATCH 11/17] wctablet: revamp command parser Gerd Hoffmann
2017-01-06 8:55 ` Gerd Hoffmann [this message]
2017-01-06 8:55 ` [Qemu-devel] [PATCH 13/17] wctablet: drop DPRINTF, add trace events instead Gerd Hoffmann
2017-01-06 13:19 ` Eric Blake
2017-01-06 8:55 ` [Qemu-devel] [PATCH 14/17] wctablet: misc cleanups Gerd Hoffmann
2017-01-06 13:19 ` Eric Blake
2017-01-06 8:55 ` [Qemu-devel] [PATCH 15/17] wctablet: switch to new input interface Gerd Hoffmann
2017-01-06 8:55 ` [Qemu-devel] [PATCH 16/17] wctablet: update file comment Gerd Hoffmann
2017-01-06 13:20 ` Eric Blake
2017-01-06 8:55 ` [Qemu-devel] [PATCH 17/17] wctablet: implement ST and SP commands Gerd Hoffmann
2017-01-06 9:22 ` [Qemu-devel] [PATCH 00/17] add serial wacom tablet emulation (gsoc 2016) no-reply
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=1483692945-9866-13-git-send-email-kraxel@redhat.com \
--to=kraxel@redhat.com \
--cc=avg.tolik@gmail.com \
--cc=qemu-devel@nongnu.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).