Input: adi - make array seq static, reduces object code size
authorColin Ian King <colin.king@canonical.com>
Mon, 4 Sep 2017 16:17:39 +0000 (09:17 -0700)
committerDmitry Torokhov <dmitry.torokhov@gmail.com>
Mon, 4 Sep 2017 16:28:15 +0000 (09:28 -0700)
Don't populate the array seq on the stack, instead make it static.
Makes the object code smaller by over 170 bytes:

Before:
   text    data     bss     dec     hex filename
  13227    3232       0   16459    404b drivers/input/joystick/adi.o

After:
   text    data     bss     dec     hex filename
  12957    3328       0   16285    3f9d drivers/input/joystick/adi.o

Signed-off-by: Colin Ian King <colin.king@canonical.com>
Signed-off-by: Dmitry Torokhov <dmitry.torokhov@gmail.com>
drivers/input/joystick/adi.c

index d09cefa379316a302df754394e342cc6fe4b088c..15a71acb6997265fe7ad261f289b544bc473e03e 100644 (file)
@@ -313,7 +313,7 @@ static void adi_close(struct input_dev *dev)
 
 static void adi_init_digital(struct gameport *gameport)
 {
-       int seq[] = { 4, -2, -3, 10, -6, -11, -7, -9, 11, 0 };
+       static const int seq[] = { 4, -2, -3, 10, -6, -11, -7, -9, 11, 0 };
        int i;
 
        for (i = 0; seq[i]; i++) {