From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([209.51.188.92]:41086) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1gwGQp-00052u-1P for qemu-devel@nongnu.org; Tue, 19 Feb 2019 20:12:15 -0500 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1gwGJ3-0001nT-Uw for qemu-devel@nongnu.org; Tue, 19 Feb 2019 20:04:19 -0500 From: =?UTF-8?q?Philippe=20Mathieu-Daud=C3=A9?= Date: Wed, 20 Feb 2019 02:02:10 +0100 Message-Id: <20190220010232.18731-4-philmd@redhat.com> In-Reply-To: <20190220010232.18731-1-philmd@redhat.com> References: <20190220010232.18731-1-philmd@redhat.com> MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: quoted-printable Subject: [Qemu-devel] [PATCH v3 03/25] chardev/wctablet: Use unsigned type to hold unsigned value List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: qemu-devel@nongnu.org, Prasad J Pandit , =?UTF-8?q?Marc-Andr=C3=A9=20Lureau?= , Paolo Bonzini Cc: Jason Wang , Anthony Perard , qemu-ppc@nongnu.org, Stefan Berger , David Gibson , Gerd Hoffmann , Zhang Chen , xen-devel@lists.xenproject.org, Cornelia Huck , Samuel Thibault , Christian Borntraeger , Amit Shah , Li Zhijian , Corey Minyard , "Michael S. Tsirkin" , Paul Durrant , Halil Pasic , Stefano Stabellini , qemu-s390x@nongnu.org, Pavel Dovgalyuk , =?UTF-8?q?Philippe=20Mathieu-Daud=C3=A9?= TabletChardev::query is an array of uint8_t. Use the same type to hold it (this also silent a -Wsign-conversion warning in the trace function). Signed-off-by: Philippe Mathieu-Daud=C3=A9 --- chardev/trace-events | 2 +- chardev/wctablet.c | 5 +++-- 2 files changed, 4 insertions(+), 3 deletions(-) diff --git a/chardev/trace-events b/chardev/trace-events index d0e5f3bbc1..562bfe70e9 100644 --- a/chardev/trace-events +++ b/chardev/trace-events @@ -5,7 +5,7 @@ wct_init(void) "" wct_cmd_re(void) "" wct_cmd_st(void) "" wct_cmd_sp(void) "" -wct_cmd_ts(int input) "0x%02x" +wct_cmd_ts(uint8_t input) "0x%02x" wct_cmd_other(const char *cmd) "%s" wct_speed(int speed) "%d" =20 diff --git a/chardev/wctablet.c b/chardev/wctablet.c index 35dbd29a33..cf7a08a363 100644 --- a/chardev/wctablet.c +++ b/chardev/wctablet.c @@ -207,7 +207,8 @@ static int wctablet_chr_write(struct Chardev *chr, const uint8_t *buf, int len) { TabletChardev *tablet =3D WCTABLET_CHARDEV(chr); - unsigned int i, clen; + size_t i; + unsigned int clen; char *pos; =20 if (tablet->line_speed !=3D 9600) { @@ -269,7 +270,7 @@ static int wctablet_chr_write(struct Chardev *chr, =20 } else if (strncmp((char *)tablet->query, "TS", 2) =3D=3D 0 && clen =3D=3D 3) { - unsigned int input =3D tablet->query[2]; + uint8_t input =3D tablet->query[2]; uint8_t codes[7] =3D { 0xa3, ((input & 0x80) =3D=3D 0) ? 0x7e : 0x7f, --=20 2.20.1