input: Adjust structure of code in process_modifier()
authorSimon Glass <sjg@chromium.org>
Wed, 11 Nov 2015 17:05:38 +0000 (10:05 -0700)
committerSimon Glass <sjg@chromium.org>
Fri, 20 Nov 2015 03:13:41 +0000 (20:13 -0700)
Move all the '!release' code into one block so that it is clear that it only
applies on key release.

Signed-off-by: Simon Glass <sjg@chromium.org>
Reviewed-by: Bin Meng <bmeng.cn@gmail.com>
drivers/input/input.c

index ceed5a6..a64bd87 100644 (file)
@@ -236,7 +236,6 @@ static struct input_key_xlate *process_modifier(struct input_config *config,
                                                int key, int release)
 {
        struct input_key_xlate *table;
-       int flip = -1;
        int i;
 
        /* Start with the main table, and see what modifiers change it */
@@ -251,6 +250,8 @@ static struct input_key_xlate *process_modifier(struct input_config *config,
 
        /* Handle the lighted keys */
        if (!release) {
+               int flip = -1;
+
                switch (key) {
                case KEY_SCROLLLOCK:
                        flip = FLAG_SCROLL_LOCK;
@@ -262,19 +263,19 @@ static struct input_key_xlate *process_modifier(struct input_config *config,
                        flip = FLAG_CAPS_LOCK;
                        break;
                }
-       }
 
-       if (flip != -1) {
-               int leds = 0;
-
-               config->leds ^= flip;
-               if (config->flags & FLAG_NUM_LOCK)
-                       leds |= INPUT_LED_NUM;
-               if (config->flags & FLAG_CAPS_LOCK)
-                       leds |= INPUT_LED_CAPS;
-               if (config->flags & FLAG_SCROLL_LOCK)
-                       leds |= INPUT_LED_SCROLL;
-               config->leds = leds;
+               if (flip != -1) {
+                       int leds = 0;
+
+                       config->leds ^= flip;
+                       if (config->flags & FLAG_NUM_LOCK)
+                               leds |= INPUT_LED_NUM;
+                       if (config->flags & FLAG_CAPS_LOCK)
+                               leds |= INPUT_LED_CAPS;
+                       if (config->flags & FLAG_SCROLL_LOCK)
+                               leds |= INPUT_LED_SCROLL;
+                       config->leds = leds;
+               }
        }
 
        return table;