Merge branch 'for-linus' of git://git.kernel.org/pub/scm/linux/kernel/git/dtor/input

* 'for-linus' of git://git.kernel.org/pub/scm/linux/kernel/git/dtor/input: (53 commits)
  Input: synaptics - fix reporting of min coordinates
  Input: tegra-kbc - enable key autorepeat
  Input: kxtj9 - fix locking typo in kxtj9_set_poll()
  Input: kxtj9 - fix bug in probe()
  Input: intel-mid-touch - remove pointless checking for variable 'found'
  Input: hp_sdc - staticize hp_sdc_kicker()
  Input: pmic8xxx-keypad - fix a leak of the IRQ during init failure
  Input: cy8ctmg110_ts - set reset_pin and irq_pin from platform data
  Input: cy8ctmg110_ts - constify i2c_device_id table
  Input: cy8ctmg110_ts - fix checking return value of i2c_master_send
  Input: lifebook - make dmi callback functions return 1
  Input: atkbd - make dmi callback functions return 1
  Input: gpio_keys - switch to using SIMPLE_DEV_PM_OPS
  Input: gpio_keys - add support for device-tree platform data
  Input: aiptek - remove double define
  Input: synaptics - set minimum coordinates as reported by firmware
  Input: synaptics - process button bits in AGM packets
  Input: synaptics - rename set_slot to be more descriptive
  Input: synaptics - fuzz position for touchpad with reduced filtering
  Input: synaptics - set resolution for MT_POSITION_X/Y axes
  ...

11 files changed, 164 insertions, 63 deletions

diff --git a/drivers/input/keyboard/adp5588-keys.c b/drivers/input/keyboard/adp5588-keys.c
index af45d275f686..7b404e5443ed 100644
--- a/drivers/input/keyboard/adp5588-keys.c
+++ b/drivers/input/keyboard/adp5588-keys.c
@@ -9,7 +9,6 @@
  */
 
 #include <linux/module.h>
-#include <linux/version.h>
 #include <linux/init.h>
 #include <linux/interrupt.h>
 #include <linux/irq.h>
diff --git a/drivers/input/keyboard/adp5589-keys.c b/drivers/input/keyboard/adp5589-keys.c
index 631598663aab..c7708263051b 100644
--- a/drivers/input/keyboard/adp5589-keys.c
+++ b/drivers/input/keyboard/adp5589-keys.c
@@ -8,7 +8,6 @@
  */
 
 #include <linux/module.h>
-#include <linux/version.h>
 #include <linux/init.h>
 #include <linux/interrupt.h>
 #include <linux/irq.h>
diff --git a/drivers/input/keyboard/atkbd.c b/drivers/input/keyboard/atkbd.c
index 11478eb2c27d..19cfc0cf558c 100644
--- a/drivers/input/keyboard/atkbd.c
+++ b/drivers/input/keyboard/atkbd.c
@@ -1578,14 +1578,14 @@ static int __init atkbd_setup_forced_release(const struct dmi_system_id *id)
         atkbd_platform_fixup = atkbd_apply_forced_release_keylist;
         atkbd_platform_fixup_data = id->driver_data;
 
