diff options
| author | Mark Cave-Ayland <mark.cave-ayland@ilande.co.uk> | 2020-06-23 21:49:16 +0100 |
|---|---|---|
| committer | Mark Cave-Ayland <mark.cave-ayland@ilande.co.uk> | 2020-06-26 10:13:51 +0100 |
| commit | 167f1667b120ddac301163da64b57b19ce7cf622 (patch) | |
| tree | 428bd195a3aa3991e33924f23419b50cf4747868 /include/hw | |
| parent | bcaaefdbb2167858d3b92745b79473deb49c417a (diff) | |
| download | qemu-167f1667b120ddac301163da64b57b19ce7cf622.zip qemu-167f1667b120ddac301163da64b57b19ce7cf622.tar.gz qemu-167f1667b120ddac301163da64b57b19ce7cf622.tar.bz2 | |
adb: fix adb-mouse read length and revert disable-reg3-direct-writes workaround
Commit 84051eb400 "adb: add property to disable direct reg 3 writes" introduced
a workaround for spurious writes to ADB register 3 when MacOS 9 enables
autopoll on the mouse device. Further analysis shows that the problem is that
only a partial request is sent, and since the len parameter is ignored then
stale data from the previous request is used causing the incorrect address
assignment.
Remove the disable-reg3-direct-writes workaround and instead check the length
parameter when the write is attempted, discarding the invalid request.
Signed-off-by: Mark Cave-Ayland <mark.cave-ayland@ilande.co.uk>
Tested-by: Finn Thain <fthain@telegraphics.com.au>
Acked-by: Laurent Vivier <laurent@vivier.eu>
Message-Id: <20200623204936.24064-3-mark.cave-ayland@ilande.co.uk>
Diffstat (limited to 'include/hw')
| -rw-r--r-- | include/hw/input/adb.h | 1 |
1 files changed, 0 insertions, 1 deletions
diff --git a/include/hw/input/adb.h b/include/hw/input/adb.h index b7b32e2..4d2c565 100644 --- a/include/hw/input/adb.h +++ b/include/hw/input/adb.h @@ -49,7 +49,6 @@ struct ADBDevice { int devaddr; int handler; - bool disable_direct_reg3_writes; }; #define ADB_DEVICE_CLASS(cls) \ |