[media] cx231xx: fix compiler warning

Fix this compiler warning by allocating a structure to read the eeprom instead
of doing it on the stack and worse: the eeprom array is static, so that can
cause problems if there are multiple cx231xx instances.

cx231xx-cards.c: In function 'cx231xx_card_setup':
cx231xx-cards.c:1110:1: warning: the frame size of 2064 bytes is larger than 2048 bytes [-Wframe-larger-than=]
 }
 ^

I did consider removing the code altogether since the result is actually
not used at the moment, but I decided against it since it is used in other
drivers and someone might want to start using it in this driver as well. And
then it is useful that the code is already there.

Signed-off-by: Hans Verkuil <hans.verkuil@cisco.com>
Signed-off-by: Mauro Carvalho Chehab <mchehab@osg.samsung.com>

diff --git a/drivers/media/usb/cx231xx/cx231xx-cards.c b/drivers/media/usb/cx231xx/cx231xx-cards.c
index fe00da105e77519bbf7c15684db8ae55593dac7e..a4aa2851339a7e07da45cc500a2d440d86139ce2 100644 (file)
--- a/drivers/media/usb/cx231xx/cx231xx-cards.c
+++ b/drivers/media/usb/cx231xx/cx231xx-cards.c
@@ -1092,17 +1092,25 @@ void cx231xx_card_setup(struct cx231xx *dev)
        case CX231XX_BOARD_HAUPPAUGE_930C_HD_1114xx:
        case CX231XX_BOARD_HAUPPAUGE_955Q:
                {
-                       struct tveeprom tvee;
-                       static u8 eeprom[256];
-                       struct i2c_client client;
-
-                       memset(&client, 0, sizeof(client));
-                       client.adapter = cx231xx_get_i2c_adap(dev, I2C_1_MUX_1);
-                       client.addr = 0xa0 >> 1;
+                       struct eeprom {
+                               struct tveeprom tvee;
+                               u8 eeprom[256];
+                               struct i2c_client client;
+                       };
+                       struct eeprom *e = kzalloc(sizeof(*e), GFP_KERNEL);
+
+                       if (e == NULL) {
+                               dev_err(dev->dev,
+                                       "failed to allocate memory to read eeprom\n");
+                               break;
+                       }
+                       e->client.adapter = cx231xx_get_i2c_adap(dev, I2C_1_MUX_1);
+                       e->client.addr = 0xa0 >> 1;
 
-                       read_eeprom(dev, &client, eeprom, sizeof(eeprom));
-                       tveeprom_hauppauge_analog(&client,
-                                               &tvee, eeprom + 0xc0);
+                       read_eeprom(dev, &e->client, e->eeprom, sizeof(e->eeprom));
+                       tveeprom_hauppauge_analog(&e->client,
+                                               &e->tvee, e->eeprom + 0xc0);
+                       kfree(e);
                        break;
                }
        }