-        return 0;
+        return 1;
 }
 
 static int __init atkbd_setup_scancode_fixup(const struct dmi_system_id *id)
 {
         atkbd_platform_scancode_fixup = id->driver_data;
 
-        return 0;
+        return 1;
 }
 
 static const struct dmi_system_id atkbd_dmi_quirk_table[] __initconst = {
diff --git a/drivers/input/keyboard/gpio_keys.c b/drivers/input/keyboard/gpio_keys.c
index 6e6145b9a4c1..ce281d152275 100644
--- a/drivers/input/keyboard/gpio_keys.c
+++ b/drivers/input/keyboard/gpio_keys.c
@@ -2,6 +2,7 @@
  * Driver for keys on GPIO lines capable of generating interrupts.
  *
  * Copyright 2005 Phil Blundell
+ * Copyright 2010, 2011 David Jander <david@protonic.nl>
  *
  * This program is free software; you can redistribute it and/or modify
  * it under the terms of the GNU General Public License version 2 as
@@ -25,6 +26,8 @@
 #include <linux/gpio_keys.h>
 #include <linux/workqueue.h>
 #include <linux/gpio.h>
+#include <linux/of_platform.h>
+#include <linux/of_gpio.h>
 
 struct gpio_button_data {
         struct gpio_keys_button *button;
@@ -415,7 +418,7 @@ static int __devinit gpio_keys_setup_key(struct platform_device *pdev,
         if (!button->can_disable)
                 irqflags |= IRQF_SHARED;
 
-        error = request_any_context_irq(irq, gpio_keys_isr, irqflags, desc, bdata);
+        error = request_threaded_irq(irq, NULL, gpio_keys_isr, irqflags, desc, bdata);
         if (error < 0) {
                 dev_err(dev, "Unable to claim irq %d; error %d\n",
                         irq, error);
@@ -445,15 +448,120 @@ static void gpio_keys_close(struct input_dev *input)
                 ddata->disable(input->dev.parent);
 }
 
+/*
+ * Handlers for alternative sources of platform_data
+ */
+#ifdef CONFIG_OF
+/*
+ * Translate OpenFirmware node properties into platform_data
+ */
+static int gpio_keys_get_devtree_pdata(struct device *dev,
+                            struct gpio_keys_platform_data *pdata)
+{
+        struct device_node *node, *pp;
+        int i;
+        struct gpio_keys_button *buttons;
+        const u32 *reg;
+        int len;
+
+        node = dev->of_node;
+        if (node == NULL)
+                return -ENODEV;
+
+        memset(pdata, 0, sizeof *pdata);
+
+        pdata->rep = !!of_get_property(node, "autorepeat", &len);
+
+        /* First count the subnodes */
+        pdata->nbuttons = 0;
+        pp = NULL;
+        while ((pp = of_get_next_child(node, pp)))
+                pdata->nbuttons++;
+
+        if (pdata->nbuttons == 0)
+                return -ENODEV;
+
+        buttons = kzalloc(pdata->nbuttons * (sizeof *buttons), GFP_KERNEL);
+        if (!buttons)
+                return -ENODEV;
+
+        pp = NULL;
+        i = 0;
+        while ((pp = of_get_next_child(node, pp))) {
+                enum of_gpio_flags flags;
+
+                if (!of_find_property(pp, "gpios", NULL)) {
+                        pdata->nbuttons--;
+                        dev_warn(dev, "Found button without gpios\n");
+                        continue;
+                }
+                buttons[i].gpio = of_get_gpio_flags(pp, 0, &flags);
+                buttons[i].active_low = flags & OF_GPIO_ACTIVE_LOW;
+
+                reg = of_get_property(pp, "linux,code", &len);
+                if (!reg) {
+                        dev_err(dev, "Button without keycode: 0x%x\n", buttons[i].gpio);
+                        goto out_fail;
+                }
+                buttons[i].code = be32_to_cpup(reg);
+
+                buttons[i].desc = of_get_property(pp, "label", &len);
+
+                reg = of_get_property(pp, "linux,input-type", &len);
+                buttons[i].type = reg ? be32_to_cpup(reg) : EV_KEY;
+
+                buttons[i].wakeup = !!of_get_property(pp, "gpio-key,wakeup", NULL);
+
+                reg = of_get_property(pp, "debounce-interval", &len);
+                buttons[i].debounce_interval = reg ? be32_to_cpup(reg) : 5;
+
+                i++;
+        }
+
+        pdata->buttons = buttons;
+
+        return 0;
+
+out_fail:
+        kfree(buttons);
+        return -ENODEV;
+}
+
+static struct of_device_id gpio_keys_of_match[] = {
+        { .compatible = "gpio-keys", },
+        { },
+};
+MODULE_DEVICE_TABLE(of, gpio_keys_of_match);
+
+#else
+
+static int gpio_keys_get_devtree_pdata(struct device *dev,
+                            struct gpio_keys_platform_data *altp)
+{
+        return -ENODEV;
+}
+
+#define gpio_keys_of_match NULL
+
+#endif
+
 static int __devinit gpio_keys_probe(struct platform_device *pdev)
 {
         struct gpio_keys_platform_data *pdata = pdev->dev.platform_data;
         struct gpio_keys_drvdata *ddata;
         struct device *dev = &pdev->dev;
+        struct gpio_keys_platform_data alt_pdata;
         struct input_dev *input;
         int i, error;
         int wakeup = 0;
 
+        if (!pdata) {
+                error = gpio_keys_get_devtree_pdata(dev, &alt_pdata);
+                if (error)
+                        return error;
+                pdata = &alt_pdata;
+        }
+
         ddata = kzalloc(sizeof(struct gpio_keys_drvdata) +
                         pdata->nbuttons * sizeof(struct gpio_button_data),
                         GFP_KERNEL);
@@ -544,13 +652,15 @@ static int __devinit gpio_keys_probe(struct platform_device *pdev)
  fail1:
         input_free_device(input);
         kfree(ddata);
+        /* If we have no platform_data, we allocated buttons dynamically. */
+        if (!pdev->dev.platform_data)
+                kfree(pdata->buttons);
 
         return error;
 }
 
 static int __devexit gpio_keys_remove(struct platform_device *pdev)
 {
-        struct gpio_keys_platform_data *pdata = pdev->dev.platform_data;
         struct gpio_keys_drvdata *ddata = platform_get_drvdata(pdev);
         struct input_dev *input = ddata->input;
         int i;
@@ -559,31 +669,39 @@ static int __devexit gpio_keys_remove(struct platform_device *pdev)
 
         device_init_wakeup(&pdev->dev, 0);
 
-        for (i = 0; i < pdata->nbuttons; i++) {
-                int irq = gpio_to_irq(pdata->buttons[i].gpio);
+        for (i = 0; i < ddata->n_buttons; i++) {
+                int irq = gpio_to_irq(ddata->data[i].button->gpio);
                 free_irq(irq, &ddata->data[i]);
                 if (ddata->data[i].timer_debounce)
                         del_timer_sync(&ddata->data[i].timer);
                 cancel_work_sync(&ddata->data[i].work);
-                gpio_free(pdata->buttons[i].gpio);
+                gpio_free(ddata->data[i].button->gpio);
         }
 
         input_unregister_device(input);
 
+        /*
+         * If we had no platform_data, we allocated buttons dynamically, and
+         * must free them here. ddata->data[0].button is the pointer to the
+         * beginning of the allocated array.
+         */
+        if (!pdev->dev.platform_data)
+                kfree(ddata->data[0].button);
+
+        kfree(ddata);
+
         return 0;
 }
 
-
-#ifdef CONFIG_PM
+#ifdef CONFIG_PM_SLEEP
 static int gpio_keys_suspend(struct device *dev)
 {
-        struct platform_device *pdev = to_platform_device(dev);
-        struct gpio_keys_platform_data *pdata = pdev->dev.platform_data;
+        struct gpio_keys_drvdata *ddata = dev_get_drvdata(dev);
         int i;
 
-        if (device_may_wakeup(&pdev->dev)) {
-                for (i = 0; i < pdata->nbuttons; i++) {
-                        struct gpio_keys_button *button = &pdata->buttons[i];
+        if (device_may_wakeup(dev)) {
+                for (i = 0; i < ddata->n_buttons; i++) {
+                        struct gpio_keys_button *button = ddata->data[i].button;
                         if (button->wakeup) {
                                 int irq = gpio_to_irq(button->gpio);
                                 enable_irq_wake(irq);
@@ -596,15 +714,13 @@ static int gpio_keys_suspend(struct device *dev)
 
 static int gpio_keys_resume(struct device *dev)
 {
-        struct platform_device *pdev = to_platform_device(dev);
-        struct gpio_keys_drvdata *ddata = platform_get_drvdata(pdev);
-        struct gpio_keys_platform_data *pdata = pdev->dev.platform_data;
+        struct gpio_keys_drvdata *ddata = dev_get_drvdata(dev);
         int i;
 
-        for (i = 0; i < pdata->nbuttons; i++) {
+        for (i = 0; i < ddata->n_buttons; i++) {
 
-                struct gpio_keys_button *button = &pdata->buttons[i];
-                if (button->wakeup && device_may_wakeup(&pdev->dev)) {
+                struct gpio_keys_button *button = ddata->data[i].button;
+                if (button->wakeup && device_may_wakeup(dev)) {
                         int irq = gpio_to_irq(button->gpio);
                         disable_irq_wake(irq);
                 }
@@ -615,22 +731,18 @@ static int gpio_keys_resume(struct device *dev)
 
         return 0;
 }
-
-static const struct dev_pm_ops gpio_keys_pm_ops = {
-        .suspend        = gpio_keys_suspend,
-        .resume         = gpio_keys_resume,
-};
 #endif
 
+static SIMPLE_DEV_PM_OPS(gpio_keys_pm_ops, gpio_keys_suspend, gpio_keys_resume);
+
 static struct platform_driver gpio_keys_device_driver = {
         .probe          = gpio_keys_probe,
         .remove         = __devexit_p(gpio_keys_remove),
         .driver         = {
                 .name   = "gpio-keys",
                 .owner  = THIS_MODULE,
-#ifdef CONFIG_PM
                 .pm     = &gpio_keys_pm_ops,
-#endif
+                .of_match_table = gpio_keys_of_match,
         }
 };
 
@@ -644,10 +756,10 @@ static void __exit gpio_keys_exit(void)
         platform_driver_unregister(&gpio_keys_device_driver);
 }
 
-module_init(gpio_keys_init);
+late_initcall(gpio_keys_init);
 module_exit(gpio_keys_exit);
 
 MODULE_LICENSE("GPL");
 MODULE_AUTHOR("Phil Blundell <pb@handhelds.org>");
-MODULE_DESCRIPTION("Keyboard driver for CPU GPIOs");
+MODULE_DESCRIPTION("Keyboard driver for GPIOs");
 MODULE_ALIAS("platform:gpio-keys");
diff --git a/drivers/input/keyboard/lm8323.c b/drivers/input/keyboard/lm8323.c
index 71f744a8e686..ab0acaf7fe8f 100644
--- a/drivers/input/keyboard/lm8323.c
+++ b/drivers/input/keyboard/lm8323.c
@@ -146,7 +146,6 @@ struct lm8323_chip {
         /* device lock */
         struct mutex            lock;
         struct i2c_client       *client;
-        struct work_struct      work;
         struct input_dev        *idev;
         bool                    kp_enabled;
         bool                    pm_suspend;
@@ -162,7 +161,6 @@ struct lm8323_chip {
 
 #define client_to_lm8323(c)     container_of(c, struct lm8323_chip, client)
 #define dev_to_lm8323(d)        container_of(d, struct lm8323_chip, client->dev)
-#define work_to_lm8323(w)       container_of(w, struct lm8323_chip, work)
 #define cdev_to_pwm(c)          container_of(c, struct lm8323_pwm, cdev)
 #define work_to_pwm(w)          container_of(w, struct lm8323_pwm, work)
 
@@ -375,9 +373,9 @@ static void pwm_done(struct lm8323_pwm *pwm)
  * Bottom half: handle the interrupt by posting key events, or dealing with
  * errors appropriately.
  */
-static void lm8323_work(struct work_struct *work)
+static irqreturn_t lm8323_irq(int irq, void *_lm)
 {
-        struct lm8323_chip *lm = work_to_lm8323(work);
+        struct lm8323_chip *lm = _lm;
         u8 ints;
         int i;
 
@@ -409,16 +407,6 @@ static void lm8323_work(struct work_struct *work)
         }
 
         mutex_unlock(&lm->lock);
-}
-
-/*
- * We cannot use I2C in interrupt context, so we just schedule work.
- */
-static irqreturn_t lm8323_irq(int irq, void *data)
-{
-        struct lm8323_chip *lm = data;
-
-        schedule_work(&lm->work);
 
         return IRQ_HANDLED;
 }
@@ -675,7 +663,6 @@ static int __devinit lm8323_probe(struct i2c_client *client,
         lm->client = client;
         lm->idev = idev;
         mutex_init(&lm->lock);
-        INIT_WORK(&lm->work, lm8323_work);
 
         lm->size_x = pdata->size_x;
         lm->size_y = pdata->size_y;
@@ -746,9 +733,8 @@ static int __devinit lm8323_probe(struct i2c_client *client,
                 goto fail3;
         }
 
-        err = request_irq(client->irq, lm8323_irq,
-                          IRQF_TRIGGER_FALLING | IRQF_DISABLED,
-                          "lm8323", lm);
+        err = request_threaded_irq(client->irq, NULL, lm8323_irq,
+                          IRQF_TRIGGER_LOW|IRQF_ONESHOT, "lm8323", lm);
         if (err) {
                 dev_err(&client->dev, "could not get IRQ %d\n", client->irq);
                 goto fail4;
@@ -783,7 +769,6 @@ static int __devexit lm8323_remove(struct i2c_client *client)
 
         disable_irq_wake(client->irq);
         free_irq(client->irq, lm);
-        cancel_work_sync(&lm->work);
 
         input_unregister_device(lm->idev);
 
diff --git a/drivers/input/keyboard/mpr121_touchkey.c b/drivers/input/keyboard/mpr121_touchkey.c
index 0a9e81194888..1c1615d9a7f9 100644
--- a/drivers/input/keyboard/mpr121_touchkey.c
+++ b/drivers/input/keyboard/mpr121_touchkey.c
@@ -43,14 +43,15 @@
  * enabled capacitance sensing inputs and its run/suspend mode.
  */
 #define ELECTRODE_CONF_ADDR             0x5e
+#define ELECTRODE_CONF_QUICK_CHARGE     0x80
 #define AUTO_CONFIG_CTRL_ADDR           0x7b
 #define AUTO_CONFIG_USL_ADDR            0x7d
 #define AUTO_CONFIG_LSL_ADDR            0x7e
 #define AUTO_CONFIG_TL_ADDR             0x7f
 
 /* Threshold of touch/release trigger */
-#define TOUCH_THRESHOLD                 0x0f
-#define RELEASE_THRESHOLD               0x0a
+#define TOUCH_THRESHOLD                 0x08
+#define RELEASE_THRESHOLD               0x05
 /* Masks for touch and release triggers */
 #define TOUCH_STATUS_MASK               0xfff
 /* MPR121 has 12 keys */
@@ -127,7 +128,7 @@ static int __devinit mpr121_phys_init(const struct mpr121_platform_data *pdata,
                                       struct i2c_client *client)
 {
         const struct mpr121_init_register *reg;
-        unsigned char usl, lsl, tl;
+        unsigned char usl, lsl, tl, eleconf;
         int i, t, vdd, ret;
 
         /* Set up touch/release threshold for ele0-ele11 */
@@ -163,8 +164,15 @@ static int __devinit mpr121_phys_init(const struct mpr121_platform_data *pdata,
         ret = i2c_smbus_write_byte_data(client, AUTO_CONFIG_USL_ADDR, usl);
         ret |= i2c_smbus_write_byte_data(client, AUTO_CONFIG_LSL_ADDR, lsl);
         ret |= i2c_smbus_write_byte_data(client, AUTO_CONFIG_TL_ADDR, tl);
+
+        /*
+         * Quick charge bit will let the capacitive charge to ready
+         * state quickly, or the buttons may not function after system
+         * boot.
+         */
+        eleconf = mpr121->keycount | ELECTRODE_CONF_QUICK_CHARGE;
         ret |= i2c_smbus_write_byte_data(client, ELECTRODE_CONF_ADDR,
-                                         mpr121->keycount);
+                                         eleconf);
         if (ret != 0)
                 goto err_i2c_write;
 
diff --git a/drivers/input/keyboard/pmic8xxx-keypad.c b/drivers/input/keyboard/pmic8xxx-keypad.c
index 6229c3e8e78b..e7cc51d0fb34 100644
--- a/drivers/input/keyboard/pmic8xxx-keypad.c
+++ b/drivers/input/keyboard/pmic8xxx-keypad.c
@@ -700,9 +700,9 @@ static int __devinit pmic8xxx_kp_probe(struct platform_device *pdev)
         return 0;
 
 err_pmic_reg_read:
-        free_irq(kp->key_stuck_irq, NULL);
+        free_irq(kp->key_stuck_irq, kp);
 err_req_stuck_irq:
-        free_irq(kp->key_sense_irq, NULL);
+        free_irq(kp->key_sense_irq, kp);
 err_gpio_config:
 err_get_irq:
         input_free_device(kp->input);
@@ -717,8 +717,8 @@ static int __devexit pmic8xxx_kp_remove(struct platform_device *pdev)
         struct pmic8xxx_kp *kp = platform_get_drvdata(pdev);
 
         device_init_wakeup(&pdev->dev, 0);
-        free_irq(kp->key_stuck_irq, NULL);
-        free_irq(kp->key_sense_irq, NULL);
+        free_irq(kp->key_stuck_irq, kp);
+        free_irq(kp->key_sense_irq, kp);
         input_unregister_device(kp->input);
         kfree(kp);
 
diff --git a/drivers/input/keyboard/qt1070.c b/drivers/input/keyboard/qt1070.c
index ca7b89196ab7..b21bf5b876bb 100644
--- a/drivers/input/keyboard/qt1070.c
+++ b/drivers/input/keyboard/qt1070.c
@@ -239,8 +239,6 @@ static int __devexit qt1070_remove(struct i2c_client *client)
         input_unregister_device(data->input);
         kfree(data);
 
-        i2c_set_clientdata(client, NULL);
-
         return 0;
 }
 
diff --git a/drivers/input/keyboard/sh_keysc.c b/drivers/input/keyboard/sh_keysc.c
index 6876700a4469..934aeb583b30 100644
--- a/drivers/input/keyboard/sh_keysc.c
+++ b/drivers/input/keyboard/sh_keysc.c
@@ -291,7 +291,7 @@ static int __devexit sh_keysc_remove(struct platform_device *pdev)
         return 0;
 }
 
-#if CONFIG_PM_SLEEP
+#ifdef CONFIG_PM_SLEEP
 static int sh_keysc_suspend(struct device *dev)
 {
         struct platform_device *pdev = to_platform_device(dev);
diff --git a/drivers/input/keyboard/tegra-kbc.c b/drivers/input/keyboard/tegra-kbc.c
index 2b3b73ec6689..da3828fc2c09 100644
--- a/drivers/input/keyboard/tegra-kbc.c
+++ b/drivers/input/keyboard/tegra-kbc.c
@@ -657,7 +657,7 @@ static int __devinit tegra_kbc_probe(struct platform_device *pdev)
 
         input_set_drvdata(input_dev, kbc);
 
-        input_dev->evbit[0] = BIT_MASK(EV_KEY);
+        input_dev->evbit[0] = BIT_MASK(EV_KEY) | BIT_MASK(EV_REP);
         input_set_capability(input_dev, EV_MSC, MSC_SCAN);
 
         input_dev->keycode = kbc->keycode;
diff --git a/drivers/input/keyboard/tnetv107x-keypad.c b/drivers/input/keyboard/tnetv107x-keypad.c
index c8f097a15d89..1c58681de81f 100644
--- a/drivers/input/keyboard/tnetv107x-keypad.c
+++ b/drivers/input/keyboard/tnetv107x-keypad.c
@@ -337,5 +337,5 @@ module_exit(keypad_exit);
 
 MODULE_AUTHOR("Cyril Chemparathy");
 MODULE_DESCRIPTION("TNETV107X Keypad Driver");
-MODULE_ALIAS("platform: tnetv107x-keypad");
+MODULE_ALIAS("platform:tnetv107x-keypad");
 MODULE_LICENSE("GPL");