]> Pileus Git - ~andy/linux/commit
[media] saa7115: Don't use a dynamic array
authorMauro Carvalho Chehab <mchehab@redhat.com>
Tue, 21 May 2013 14:36:30 +0000 (11:36 -0300)
committerMauro Carvalho Chehab <mchehab@redhat.com>
Tue, 21 May 2013 15:04:16 +0000 (12:04 -0300)
commit6a084d6b3dc200b855ae8a3c6771abe285a3835d
treebb4c91ccd65039088c41d49997a83a3e895ccab2
parent2a848b2c7fcc8f5cb370a73ec9ca40d32c6cf18c
[media] saa7115: Don't use a dynamic array

At least on s390, gcc complains about that:
    drivers/media/i2c/saa7115.c: In function 'saa711x_detect_chip.constprop.2':
    drivers/media/i2c/saa7115.c:1647:1: warning: 'saa711x_detect_chip.constprop.2' uses dynamic stack allocation [enabled by default]

While for me the above report seems utterly bogus, as the
compiler should be optimizing saa711x_detect_chip, merging
it with saa711x_detect_chip and changing:
char chip_ver[size - 1];
to
char chip_ver[16];

because this function is only called on this code snippet:
char name[17];
...
ident = saa711x_detect_chip(client, id, name, sizeof(name));

It seems that gcc is not optimizing it, at least on s390.

As getting rid of it is easy, let's do it.

Reported-by: kbuild test robot <fengguang.wu@intel.com>
Acked-by: Hans Verkuil <hans.verkuil@cisco.com>
Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>
drivers/media/i2c/saa7115.c