Commit ae821c1b authored by Lee, Chun-Yi's avatar Lee, Chun-Yi Committed by Matthew Garrett

acer-wmi: fix bitwise bug when set device state

Fix a bitwise bug that was found by Joern Heissler, it must be OR
but not AND when we query current device state.
Acked-by: default avatarJoern Heissler <linux-acpi@joern.heissler.de>
Cc: Carlos Corbacho <carlos@strangeworlds.co.uk>
Cc: Matthew Garrett <mjg@redhat.com>
Cc: Dmitry Torokhov <dtor@mail.ru>
Cc: Corentin Chary <corentincj@iksaif.net>
Cc: Thomas Renninger <trenn@suse.de>
Signed-off-by: default avatarLee, Chun-Yi <jlee@novell.com>
Signed-off-by: default avatarMatthew Garrett <mjg@redhat.com>
parent 92530664
...@@ -1156,9 +1156,9 @@ static acpi_status wmid3_set_device_status(u32 value, u16 device) ...@@ -1156,9 +1156,9 @@ static acpi_status wmid3_set_device_status(u32 value, u16 device)
struct wmid3_gds_input_param params = { struct wmid3_gds_input_param params = {
.function_num = 0x1, .function_num = 0x1,
.hotkey_number = 0x01, .hotkey_number = 0x01,
.devices = ACER_WMID3_GDS_WIRELESS & .devices = ACER_WMID3_GDS_WIRELESS |
ACER_WMID3_GDS_THREEG & ACER_WMID3_GDS_THREEG |
ACER_WMID3_GDS_WIMAX & ACER_WMID3_GDS_WIMAX |
ACER_WMID3_GDS_BLUETOOTH, ACER_WMID3_GDS_BLUETOOTH,
}; };
struct acpi_buffer input = { struct acpi_buffer input = {
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment