]> www.pilppa.org Git - linux-2.6-omap-h63xx.git/commitdiff
Input: gpio-keys - simplify argument list for report_event
authorUwe Kleine-König <Uwe.Kleine-Koenig@digi.com>
Fri, 8 Aug 2008 16:14:36 +0000 (12:14 -0400)
committerDmitry Torokhov <dmitry.torokhov@gmail.com>
Fri, 8 Aug 2008 16:20:22 +0000 (12:20 -0400)
For now this only saves a few instructions (for gpio_keys_report_event,
gpio_keys_isr and gpio_check_button one instraction each on ARM using
arm-linux-gnu-gcc 4.2.3---I assume this is similar for other arch/compiler
combinations).

Signed-off-by: Uwe Kleine-König <Uwe.Kleine-Koenig@digi.com>
Signed-off-by: Dmitry Torokhov <dtor@mail.ru>
drivers/input/keyboard/gpio_keys.c

index e2809d29d99d9f9275579d215419ced74b8da0f9..fe22ca34d57640ea4998c4f1ce7806b2ccd41bac 100644 (file)
@@ -37,9 +37,10 @@ struct gpio_keys_drvdata {
        struct gpio_button_data data[0];
 };
 
-static void gpio_keys_report_event(struct gpio_keys_button *button,
-                                  struct input_dev *input)
+static void gpio_keys_report_event(struct gpio_button_data *bdata)
 {
+       struct gpio_keys_button *button = bdata->button;
+       struct input_dev *input = bdata->input;
        unsigned int type = button->type ?: EV_KEY;
        int state = (gpio_get_value(button->gpio) ? 1 : 0) ^ button->active_low;
 
@@ -51,7 +52,7 @@ static void gpio_check_button(unsigned long _data)
 {
        struct gpio_button_data *data = (struct gpio_button_data *)_data;
 
-       gpio_keys_report_event(data->button, data->input);
+       gpio_keys_report_event(data);
 }
 
 static irqreturn_t gpio_keys_isr(int irq, void *dev_id)
@@ -65,7 +66,7 @@ static irqreturn_t gpio_keys_isr(int irq, void *dev_id)
                mod_timer(&bdata->timer,
                        jiffies + msecs_to_jiffies(button->debounce_interval));
        else
-               gpio_keys_report_event(button, bdata->input);
+               gpio_keys_report_event(bdata);
 
        return IRQ_HANDLED;
 }