media: au0828: make const array addr_list static
authorColin Ian King <colin.king@canonical.com>
Thu, 2 Nov 2017 17:26:48 +0000 (13:26 -0400)
committerMauro Carvalho Chehab <mchehab@s-opensource.com>
Tue, 7 Nov 2017 09:03:25 +0000 (04:03 -0500)
Don't populate array addr_list on the stack but instead make it
static. Makes the object code smaller by over 360 bytes:

Before:
   text    data     bss     dec     hex filename
   8036    1488     192    9716    25f4 au0828-input.o

After:
   text    data     bss     dec     hex filename
   7696    1488     192    9376    24a0 au0828-input.o

(gcc version 7.2.0 x86_64)

Signed-off-by: Colin Ian King <colin.king@canonical.com>
Signed-off-by: Mauro Carvalho Chehab <mchehab@s-opensource.com>
drivers/media/usb/au0828/au0828-input.c

index 7996eb83a54e7c014875cc140d14dace9a70ffb4..af68afe085b55a9b3d6cf4b89d9e9f6604ce587c 100644 (file)
@@ -269,7 +269,7 @@ static void au0828_rc_stop(struct rc_dev *rc)
 static int au0828_probe_i2c_ir(struct au0828_dev *dev)
 {
        int i = 0;
-       const unsigned short addr_list[] = {
+       static const unsigned short addr_list[] = {
                 0x47, I2C_CLIENT_END
        };