gpio: pca953x.c: add interrupt handling capability

Most of the GPIO expanders controlled by the pca953x driver are able to
report changes on the input pins through an *INT pin.

This patch implements the irq_chip functionality (edge detection only).

The driver has been tested on an Arcom Zeus.

[akpm@linux-foundation.org: the compiler does inlining for us nowadays]
Signed-off-by: Marc Zyngier <maz@misterjones.org>
Cc: Eric Miao <eric.y.miao@gmail.com>
Cc: Haojian Zhuang <haojian.zhuang@gmail.com>
Cc: David Brownell <david-b@pacbell.net>
Cc: Nate Case <ncase@xes-inc.com>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>

3 files changed, 247 insertions, 12 deletions

diff --git a/drivers/gpio/Kconfig b/drivers/gpio/Kconfig
index f3549b8779d8..c5cc7d9d88e3 100644
--- a/drivers/gpio/Kconfig
+++ b/drivers/gpio/Kconfig
@@ -134,6 +134,13 @@ config GPIO_PCA953X
           This driver can also be built as a module.  If so, the module
           will be called pca953x.
 
+config GPIO_PCA953X_IRQ
+        bool "Interrupt controller support for PCA953x"
+        depends on GPIO_PCA953X=y
+        help
+          Say yes here to enable the pca953x to be used as an interrupt
+          controller. It requires the driver to be built in the kernel.
+
 config GPIO_PCF857X
         tristate "PCF857x, PCA{85,96}7x, and MAX732[89] I2C GPIO expanders"
         depends on I2C
diff --git a/drivers/gpio/pca953x.c b/drivers/gpio/pca953x.c
index 6a2fb3fbb3d9..ab5daab14bc2 100644
--- a/drivers/gpio/pca953x.c
+++ b/drivers/gpio/pca953x.c
@@ -14,6 +14,8 @@
 #include <linux/module.h>
 #include <linux/init.h>
 #include <linux/gpio.h>
+#include <linux/interrupt.h>
+#include <linux/irq.h>
 #include <linux/i2c.h>
 #include <linux/i2c/pca953x.h>
 #ifdef CONFIG_OF_GPIO
@@ -26,23 +28,28 @@
 #define PCA953X_INVERT         2
 #define PCA953X_DIRECTION      3
 
+#define PCA953X_GPIOS          0x00FF
+#define PCA953X_INT            0x0100
+
 static const struct i2c_device_id pca953x_id[] = {
-        { "pca9534", 8, },
-        { "pca9535", 16, },
+        { "pca9534", 8  | PCA953X_INT, },
+        { "pca9535", 16 | PCA953X_INT, },
         { "pca9536", 4, },
-        { "pca9537", 4, },
-        { "pca9538", 8, },
-        { "pca9539", 16, },
-        { "pca9554", 8, },
-        { "pca9555", 16, },
+        { "pca9537", 4  | PCA953X_INT, },
+        { "pca9538", 8  | PCA953X_INT, },
+        { "pca9539", 16 | PCA953X_INT, },
+        { "pca9554", 8  | PCA953X_INT, },
+        { "pca9555", 16 | PCA953X_INT, },
         { "pca9556", 8, },
         { "pca9557", 8, },
 
         { "max7310", 8, },
-        { "max7315", 8, },
-        { "pca6107", 8, },
-        { "tca6408", 8, },
-        { "tca6416", 16, },
+        { "max7312", 16 | PCA953X_INT, },
+        { "max7313", 16 | PCA953X_INT, },
+        { "max7315", 8  | PCA953X_INT, },
+        { "pca6107", 8  | PCA953X_INT, },
+        { "tca6408", 8  | PCA953X_INT, },
+        { "tca6416", 16 | PCA953X_INT, },
         /* NYET:  { "tca6424", 24, }, */
         { }
 };
