diff options
| author | Kevin O'Connor <kevin@koconnor.net> | 2017-02-11 12:13:03 -0500 |
|---|---|---|
| committer | Kevin O'Connor <kevin@koconnor.net> | 2017-02-11 12:13:03 -0500 |
| commit | dbf9dd27f3aefc171839d60c188134da8ad3089d (patch) | |
| tree | bfcc8507a87539376e0bc9457c84f6c12d7b51b8 | |
| parent | 970af16e6e5a6cb8de6b6658663b8adc00d25b8b (diff) | |
| download | qemu-seabios-dbf9dd27f3aefc171839d60c188134da8ad3089d.tar.gz | |
ps2port: Disable keyboard/mouse prior to resetting ps2 controller
If one of the ps2 ports is enabled prior to the ps2 controller reset
then it is possible a device event (eg, key press or mouse move) could
be mistaken for the controller reset response code. This would result
in a failure and the keyboard would be disabled during the boot.
Always disabling the keyboard and mouse prior to reset ensures that
the controller reset return code can be read properly.
Tested-by: Paul Menzel <paulepanter@users.sourceforge.net>
Signed-off-by: Kevin O'Connor <kevin@koconnor.net>
| -rw-r--r-- | src/hw/ps2port.c | 13 |
1 files changed, 12 insertions, 1 deletions
diff --git a/src/hw/ps2port.c b/src/hw/ps2port.c index d9727d2..88b1059 100644 --- a/src/hw/ps2port.c +++ b/src/hw/ps2port.c @@ -449,11 +449,22 @@ ps2_check_event(void) static void ps2_keyboard_setup(void *data) { - /* flush incoming keys */ + // flush incoming keys (also verifies port is likely present) int ret = i8042_flush(); if (ret) return; + // Disable keyboard / mouse and drain any input they may have sent + ret = i8042_command(I8042_CMD_KBD_DISABLE, NULL); + if (ret) + return; + ret = i8042_command(I8042_CMD_AUX_DISABLE, NULL); + if (ret) + return; + ret = i8042_flush(); + if (ret) + return; + // Controller self-test. u8 param[2]; ret = i8042_command(I8042_CMD_CTL_TEST, param); |