Input: stmpe-keypad - remove VLA usage
authorGustavo A. R. Silva <gustavo@embeddedor.com>
Sat, 10 Mar 2018 18:16:41 +0000 (10:16 -0800)
committerDmitry Torokhov <dmitry.torokhov@gmail.com>
Sat, 10 Mar 2018 18:18:33 +0000 (10:18 -0800)
In preparation to enabling -Wvla, remove VLA and replace it with a
fixed-length array instead.

Fixed as part of the directive to remove all VLAs from the kernel:
https://lkml.org/lkml/2018/3/7/621

Signed-off-by: Gustavo A. R. Silva <gustavo@embeddedor.com>
Signed-off-by: Dmitry Torokhov <dmitry.torokhov@gmail.com>
drivers/input/keyboard/stmpe-keypad.c

index 8c6c0b9109c75e22bfdc290d4ecfacb88d911ca9..d69e631cfa0a337290a183789616680cbefbdaab 100644 (file)
 #define STMPE_KEYPAD_KEYMAP_MAX_SIZE \
        (STMPE_KEYPAD_MAX_ROWS * STMPE_KEYPAD_MAX_COLS)
 
+
+#define STMPE1601_NUM_DATA     5
+#define STMPE2401_NUM_DATA     3
+#define STMPE2403_NUM_DATA     5
+
+/* Make sure it covers all cases above */
+#define MAX_NUM_DATA           5
+
 /**
  * struct stmpe_keypad_variant - model-specific attributes
  * @auto_increment: whether the KPC_DATA_BYTE register address
@@ -74,7 +82,7 @@ struct stmpe_keypad_variant {
 static const struct stmpe_keypad_variant stmpe_keypad_variants[] = {
        [STMPE1601] = {
                .auto_increment         = true,
-               .num_data               = 5,
+               .num_data               = STMPE1601_NUM_DATA,
                .num_normal_data        = 3,
                .max_cols               = 8,
                .max_rows               = 8,
@@ -84,7 +92,7 @@ static const struct stmpe_keypad_variant stmpe_keypad_variants[] = {
        [STMPE2401] = {
                .auto_increment         = false,
                .set_pullup             = true,
-               .num_data               = 3,
+               .num_data               = STMPE2401_NUM_DATA,
                .num_normal_data        = 2,
                .max_cols               = 8,
                .max_rows               = 12,
@@ -94,7 +102,7 @@ static const struct stmpe_keypad_variant stmpe_keypad_variants[] = {
        [STMPE2403] = {
                .auto_increment         = true,
                .set_pullup             = true,
-               .num_data               = 5,
+               .num_data               = STMPE2403_NUM_DATA,
                .num_normal_data        = 3,
                .max_cols               = 8,
                .max_rows               = 12,
@@ -156,7 +164,7 @@ static irqreturn_t stmpe_keypad_irq(int irq, void *dev)
        struct stmpe_keypad *keypad = dev;
        struct input_dev *input = keypad->input;
        const struct stmpe_keypad_variant *variant = keypad->variant;
-       u8 fifo[variant->num_data];
+       u8 fifo[MAX_NUM_DATA];
        int ret;
        int i;