@@ -53,6 +60,15 @@ struct pca953x_chip {
         uint16_t reg_output;
         uint16_t reg_direction;
 
+#ifdef CONFIG_GPIO_PCA953X_IRQ
+        struct mutex irq_lock;
+        uint16_t irq_mask;
+        uint16_t irq_stat;
+        uint16_t irq_trig_raise;
+        uint16_t irq_trig_fall;
+        int      irq_base;
+#endif
+
         struct i2c_client *client;
         struct pca953x_platform_data *dyn_pdata;
         struct gpio_chip gpio_chip;
@@ -202,6 +218,210 @@ static void pca953x_setup_gpio(struct pca953x_chip *chip, int gpios)
         gc->names = chip->names;
 }
 
+#ifdef CONFIG_GPIO_PCA953X_IRQ
+static int pca953x_gpio_to_irq(struct gpio_chip *gc, unsigned off)
+{
+        struct pca953x_chip *chip;
+
+        chip = container_of(gc, struct pca953x_chip, gpio_chip);
+        return chip->irq_base + off;
+}
+
+static void pca953x_irq_mask(unsigned int irq)
+{
+        struct pca953x_chip *chip = get_irq_chip_data(irq);
+
+        chip->irq_mask &= ~(1 << (irq - chip->irq_base));
+}
+
+static void pca953x_irq_unmask(unsigned int irq)
+{
+        struct pca953x_chip *chip = get_irq_chip_data(irq);
+
+        chip->irq_mask |= 1 << (irq - chip->irq_base);
+}
+
+static void pca953x_irq_bus_lock(unsigned int irq)
+{
+        struct pca953x_chip *chip = get_irq_chip_data(irq);
+
+        mutex_lock(&chip->irq_lock);
+}
+
+static void pca953x_irq_bus_sync_unlock(unsigned int irq)
+{
+        struct pca953x_chip *chip = get_irq_chip_data(irq);
+
+        mutex_unlock(&chip->irq_lock);
+}
+
+static int pca953x_irq_set_type(unsigned int irq, unsigned int type)
+{
+        struct pca953x_chip *chip = get_irq_chip_data(irq);
+        uint16_t level = irq - chip->irq_base;
+        uint16_t mask = 1 << level;
+
+        if (!(type & IRQ_TYPE_EDGE_BOTH)) {
+                dev_err(&chip->client->dev, "irq %d: unsupported type %d\n",
+                        irq, type);
+                return -EINVAL;
+        }
+
+        if (type & IRQ_TYPE_EDGE_FALLING)
+                chip->irq_trig_fall |= mask;
+        else
+                chip->irq_trig_fall &= ~mask;
+
+        if (type & IRQ_TYPE_EDGE_RISING)
+                chip->irq_trig_raise |= mask;
+        else
+                chip->irq_trig_raise &= ~mask;
+
+        return pca953x_gpio_direction_input(&chip->gpio_chip, level);
+}
+
+static struct irq_chip pca953x_irq_chip = {
+        .name                   = "pca953x",
+        .mask                   = pca953x_irq_mask,
+        .unmask                 = pca953x_irq_unmask,
+        .bus_lock               = pca953x_irq_bus_lock,
+        .bus_sync_unlock        = pca953x_irq_bus_sync_unlock,
+        .set_type               = pca953x_irq_set_type,
+};
+
+static uint16_t pca953x_irq_pending(struct pca953x_chip *chip)
+{
+        uint16_t cur_stat;
+        uint16_t old_stat;
+        uint16_t pending;
+        uint16_t trigger;
+        int ret;
+
+        ret = pca953x_read_reg(chip, PCA953X_INPUT, &cur_stat);
+        if (ret)
+                return 0;
+
+        /* Remove output pins from the equation */
+        cur_stat &= chip->reg_direction;
+
+        old_stat = chip->irq_stat;
+        trigger = (cur_stat ^ old_stat) & chip->irq_mask;
+
+        if (!trigger)
+                return 0;
+
+        chip->irq_stat = cur_stat;
+
+        pending = (old_stat & chip->irq_trig_fall) |
+                  (cur_stat & chip->irq_trig_raise);
+        pending &= trigger;
+
+        return pending;
+}
+