From 417635733eb6bc44584db5d605c8fb257d373f78 Mon Sep 17 00:00:00 2001
From: Ed Schouten <ed@FreeBSD.org>
Date: Mon, 8 Jun 2009 20:24:29 +0000
Subject: [PATCH] Use proper types in kbdmux_kbd_getc():

- The return value should be a signed integer, because -1 means failure.
- The c variable should be unsigned, to force it to be zero-extended
  when returned.

Reported by:	Andreas Tobler <andreast-list fgznet ch>
---
 sys/dev/kbdmux/kbdmux.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/sys/dev/kbdmux/kbdmux.c b/sys/dev/kbdmux/kbdmux.c
index 0f1fe42a200..990c8c36412 100644
--- a/sys/dev/kbdmux/kbdmux.c
+++ b/sys/dev/kbdmux/kbdmux.c
@@ -181,10 +181,10 @@ kbdmux_kbd_putc(kbdmux_state_t *state, char c)
 	state->ks_inq_length++;
 }
 
-static char
+static int
 kbdmux_kbd_getc(kbdmux_state_t *state)
 {
-	char c;
+	unsigned char c;
 
 	if (state->ks_inq_length == 0)
 		return (-1);