diff options
author | Dmitry Torokhov <dmitry.torokhov@gmail.com> | 2009-09-16 04:06:43 -0400 |
---|---|---|
committer | Dmitry Torokhov <dmitry.torokhov@gmail.com> | 2009-09-18 02:23:45 -0400 |
commit | 181d683d752c432635eda0f182ee71548c1f1820 (patch) | |
tree | 1fa397811ea6bcc73001fe1ab5b53525f2b88f74 /drivers/platform | |
parent | 0cb583fd2862f19ea88b02eb307d11c09e51e2f8 (diff) |
Input: libps2 - additional locking for i8042 ports
The serio ports on i8042 are not completely isolated; while we provide
enough locking to ensure proper serialization when accessing control
and data registers AUX and KBD ports can still have an effect on each
other on PS/2 protocol level. The most prominent effect is that
issuing a command for the device connected to one port may cause
abort of the command currently executing by the device connected to
another port.
Since i8042 nor serio subsystem are not aware of the details of the
PS/2 protocol (length of the commands and their replies and so on) the
locking should be done on libps2 level by adding special handling when
we see that we are dealing with serio port on i8042.
Signed-off-by: Dmitry Torokhov <dtor@mail.ru>
Diffstat (limited to 'drivers/platform')
-rw-r--r-- | drivers/platform/x86/acer-wmi.c | 2 |
1 files changed, 2 insertions, 0 deletions
diff --git a/drivers/platform/x86/acer-wmi.c b/drivers/platform/x86/acer-wmi.c index fb45f5ee8df1..454970d2d701 100644 --- a/drivers/platform/x86/acer-wmi.c +++ b/drivers/platform/x86/acer-wmi.c | |||
@@ -746,7 +746,9 @@ static acpi_status WMID_set_u32(u32 value, u32 cap, struct wmi_interface *iface) | |||
746 | return AE_BAD_PARAMETER; | 746 | return AE_BAD_PARAMETER; |
747 | if (quirks->mailled == 1) { | 747 | if (quirks->mailled == 1) { |
748 | param = value ? 0x92 : 0x93; | 748 | param = value ? 0x92 : 0x93; |
749 | i8042_lock_chip(); | ||
749 | i8042_command(¶m, 0x1059); | 750 | i8042_command(¶m, 0x1059); |
751 | i8042_unlock_chip(); | ||
750 | return 0; | 752 | return 0; |
751 | } | 753 | } |
752 | break; | 754 | break; |