OSDN Git Service

pinctrl: mcp23s08: drop comment about missing irq support
authorSebastian Reichel <sebastian.reichel@collabora.co.uk>
Mon, 15 May 2017 09:24:37 +0000 (11:24 +0200)
committerLinus Walleij <linus.walleij@linaro.org>
Tue, 23 May 2017 07:51:08 +0000 (09:51 +0200)
The driver supports using mcp23xxx as interrupt controller, so
let's drop all comments stating otherwise.

Signed-off-by: Sebastian Reichel <sebastian.reichel@collabora.co.uk>
Signed-off-by: Linus Walleij <linus.walleij@linaro.org>
drivers/pinctrl/pinctrl-mcp23s08.c
include/linux/spi/mcp23s08.h

index 29d9c1f..3e40d42 100644 (file)
@@ -1,14 +1,4 @@
-/*
- * MCP23S08 SPI/I2C GPIO gpio expander driver
- *
- * The inputs and outputs of the mcp23s08, mcp23s17, mcp23008 and mcp23017 are
- * supported.
- * For the I2C versions of the chips (mcp23008 and mcp23017) generation of
- * interrupts is also supported.
- * The hardware of the SPI versions of the chips (mcp23s08 and mcp23s17) is
- * also capable of generating interrupts, but the linux driver does not
- * support that yet.
- */
+/* MCP23S08 SPI/I2C GPIO driver */
 
 #include <linux/kernel.h>
 #include <linux/device.h>
@@ -27,7 +17,7 @@
 #include <linux/pinctrl/pinconf.h>
 #include <linux/pinctrl/pinconf-generic.h>
 
-/**
+/*
  * MCP types supported by driver
  */
 #define MCP_TYPE_S08   0
@@ -1131,11 +1121,6 @@ static int mcp23s08_probe(struct spi_device *spi)
        }
        data->ngpio = ngpio;
 
-       /* NOTE:  these chips have a relatively sane IRQ framework, with
-        * per-signal masking and level/edge triggering.  It's not yet
-        * handled here...
-        */
-
        return 0;
 }
 
index 211f3c0..4354bee 100644 (file)
@@ -1,6 +1,3 @@
-
-/* FIXME driver should be able to handle IRQs...  */
-
 struct mcp23s08_platform_data {
        /* For mcp23s08, up to 4 slaves (numbered 0..3) can share one SPI
         * chipselect, each providing 1 gpio_chip instance with 8 gpios.