[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH] ps2kbd: default to scancode_set 2, as with KBD_
From: |
Eduardo Habkost |
Subject: |
Re: [Qemu-devel] [PATCH] ps2kbd: default to scancode_set 2, as with KBD_CMD_RESET |
Date: |
Mon, 4 Apr 2016 15:50:41 -0300 |
User-agent: |
Mutt/1.5.23 (2014-03-12) |
On Wed, Mar 23, 2016 at 07:21:40AM +0100, Hervé Poussineau wrote:
> This line has been added in commit ef74679a810fe6858f625b9d52b68cc3fc61eb3d
> with
> other initializations. However, scancode set 0 doesn't exist (only 1, 2, 3).
> This works well as long as operating system is resetting keyboard, or
> overwriting
> the current scancode set with the one it wants.
>
> This fixes IBM 40p firmware, which doesn't bother sending KBD_CMD_RESET or
> KBD_CMD_SCANCODE.
>
> Signed-off-by: Hervé Poussineau <address@hidden>
Matches reset behavior documented at
http://www.computer-engineering.org/ps2keyboard/ (the best source
I could find for the PS/2 specs).
Reviewed-by: Eduardo Habkost <address@hidden>
Michael, do you want to merge this through the PC tree?
> ---
> hw/input/ps2.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/hw/input/ps2.c b/hw/input/ps2.c
> index 58892d5..a8aa36f 100644
> --- a/hw/input/ps2.c
> +++ b/hw/input/ps2.c
> @@ -628,7 +628,7 @@ static void ps2_kbd_reset(void *opaque)
> ps2_common_reset(&s->common);
> s->scan_enabled = 0;
> s->translate = 0;
> - s->scancode_set = 0;
> + s->scancode_set = 2;
> }
>
> static void ps2_mouse_reset(void *opaque)
> --
> 2.1.4
>
--
Eduardo
[Prev in Thread] |
Current Thread |
[Next in Thread] |
- Re: [Qemu-devel] [PATCH] ps2kbd: default to scancode_set 2, as with KBD_CMD_RESET,
Eduardo Habkost <=