msi-wmi: Introduced quirk_last_pressed
authorMaxim Mikityanskiy <maxtram95@gmail.com>
Sat, 15 Dec 2012 17:31:35 +0000 (19:31 +0200)
committerMatthew Garrett <matthew.garrett@nebula.com>
Sun, 24 Feb 2013 22:49:54 +0000 (14:49 -0800)
Introduced quirk_last_pressed variable that would indicate if
last_pressed is used or not. Also converted last_pressed to simple
variable in order to allow keymap to be non-contiguous.

Signed-off-by: Maxim Mikityanskiy <maxtram95@gmail.com>
Signed-off-by: Matthew Garrett <matthew.garrett@nebula.com>
drivers/platform/x86/msi-wmi.c

index 3a6061985e4ded97a9b5abc89521b0e5c7812508..739bd4d17c231a12d84cdf93a2d2a3a1866d3223 100644 (file)
@@ -43,8 +43,7 @@ MODULE_ALIAS("wmi:" MSIWMI_BIOS_GUID);
 MODULE_ALIAS("wmi:" MSIWMI_EVENT_GUID);
 
 enum msi_scancodes {
-       MSI_SCANCODE_BASE       = 0xD0,
-       MSI_KEY_BRIGHTNESSUP    = MSI_SCANCODE_BASE,
+       MSI_KEY_BRIGHTNESSUP    = 0xD0,
        MSI_KEY_BRIGHTNESSDOWN,
        MSI_KEY_VOLUMEUP,
        MSI_KEY_VOLUMEDOWN,
@@ -58,7 +57,9 @@ static struct key_entry msi_wmi_keymap[] = {
        { KE_KEY, MSI_KEY_MUTE,                 {KEY_MUTE} },
        { KE_END, 0 }
 };
-static ktime_t last_pressed[ARRAY_SIZE(msi_wmi_keymap) - 1];
+
+static ktime_t last_pressed;
+static bool quirk_last_pressed;
 
 static const char *event_wmi_guid;
 
@@ -153,7 +154,6 @@ static void msi_wmi_notify(u32 value, void *context)
        struct acpi_buffer response = { ACPI_ALLOCATE_BUFFER, NULL };
        static struct key_entry *key;
        union acpi_object *obj;
-       ktime_t cur;
        acpi_status status;
 
        status = wmi_get_event_data(value, &response);
@@ -169,12 +169,15 @@ static void msi_wmi_notify(u32 value, void *context)
                pr_debug("Eventcode: 0x%x\n", eventcode);
                key = sparse_keymap_entry_from_scancode(msi_wmi_input_dev,
                                eventcode);
-               if (key) {
-                       ktime_t diff;
-                       cur = ktime_get_real();
-                       diff = ktime_sub(cur, last_pressed[key->code -
-                                       MSI_SCANCODE_BASE]);
-                       /* Ignore event if the same event happened in a 50 ms
+               if (!key) {
+                       pr_info("Unknown key pressed - %x\n", eventcode);
+                       goto msi_wmi_notify_exit;
+               }
+
+               if (quirk_last_pressed) {
+                       ktime_t cur = ktime_get_real();
+                       ktime_t diff = ktime_sub(cur, last_pressed);
+                       /* Ignore event if any event happened in a 50 ms
                           timeframe -> Key press may result in 10-20 GPEs */
                        if (ktime_to_us(diff) < 1000 * 50) {
                                pr_debug("Suppressed key event 0x%X - "
@@ -182,21 +185,19 @@ static void msi_wmi_notify(u32 value, void *context)
                                         key->code, ktime_to_us(diff));
                                goto msi_wmi_notify_exit;
                        }
-                       last_pressed[key->code - MSI_SCANCODE_BASE] = cur;
-
-                       if (key->type == KE_KEY &&
-                       /* Brightness is served via acpi video driver */
-                       (backlight ||
-                       (key->code != MSI_KEY_BRIGHTNESSUP &&
-                       key->code != MSI_KEY_BRIGHTNESSDOWN))) {
-                               pr_debug("Send key: 0x%X - "
-                                        "Input layer keycode: %d\n",
-                                        key->code, key->keycode);
-                               sparse_keymap_report_entry(msi_wmi_input_dev,
-                                               key, 1, true);
-                       }
-               } else
-                       pr_info("Unknown key pressed - %x\n", eventcode);
+                       last_pressed = cur;
+               }
+
+               if (key->type == KE_KEY &&
+               /* Brightness is served via acpi video driver */
+               (backlight ||
+               (key->code != MSI_KEY_BRIGHTNESSUP &&
+               key->code != MSI_KEY_BRIGHTNESSDOWN))) {
+                       pr_debug("Send key: 0x%X - Input layer keycode: %d\n",
+                                key->code, key->keycode);
+                       sparse_keymap_report_entry(msi_wmi_input_dev, key, 1,
+                                                  true);
+               }
        } else
                pr_info("Unknown event received\n");
 
@@ -250,7 +251,7 @@ static int __init msi_wmi_input_setup(void)
        if (err)
                goto err_free_keymap;
 
-       memset(last_pressed, 0, sizeof(last_pressed));
+       last_pressed = ktime_set(0, 0);
 
        return 0;
 
@@ -281,6 +282,7 @@ static int __init msi_wmi_init(void)
 
                pr_debug("Event handler installed\n");
                event_wmi_guid = MSIWMI_EVENT_GUID;
+               quirk_last_pressed = true;
        }
 
        if (wmi_has_guid(MSIWMI_BIOS_GUID) && !acpi_video_backlight_support()) {