msi-wmi: replace one-condition switch-case with if statement

Signed-off-by: Anisse Astier <anisse@astier.eu>
Signed-off-by: Len Brown <len.brown@intel.com>

1 files changed, 4 insertions, 9 deletions

diff --git a/drivers/platform/x86/msi-wmi.c b/drivers/platform/x86/msi-wmi.c
index d7579359a2c5..2c2afc1828b8 100644
--- a/drivers/platform/x86/msi-wmi.c
+++ b/drivers/platform/x86/msi-wmi.c
@@ -243,14 +243,10 @@ static void msi_wmi_notify(u32 value, void *context)
                         }
                         key->last_pressed = cur;
 
-                        switch (key->type) {
-                        case KE_KEY:
-                                /* Brightness is served via acpi video driver */
-                                if (!backlight &&
-                                    (key->keycode == KEY_BRIGHTNESSUP ||
-                                     key->keycode == KEY_BRIGHTNESSDOWN))
-                                        break;
-
+                        if (key->type == KE_KEY &&
+                        /* Brightness is served via acpi video driver */
+                        (backlight || (key->keycode != KEY_BRIGHTNESSUP &&
+                        key->keycode != KEY_BRIGHTNESSDOWN))) {
                                 dprintk("Send key: 0x%X - "
                                         "Input layer keycode: %d\n", key->code,
                                          key->keycode);
@@ -260,7 +256,6 @@ static void msi_wmi_notify(u32 value, void *context)
                                 input_report_key(msi_wmi_input_dev,
                                                  key->keycode, 0);
                                 input_sync(msi_wmi_input_dev);
-                                break;
                         }
                 } else
                         printk(KERN_INFO "Unknown key pressed - %x\n",