gpio: mcp23s08: move to pinctrl
authorSebastian Reichel <sebastian.reichel@collabora.co.uk>
Mon, 15 May 2017 09:24:25 +0000 (11:24 +0200)
committerLinus Walleij <linus.walleij@linaro.org>
Tue, 23 May 2017 07:45:29 +0000 (09:45 +0200)
This moves the mcp23s08 driver from gpio to pinctrl. Actual
pinctrl support for configuration of the pull-up resistors
follows in its own patch.

Signed-off-by: Sebastian Reichel <sebastian.reichel@collabora.co.uk>
Acked-by: Sylvain Lemieux <slemieux.tyco@gmail.com>
Signed-off-by: Linus Walleij <linus.walleij@linaro.org>
arch/arm/configs/lpc32xx_defconfig
arch/blackfin/configs/BF609-EZKIT_defconfig
arch/blackfin/mach-bf527/boards/tll6527m.c
arch/blackfin/mach-bf609/boards/ezkit.c
drivers/gpio/Kconfig
drivers/gpio/Makefile
drivers/gpio/gpio-mcp23s08.c [deleted file]
drivers/pinctrl/Kconfig
drivers/pinctrl/Makefile
drivers/pinctrl/pinctrl-mcp23s08.c [new file with mode: 0644]

index 6ba430d2b5b2572f21f688c92a7a0fc39c8825e7..e15fa5f168bba0a72a1a26daf7e1c24dd4527138 100644 (file)
@@ -112,7 +112,7 @@ CONFIG_GPIO_SX150X=y
 CONFIG_GPIO_74X164=y
 CONFIG_GPIO_MAX7301=y
 CONFIG_GPIO_MC33880=y
-CONFIG_GPIO_MCP23S08=y
+CONFIG_PINCTRL_MCP23S08=y
 CONFIG_SENSORS_DS620=y
 CONFIG_SENSORS_MAX6639=y
 CONFIG_WATCHDOG=y
index ba4267f658af4dd86abbd09232aa584442837a90..3ce77f07208a3580edf13443a4cc03c101591b0b 100644 (file)
@@ -105,7 +105,7 @@ CONFIG_SPI=y
 CONFIG_SPI_ADI_V3=y
 CONFIG_GPIOLIB=y
 CONFIG_GPIO_SYSFS=y
-CONFIG_GPIO_MCP23S08=y
+CONFIG_PINCTRL_MCP23S08=y
 # CONFIG_HWMON is not set
 CONFIG_WATCHDOG=y
 CONFIG_BFIN_WDT=y
index c1acce4c2e4569883ec28f32a256df706aa43649..be61477826f321aa15247369acd11d860a4817bc 100644 (file)
@@ -348,7 +348,7 @@ static struct platform_device bfin_i2s = {
 };
 #endif
 
-#if IS_ENABLED(CONFIG_GPIO_MCP23S08)
+#if IS_ENABLED(CONFIG_PINCTRL_MCP23S08)
 #include <linux/spi/mcp23s08.h>
 static const struct mcp23s08_platform_data bfin_mcp23s08_sys_gpio_info = {
        .chip[0].is_present = true,
@@ -423,7 +423,7 @@ static struct spi_board_info bfin_spi_board_info[] __initdata = {
                .mode = SPI_CPHA | SPI_CPOL,
        },
 #endif
-#if IS_ENABLED(CONFIG_GPIO_MCP23S08)
+#if IS_ENABLED(CONFIG_PINCTRL_MCP23S08)
        {
                .modalias = "mcp23s08",
                .platform_data = &bfin_mcp23s08_sys_gpio_info,
index 9231e5a72b93e7611d7a47721ed49c75adf423fe..51157a2558241dc54c1cc18e22c46d74e29969ef 100644 (file)
@@ -1887,7 +1887,7 @@ static struct platform_device i2c_bfin_twi1_device = {
 };
 #endif
 
-#if IS_ENABLED(CONFIG_GPIO_MCP23S08)
+#if IS_ENABLED(CONFIG_PINCTRL_MCP23S08)
 #include <linux/spi/mcp23s08.h>
 static const struct mcp23s08_platform_data bfin_mcp23s08_soft_switch0 = {
        .base = 120,
@@ -1929,7 +1929,7 @@ static struct i2c_board_info __initdata bfin_i2c_board_info0[] = {
                I2C_BOARD_INFO("ssm2602", 0x1b),
        },
 #endif
-#if IS_ENABLED(CONFIG_GPIO_MCP23S08)
+#if IS_ENABLED(CONFIG_PINCTRL_MCP23S08)
        {
                I2C_BOARD_INFO("mcp23017", 0x21),
                .platform_data = (void *)&bfin_mcp23s08_soft_switch0
index 23ca51ee6b28e416bc7472ca5525a71652bbde92..5f88d7324e02fbdf3028dcc4f1037788d8bd93e8 100644 (file)
@@ -1227,23 +1227,6 @@ config GPIO_PISOSR
 
 endmenu
 
-menu "SPI or I2C GPIO expanders"
-       depends on (SPI_MASTER && !I2C) || I2C
-
-config GPIO_MCP23S08
-       tristate "Microchip MCP23xxx I/O expander"
-       depends on OF_GPIO
-       select GPIOLIB_IRQCHIP
-       select REGMAP_I2C if I2C
-       select REGMAP if SPI_MASTER
-       help
-         SPI/I2C driver for Microchip MCP23S08/MCP23S17/MCP23008/MCP23017
-         I/O expanders.
-         This provides a GPIO interface supporting inputs and outputs.
-         The I2C versions of the chips can be used as interrupt-controller.
-
-endmenu
-
 menu "USB GPIO expanders"
        depends on USB
 
index 68b96277d9fa463bc641b0211eb4d719d75ac481..89f10061a5c15a3a5e70988f33c82b8f565b0f37 100644 (file)
@@ -77,7 +77,6 @@ obj-$(CONFIG_GPIO_MENZ127)    += gpio-menz127.o
 obj-$(CONFIG_GPIO_MERRIFIELD)  += gpio-merrifield.o
 obj-$(CONFIG_GPIO_MC33880)     += gpio-mc33880.o
 obj-$(CONFIG_GPIO_MC9S08DZ60)  += gpio-mc9s08dz60.o
-obj-$(CONFIG_GPIO_MCP23S08)    += gpio-mcp23s08.o
 obj-$(CONFIG_GPIO_ML_IOH)      += gpio-ml-ioh.o
 obj-$(CONFIG_GPIO_MM_LANTIQ)   += gpio-mm-lantiq.o
 obj-$(CONFIG_GPIO_MOCKUP)      += gpio-mockup.o
diff --git a/drivers/gpio/gpio-mcp23s08.c b/drivers/gpio/gpio-mcp23s08.c
deleted file mode 100644 (file)
index 2a57d02..0000000
+++ /dev/null
@@ -1,1047 +0,0 @@
-/*
- * 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.
- */
-
-#include <linux/kernel.h>
-#include <linux/device.h>
-#include <linux/mutex.h>
-#include <linux/module.h>
-#include <linux/gpio.h>
-#include <linux/i2c.h>
-#include <linux/spi/spi.h>
-#include <linux/spi/mcp23s08.h>
-#include <linux/slab.h>
-#include <asm/byteorder.h>
-#include <linux/interrupt.h>
-#include <linux/of_irq.h>
-#include <linux/of_device.h>
-#include <linux/regmap.h>
-
-/**
- * MCP types supported by driver
- */
-#define MCP_TYPE_S08   0
-#define MCP_TYPE_S17   1
-#define MCP_TYPE_008   2
-#define MCP_TYPE_017   3
-#define MCP_TYPE_S18    4
-
-/* Registers are all 8 bits wide.
- *
- * The mcp23s17 has twice as many bits, and can be configured to work
- * with either 16 bit registers or with two adjacent 8 bit banks.
- */
-#define MCP_IODIR      0x00            /* init/reset:  all ones */
-#define MCP_IPOL       0x01
-#define MCP_GPINTEN    0x02
-#define MCP_DEFVAL     0x03
-#define MCP_INTCON     0x04
-#define MCP_IOCON      0x05
-#      define IOCON_MIRROR     (1 << 6)
-#      define IOCON_SEQOP      (1 << 5)
-#      define IOCON_HAEN       (1 << 3)
-#      define IOCON_ODR        (1 << 2)
-#      define IOCON_INTPOL     (1 << 1)
-#      define IOCON_INTCC      (1)
-#define MCP_GPPU       0x06
-#define MCP_INTF       0x07
-#define MCP_INTCAP     0x08
-#define MCP_GPIO       0x09
-#define MCP_OLAT       0x0a
-
-struct mcp23s08;
-
-struct mcp23s08 {
-       u8                      addr;
-       bool                    irq_active_high;
-       bool                    reg_shift;
-
-       u16                     cache[11];
-       u16                     irq_rise;
-       u16                     irq_fall;
-       int                     irq;
-       bool                    irq_controller;
-       /* lock protects the cached values */
-       struct mutex            lock;
-       struct mutex            irq_lock;
-
-       struct gpio_chip        chip;
-
-       struct regmap           *regmap;
-       struct device           *dev;
-};
-
-static const struct regmap_config mcp23x08_regmap = {
-       .reg_bits = 8,
-       .val_bits = 8,
-
-       .reg_stride = 1,
-       .max_register = MCP_OLAT,
-};
-
-static const struct regmap_config mcp23x17_regmap = {
-       .reg_bits = 8,
-       .val_bits = 16,
-
-       .reg_stride = 2,
-       .max_register = MCP_OLAT << 1,
-       .val_format_endian = REGMAP_ENDIAN_LITTLE,
-};
-
-/*----------------------------------------------------------------------*/
-
-#ifdef CONFIG_SPI_MASTER
-
-static int mcp23sxx_spi_write(void *context, const void *data, size_t count)
-{
-       struct mcp23s08 *mcp = context;
-       struct spi_device *spi = to_spi_device(mcp->dev);
-       struct spi_message m;
-       struct spi_transfer t[2] = { { .tx_buf = &mcp->addr, .len = 1, },
-                                    { .tx_buf = data, .len = count, }, };
-
-       spi_message_init(&m);
-       spi_message_add_tail(&t[0], &m);
-       spi_message_add_tail(&t[1], &m);
-
-       return spi_sync(spi, &m);
-}
-
-static int mcp23sxx_spi_gather_write(void *context,
-                               const void *reg, size_t reg_size,
-                               const void *val, size_t val_size)
-{
-       struct mcp23s08 *mcp = context;
-       struct spi_device *spi = to_spi_device(mcp->dev);
-       struct spi_message m;
-       struct spi_transfer t[3] = { { .tx_buf = &mcp->addr, .len = 1, },
-                                    { .tx_buf = reg, .len = reg_size, },
-                                    { .tx_buf = val, .len = val_size, }, };
-
-       spi_message_init(&m);
-       spi_message_add_tail(&t[0], &m);
-       spi_message_add_tail(&t[1], &m);
-       spi_message_add_tail(&t[2], &m);
-
-       return spi_sync(spi, &m);
-}
-
-static int mcp23sxx_spi_read(void *context, const void *reg, size_t reg_size,
-                               void *val, size_t val_size)
-{
-       struct mcp23s08 *mcp = context;
-       struct spi_device *spi = to_spi_device(mcp->dev);
-       u8 tx[2];
-
-       if (reg_size != 1)
-               return -EINVAL;
-
-       tx[0] = mcp->addr | 0x01;
-       tx[1] = *((u8 *) reg);
-
-       return spi_write_then_read(spi, tx, sizeof(tx), val, val_size);
-}
-
-static const struct regmap_bus mcp23sxx_spi_regmap = {
-       .write = mcp23sxx_spi_write,
-       .gather_write = mcp23sxx_spi_gather_write,
-       .read = mcp23sxx_spi_read,
-};
-
-#endif /* CONFIG_SPI_MASTER */
-
-static int mcp_read(struct mcp23s08 *mcp, unsigned int reg, unsigned int *val)
-{
-       return regmap_read(mcp->regmap, reg << mcp->reg_shift, val);
-}
-
-static int mcp_write(struct mcp23s08 *mcp, unsigned int reg, unsigned int val)
-{
-       return regmap_write(mcp->regmap, reg << mcp->reg_shift, val);
-}
-
-static int mcp_update_cache(struct mcp23s08 *mcp)
-{
-       int ret, reg, i;
-
-       for (i = 0; i < ARRAY_SIZE(mcp->cache); i++) {
-               ret = mcp_read(mcp, i, &reg);
-               if (ret < 0)
-                       return ret;
-               mcp->cache[i] = reg;
-       }
-
-       return 0;
-}
-
-/*----------------------------------------------------------------------*/
-
-/* A given spi_device can represent up to eight mcp23sxx chips
- * sharing the same chipselect but using different addresses
- * (e.g. chips #0 and #3 might be populated, but not #1 or $2).
- * Driver data holds all the per-chip data.
- */
-struct mcp23s08_driver_data {
-       unsigned                ngpio;
-       struct mcp23s08         *mcp[8];
-       struct mcp23s08         chip[];
-};
-
-
-static int mcp23s08_direction_input(struct gpio_chip *chip, unsigned offset)
-{
-       struct mcp23s08 *mcp = gpiochip_get_data(chip);
-       int status;
-
-       mutex_lock(&mcp->lock);
-       mcp->cache[MCP_IODIR] |= (1 << offset);
-       status = mcp_write(mcp, MCP_IODIR, mcp->cache[MCP_IODIR]);
-       mutex_unlock(&mcp->lock);
-       return status;
-}
-
-static int mcp23s08_get(struct gpio_chip *chip, unsigned offset)
-{
-       struct mcp23s08 *mcp = gpiochip_get_data(chip);
-       int status, ret;
-
-       mutex_lock(&mcp->lock);
-
-       /* REVISIT reading this clears any IRQ ... */
-       ret = mcp_read(mcp, MCP_GPIO, &status);
-       if (ret < 0)
-               status = 0;
-       else {
-               mcp->cache[MCP_GPIO] = status;
-               status = !!(status & (1 << offset));
-       }
-       mutex_unlock(&mcp->lock);
-       return status;
-}
-
-static int __mcp23s08_set(struct mcp23s08 *mcp, unsigned mask, int value)
-{
-       unsigned olat = mcp->cache[MCP_OLAT];
-
-       if (value)
-               olat |= mask;
-       else
-               olat &= ~mask;
-       mcp->cache[MCP_OLAT] = olat;
-       return mcp_write(mcp, MCP_OLAT, olat);
-}
-
-static void mcp23s08_set(struct gpio_chip *chip, unsigned offset, int value)
-{
-       struct mcp23s08 *mcp = gpiochip_get_data(chip);
-       unsigned mask = 1 << offset;
-
-       mutex_lock(&mcp->lock);
-       __mcp23s08_set(mcp, mask, value);
-       mutex_unlock(&mcp->lock);
-}
-
-static int
-mcp23s08_direction_output(struct gpio_chip *chip, unsigned offset, int value)
-{
-       struct mcp23s08 *mcp = gpiochip_get_data(chip);
-       unsigned mask = 1 << offset;
-       int status;
-
-       mutex_lock(&mcp->lock);
-       status = __mcp23s08_set(mcp, mask, value);
-       if (status == 0) {
-               mcp->cache[MCP_IODIR] &= ~mask;
-               status = mcp_write(mcp, MCP_IODIR, mcp->cache[MCP_IODIR]);
-       }
-       mutex_unlock(&mcp->lock);
-       return status;
-}
-
-/*----------------------------------------------------------------------*/
-static irqreturn_t mcp23s08_irq(int irq, void *data)
-{
-       struct mcp23s08 *mcp = data;
-       int intcap, intf, i, gpio, gpio_orig, intcap_mask;
-       unsigned int child_irq;
-       bool intf_set, intcap_changed, gpio_bit_changed,
-               defval_changed, gpio_set;
-
-       mutex_lock(&mcp->lock);
-       if (mcp_read(mcp, MCP_INTF, &intf) < 0) {
-               mutex_unlock(&mcp->lock);
-               return IRQ_HANDLED;
-       }
-
-       mcp->cache[MCP_INTF] = intf;
-
-       if (mcp_read(mcp, MCP_INTCAP, &intcap) < 0) {
-               mutex_unlock(&mcp->lock);
-               return IRQ_HANDLED;
-       }
-
-       mcp->cache[MCP_INTCAP] = intcap;
-
-       /* This clears the interrupt(configurable on S18) */
-       if (mcp_read(mcp, MCP_GPIO, &gpio) < 0) {
-               mutex_unlock(&mcp->lock);
-               return IRQ_HANDLED;
-       }
-       gpio_orig = mcp->cache[MCP_GPIO];
-       mcp->cache[MCP_GPIO] = gpio;
-       mutex_unlock(&mcp->lock);
-
-       if (mcp->cache[MCP_INTF] == 0) {
-               /* There is no interrupt pending */
-               return IRQ_HANDLED;
-       }
-
-       dev_dbg(mcp->chip.parent,
-               "intcap 0x%04X intf 0x%04X gpio_orig 0x%04X gpio 0x%04X\n",
-               intcap, intf, gpio_orig, gpio);
-
-       for (i = 0; i < mcp->chip.ngpio; i++) {
-               /* We must check all of the inputs on the chip,
-                * otherwise we may not notice a change on >=2 pins.
-                *
-                * On at least the mcp23s17, INTCAP is only updated
-                * one byte at a time(INTCAPA and INTCAPB are
-                * not written to at the same time - only on a per-bank
-                * basis).
-                *
-                * INTF only contains the single bit that caused the
-                * interrupt per-bank.  On the mcp23s17, there is
-                * INTFA and INTFB.  If two pins are changed on the A
-                * side at the same time, INTF will only have one bit
-                * set.  If one pin on the A side and one pin on the B
-                * side are changed at the same time, INTF will have
-                * two bits set.  Thus, INTF can't be the only check
-                * to see if the input has changed.
-                */
-
-               intf_set = BIT(i) & mcp->cache[MCP_INTF];
-               if (i < 8 && intf_set)
-                       intcap_mask = 0x00FF;
-               else if (i >= 8 && intf_set)
-                       intcap_mask = 0xFF00;
-               else
-                       intcap_mask = 0x00;
-
-               intcap_changed = (intcap_mask &
-                       (BIT(i) & mcp->cache[MCP_INTCAP])) !=
-                       (intcap_mask & (BIT(i) & gpio_orig));
-               gpio_set = BIT(i) & mcp->cache[MCP_GPIO];
-               gpio_bit_changed = (BIT(i) & gpio_orig) !=
-                       (BIT(i) & mcp->cache[MCP_GPIO]);
-               defval_changed = (BIT(i) & mcp->cache[MCP_INTCON]) &&
-                       ((BIT(i) & mcp->cache[MCP_GPIO]) !=
-                       (BIT(i) & mcp->cache[MCP_DEFVAL]));
-
-               if (((gpio_bit_changed || intcap_changed) &&
-                       (BIT(i) & mcp->irq_rise) && gpio_set) ||
-                   ((gpio_bit_changed || intcap_changed) &&
-                       (BIT(i) & mcp->irq_fall) && !gpio_set) ||
-                   defval_changed) {
-                       child_irq = irq_find_mapping(mcp->chip.irqdomain, i);
-                       handle_nested_irq(child_irq);
-               }
-       }
-
-       return IRQ_HANDLED;
-}
-
-static void mcp23s08_irq_mask(struct irq_data *data)
-{
-       struct gpio_chip *gc = irq_data_get_irq_chip_data(data);
-       struct mcp23s08 *mcp = gpiochip_get_data(gc);
-       unsigned int pos = data->hwirq;
-
-       mcp->cache[MCP_GPINTEN] &= ~BIT(pos);
-}
-
-static void mcp23s08_irq_unmask(struct irq_data *data)
-{
-       struct gpio_chip *gc = irq_data_get_irq_chip_data(data);
-       struct mcp23s08 *mcp = gpiochip_get_data(gc);
-       unsigned int pos = data->hwirq;
-
-       mcp->cache[MCP_GPINTEN] |= BIT(pos);
-}
-
-static int mcp23s08_irq_set_type(struct irq_data *data, unsigned int type)
-{
-       struct gpio_chip *gc = irq_data_get_irq_chip_data(data);
-       struct mcp23s08 *mcp = gpiochip_get_data(gc);
-       unsigned int pos = data->hwirq;
-       int status = 0;
-
-       if ((type & IRQ_TYPE_EDGE_BOTH) == IRQ_TYPE_EDGE_BOTH) {
-               mcp->cache[MCP_INTCON] &= ~BIT(pos);
-               mcp->irq_rise |= BIT(pos);
-               mcp->irq_fall |= BIT(pos);
-       } else if (type & IRQ_TYPE_EDGE_RISING) {
-               mcp->cache[MCP_INTCON] &= ~BIT(pos);
-               mcp->irq_rise |= BIT(pos);
-               mcp->irq_fall &= ~BIT(pos);
-       } else if (type & IRQ_TYPE_EDGE_FALLING) {
-               mcp->cache[MCP_INTCON] &= ~BIT(pos);
-               mcp->irq_rise &= ~BIT(pos);
-               mcp->irq_fall |= BIT(pos);
-       } else if (type & IRQ_TYPE_LEVEL_HIGH) {
-               mcp->cache[MCP_INTCON] |= BIT(pos);
-               mcp->cache[MCP_DEFVAL] &= ~BIT(pos);
-       } else if (type & IRQ_TYPE_LEVEL_LOW) {
-               mcp->cache[MCP_INTCON] |= BIT(pos);
-               mcp->cache[MCP_DEFVAL] |= BIT(pos);
-       } else
-               return -EINVAL;
-
-       return status;
-}
-
-static void mcp23s08_irq_bus_lock(struct irq_data *data)
-{
-       struct gpio_chip *gc = irq_data_get_irq_chip_data(data);
-       struct mcp23s08 *mcp = gpiochip_get_data(gc);
-
-       mutex_lock(&mcp->irq_lock);
-}
-
-static void mcp23s08_irq_bus_unlock(struct irq_data *data)
-{
-       struct gpio_chip *gc = irq_data_get_irq_chip_data(data);
-       struct mcp23s08 *mcp = gpiochip_get_data(gc);
-
-       mutex_lock(&mcp->lock);
-       mcp_write(mcp, MCP_GPINTEN, mcp->cache[MCP_GPINTEN]);
-       mcp_write(mcp, MCP_DEFVAL, mcp->cache[MCP_DEFVAL]);
-       mcp_write(mcp, MCP_INTCON, mcp->cache[MCP_INTCON]);
-       mutex_unlock(&mcp->lock);
-       mutex_unlock(&mcp->irq_lock);
-}
-
-static struct irq_chip mcp23s08_irq_chip = {
-       .name = "gpio-mcp23xxx",
-       .irq_mask = mcp23s08_irq_mask,
-       .irq_unmask = mcp23s08_irq_unmask,
-       .irq_set_type = mcp23s08_irq_set_type,
-       .irq_bus_lock = mcp23s08_irq_bus_lock,
-       .irq_bus_sync_unlock = mcp23s08_irq_bus_unlock,
-};
-
-static int mcp23s08_irq_setup(struct mcp23s08 *mcp)
-{
-       struct gpio_chip *chip = &mcp->chip;
-       int err;
-       unsigned long irqflags = IRQF_ONESHOT | IRQF_SHARED;
-
-       mutex_init(&mcp->irq_lock);
-
-       if (mcp->irq_active_high)
-               irqflags |= IRQF_TRIGGER_HIGH;
-       else
-               irqflags |= IRQF_TRIGGER_LOW;
-
-       err = devm_request_threaded_irq(chip->parent, mcp->irq, NULL,
-                                       mcp23s08_irq,
-                                       irqflags, dev_name(chip->parent), mcp);
-       if (err != 0) {
-               dev_err(chip->parent, "unable to request IRQ#%d: %d\n",
-                       mcp->irq, err);
-               return err;
-       }
-
-       err =  gpiochip_irqchip_add_nested(chip,
-                                          &mcp23s08_irq_chip,
-                                          0,
-                                          handle_simple_irq,
-                                          IRQ_TYPE_NONE);
-       if (err) {
-               dev_err(chip->parent,
-                       "could not connect irqchip to gpiochip: %d\n", err);
-               return err;
-       }
-
-       gpiochip_set_nested_irqchip(chip,
-                                   &mcp23s08_irq_chip,
-                                   mcp->irq);
-
-       return 0;
-}
-
-/*----------------------------------------------------------------------*/
-
-#ifdef CONFIG_DEBUG_FS
-
-#include <linux/seq_file.h>
-
-/*
- * This shows more info than the generic gpio dump code:
- * pullups, deglitching, open drain drive.
- */
-static void mcp23s08_dbg_show(struct seq_file *s, struct gpio_chip *chip)
-{
-       struct mcp23s08 *mcp;
-       char            bank;
-       int             t;
-       unsigned        mask;
-
-       mcp = gpiochip_get_data(chip);
-
-       /* NOTE: we only handle one bank for now ... */
-       bank = '0' + ((mcp->addr >> 1) & 0x7);
-
-       mutex_lock(&mcp->lock);
-       t = mcp_update_cache(mcp);
-       if (t < 0) {
-               seq_printf(s, " I/O ERROR %d\n", t);
-               goto done;
-       }
-
-       for (t = 0, mask = 1; t < chip->ngpio; t++, mask <<= 1) {
-               const char      *label;
-
-               label = gpiochip_is_requested(chip, t);
-               if (!label)
-                       continue;
-
-               seq_printf(s, " gpio-%-3d P%c.%d (%-12s) %s %s %s",
-                       chip->base + t, bank, t, label,
-                       (mcp->cache[MCP_IODIR] & mask) ? "in " : "out",
-                       (mcp->cache[MCP_GPIO] & mask) ? "hi" : "lo",
-                       (mcp->cache[MCP_GPPU] & mask) ? "up" : "  ");
-               /* NOTE:  ignoring the irq-related registers */
-               seq_puts(s, "\n");
-       }
-done:
-       mutex_unlock(&mcp->lock);
-}
-
-#else
-#define mcp23s08_dbg_show      NULL
-#endif
-
-/*----------------------------------------------------------------------*/
-
-static int mcp23s08_probe_one(struct mcp23s08 *mcp, struct device *dev,
-                             void *data, unsigned addr, unsigned type,
-                             struct mcp23s08_platform_data *pdata, int cs)
-{
-       int status, ret;
-       bool mirror = false;
-
-       mutex_init(&mcp->lock);
-
-       mcp->dev = dev;
-       mcp->addr = addr;
-       mcp->irq_active_high = false;
-
-       mcp->chip.direction_input = mcp23s08_direction_input;
-       mcp->chip.get = mcp23s08_get;
-       mcp->chip.direction_output = mcp23s08_direction_output;
-       mcp->chip.set = mcp23s08_set;
-       mcp->chip.dbg_show = mcp23s08_dbg_show;
-#ifdef CONFIG_OF_GPIO
-       mcp->chip.of_gpio_n_cells = 2;
-       mcp->chip.of_node = dev->of_node;
-#endif
-
-       switch (type) {
-#ifdef CONFIG_SPI_MASTER
-       case MCP_TYPE_S08:
-               mcp->regmap = devm_regmap_init(dev, &mcp23sxx_spi_regmap, mcp,
-                                              &mcp23x08_regmap);
-               mcp->reg_shift = 0;
-               mcp->chip.ngpio = 8;
-               mcp->chip.label = "mcp23s08";
-               break;
-
-       case MCP_TYPE_S17:
-               mcp->regmap = devm_regmap_init(dev, &mcp23sxx_spi_regmap, mcp,
-                                              &mcp23x17_regmap);
-               mcp->reg_shift = 1;
-               mcp->chip.ngpio = 16;
-               mcp->chip.label = "mcp23s17";
-               break;
-
-       case MCP_TYPE_S18:
-               mcp->regmap = devm_regmap_init(dev, &mcp23sxx_spi_regmap, mcp,
-                                              &mcp23x17_regmap);
-               mcp->reg_shift = 1;
-               mcp->chip.ngpio = 16;
-               mcp->chip.label = "mcp23s18";
-               break;
-#endif /* CONFIG_SPI_MASTER */
-
-#if IS_ENABLED(CONFIG_I2C)
-       case MCP_TYPE_008:
-               mcp->regmap = devm_regmap_init_i2c(data, &mcp23x08_regmap);
-               mcp->reg_shift = 0;
-               mcp->chip.ngpio = 8;
-               mcp->chip.label = "mcp23008";
-               break;
-
-       case MCP_TYPE_017:
-               mcp->regmap = devm_regmap_init_i2c(data, &mcp23x17_regmap);
-               mcp->reg_shift = 1;
-               mcp->chip.ngpio = 16;
-               mcp->chip.label = "mcp23017";
-               break;
-#endif /* CONFIG_I2C */
-
-       default:
-               dev_err(dev, "invalid device type (%d)\n", type);
-               return -EINVAL;
-       }
-
-       if (IS_ERR(mcp->regmap))
-               return PTR_ERR(mcp->regmap);
-
-       mcp->chip.base = pdata->base;
-       mcp->chip.can_sleep = true;
-       mcp->chip.parent = dev;
-       mcp->chip.owner = THIS_MODULE;
-
-       /* verify MCP_IOCON.SEQOP = 0, so sequential reads work,
-        * and MCP_IOCON.HAEN = 1, so we work with all chips.
-        */
-
-       ret = mcp_read(mcp, MCP_IOCON, &status);
-       if (ret < 0)
-               goto fail;
-
-       mcp->irq_controller = pdata->irq_controller;
-       if (mcp->irq && mcp->irq_controller) {
-               mcp->irq_active_high =
-                       of_property_read_bool(mcp->chip.parent->of_node,
-                                             "microchip,irq-active-high");
-
-               mirror = pdata->mirror;
-       }
-
-       if ((status & IOCON_SEQOP) || !(status & IOCON_HAEN) || mirror ||
-            mcp->irq_active_high) {
-               /* mcp23s17 has IOCON twice, make sure they are in sync */
-               status &= ~(IOCON_SEQOP | (IOCON_SEQOP << 8));
-               status |= IOCON_HAEN | (IOCON_HAEN << 8);
-               if (mcp->irq_active_high)
-                       status |= IOCON_INTPOL | (IOCON_INTPOL << 8);
-               else
-                       status &= ~(IOCON_INTPOL | (IOCON_INTPOL << 8));
-
-               if (mirror)
-                       status |= IOCON_MIRROR | (IOCON_MIRROR << 8);
-
-               if (type == MCP_TYPE_S18)
-                       status |= IOCON_INTCC | (IOCON_INTCC << 8);
-
-               ret = mcp_write(mcp, MCP_IOCON, status);
-               if (ret < 0)
-                       goto fail;
-       }
-
-       /* configure ~100K pullups */
-       ret = mcp_write(mcp, MCP_GPPU, pdata->chip[cs].pullups);
-       if (ret < 0)
-               goto fail;
-
-       ret = mcp_update_cache(mcp);
-       if (ret < 0)
-               goto fail;
-
-       /* disable inverter on input */
-       if (mcp->cache[MCP_IPOL] != 0) {
-               mcp->cache[MCP_IPOL] = 0;
-               ret = mcp_write(mcp, MCP_IPOL, 0);
-               if (ret < 0)
-                       goto fail;
-       }
-
-       /* disable irqs */
-       if (mcp->cache[MCP_GPINTEN] != 0) {
-               mcp->cache[MCP_GPINTEN] = 0;
-               ret = mcp_write(mcp, MCP_GPINTEN, 0);
-               if (ret < 0)
-                       goto fail;
-       }
-
-       ret = gpiochip_add_data(&mcp->chip, mcp);
-       if (ret < 0)
-               goto fail;
-
-       if (mcp->irq && mcp->irq_controller) {
-               ret = mcp23s08_irq_setup(mcp);
-               if (ret)
-                       goto fail;
-       }
-fail:
-       if (ret < 0)
-               dev_dbg(dev, "can't setup chip %d, --> %d\n", addr, ret);
-       return ret;
-}
-
-/*----------------------------------------------------------------------*/
-
-#ifdef CONFIG_OF
-#ifdef CONFIG_SPI_MASTER
-static const struct of_device_id mcp23s08_spi_of_match[] = {
-       {
-               .compatible = "microchip,mcp23s08",
-               .data = (void *) MCP_TYPE_S08,
-       },
-       {
-               .compatible = "microchip,mcp23s17",
-               .data = (void *) MCP_TYPE_S17,
-       },
-       {
-               .compatible = "microchip,mcp23s18",
-               .data = (void *) MCP_TYPE_S18,
-       },
-/* NOTE: The use of the mcp prefix is deprecated and will be removed. */
-       {
-               .compatible = "mcp,mcp23s08",
-               .data = (void *) MCP_TYPE_S08,
-       },
-       {
-               .compatible = "mcp,mcp23s17",
-               .data = (void *) MCP_TYPE_S17,
-       },
-       { },
-};
-MODULE_DEVICE_TABLE(of, mcp23s08_spi_of_match);
-#endif
-
-#if IS_ENABLED(CONFIG_I2C)
-static const struct of_device_id mcp23s08_i2c_of_match[] = {
-       {
-               .compatible = "microchip,mcp23008",
-               .data = (void *) MCP_TYPE_008,
-       },
-       {
-               .compatible = "microchip,mcp23017",
-               .data = (void *) MCP_TYPE_017,
-       },
-/* NOTE: The use of the mcp prefix is deprecated and will be removed. */
-       {
-               .compatible = "mcp,mcp23008",
-               .data = (void *) MCP_TYPE_008,
-       },
-       {
-               .compatible = "mcp,mcp23017",
-               .data = (void *) MCP_TYPE_017,
-       },
-       { },
-};
-MODULE_DEVICE_TABLE(of, mcp23s08_i2c_of_match);
-#endif
-#endif /* CONFIG_OF */
-
-
-#if IS_ENABLED(CONFIG_I2C)
-
-static int mcp230xx_probe(struct i2c_client *client,
-                                   const struct i2c_device_id *id)
-{
-       struct mcp23s08_platform_data *pdata, local_pdata;
-       struct mcp23s08 *mcp;
-       int status;
-       const struct of_device_id *match;
-
-       match = of_match_device(of_match_ptr(mcp23s08_i2c_of_match),
-                                       &client->dev);
-       if (match) {
-               pdata = &local_pdata;
-               pdata->base = -1;
-               pdata->chip[0].pullups = 0;
-               pdata->irq_controller = of_property_read_bool(
-                                       client->dev.of_node,
-                                       "interrupt-controller");
-               pdata->mirror = of_property_read_bool(client->dev.of_node,
-                                                     "microchip,irq-mirror");
-               client->irq = irq_of_parse_and_map(client->dev.of_node, 0);
-       } else {
-               pdata = dev_get_platdata(&client->dev);
-               if (!pdata) {
-                       pdata = devm_kzalloc(&client->dev,
-                                       sizeof(struct mcp23s08_platform_data),
-                                       GFP_KERNEL);
-                       if (!pdata)
-                               return -ENOMEM;
-                       pdata->base = -1;
-               }
-       }
-
-       mcp = kzalloc(sizeof(*mcp), GFP_KERNEL);
-       if (!mcp)
-               return -ENOMEM;
-
-       mcp->irq = client->irq;
-       status = mcp23s08_probe_one(mcp, &client->dev, client, client->addr,
-                                   id->driver_data, pdata, 0);
-       if (status)
-               goto fail;
-
-       i2c_set_clientdata(client, mcp);
-
-       return 0;
-
-fail:
-       kfree(mcp);
-
-       return status;
-}
-
-static int mcp230xx_remove(struct i2c_client *client)
-{
-       struct mcp23s08 *mcp = i2c_get_clientdata(client);
-
-       gpiochip_remove(&mcp->chip);
-       kfree(mcp);
-
-       return 0;
-}
-
-static const struct i2c_device_id mcp230xx_id[] = {
-       { "mcp23008", MCP_TYPE_008 },
-       { "mcp23017", MCP_TYPE_017 },
-       { },
-};
-MODULE_DEVICE_TABLE(i2c, mcp230xx_id);
-
-static struct i2c_driver mcp230xx_driver = {
-       .driver = {
-               .name   = "mcp230xx",
-               .of_match_table = of_match_ptr(mcp23s08_i2c_of_match),
-       },
-       .probe          = mcp230xx_probe,
-       .remove         = mcp230xx_remove,
-       .id_table       = mcp230xx_id,
-};
-
-static int __init mcp23s08_i2c_init(void)
-{
-       return i2c_add_driver(&mcp230xx_driver);
-}
-
-static void mcp23s08_i2c_exit(void)
-{
-       i2c_del_driver(&mcp230xx_driver);
-}
-
-#else
-
-static int __init mcp23s08_i2c_init(void) { return 0; }
-static void mcp23s08_i2c_exit(void) { }
-
-#endif /* CONFIG_I2C */
-
-/*----------------------------------------------------------------------*/
-
-#ifdef CONFIG_SPI_MASTER
-
-static int mcp23s08_probe(struct spi_device *spi)
-{
-       struct mcp23s08_platform_data   *pdata, local_pdata;
-       unsigned                        addr;
-       int                             chips = 0;
-       struct mcp23s08_driver_data     *data;
-       int                             status, type;
-       unsigned                        ngpio = 0;
-       const struct                    of_device_id *match;
-       u32                             spi_present_mask = 0;
-
-       match = of_match_device(of_match_ptr(mcp23s08_spi_of_match), &spi->dev);
-       if (match) {
-               type = (int)(uintptr_t)match->data;
-               status = of_property_read_u32(spi->dev.of_node,
-                           "microchip,spi-present-mask", &spi_present_mask);
-               if (status) {
-                       status = of_property_read_u32(spi->dev.of_node,
-                                   "mcp,spi-present-mask", &spi_present_mask);
-                       if (status) {
-                               dev_err(&spi->dev,
-                                       "DT has no spi-present-mask\n");
-                               return -ENODEV;
-                       }
-               }
-               if ((spi_present_mask <= 0) || (spi_present_mask >= 256)) {
-                       dev_err(&spi->dev, "invalid spi-present-mask\n");
-                       return -ENODEV;
-               }
-
-               pdata = &local_pdata;
-               pdata->base = -1;
-               for (addr = 0; addr < ARRAY_SIZE(pdata->chip); addr++) {
-                       pdata->chip[addr].pullups = 0;
-                       if (spi_present_mask & (1 << addr))
-                               chips++;
-               }
-               pdata->irq_controller = of_property_read_bool(
-                                       spi->dev.of_node,
-                                       "interrupt-controller");
-               pdata->mirror = of_property_read_bool(spi->dev.of_node,
-                                                     "microchip,irq-mirror");
-       } else {
-               type = spi_get_device_id(spi)->driver_data;
-               pdata = dev_get_platdata(&spi->dev);
-               if (!pdata) {
-                       pdata = devm_kzalloc(&spi->dev,
-                                       sizeof(struct mcp23s08_platform_data),
-                                       GFP_KERNEL);
-                       pdata->base = -1;
-               }
-
-               for (addr = 0; addr < ARRAY_SIZE(pdata->chip); addr++) {
-                       if (!pdata->chip[addr].is_present)
-                               continue;
-                       chips++;
-                       if ((type == MCP_TYPE_S08) && (addr > 3)) {
-                               dev_err(&spi->dev,
-                                       "mcp23s08 only supports address 0..3\n");
-                               return -EINVAL;
-                       }
-                       spi_present_mask |= 1 << addr;
-               }
-       }
-
-       if (!chips)
-               return -ENODEV;
-
-       data = devm_kzalloc(&spi->dev,
-                           sizeof(*data) + chips * sizeof(struct mcp23s08),
-                           GFP_KERNEL);
-       if (!data)
-               return -ENOMEM;
-
-       spi_set_drvdata(spi, data);
-
-       spi->irq = irq_of_parse_and_map(spi->dev.of_node, 0);
-
-       for (addr = 0; addr < ARRAY_SIZE(pdata->chip); addr++) {
-               if (!(spi_present_mask & (1 << addr)))
-                       continue;
-               chips--;
-               data->mcp[addr] = &data->chip[chips];
-               data->mcp[addr]->irq = spi->irq;
-               status = mcp23s08_probe_one(data->mcp[addr], &spi->dev, spi,
-                                           0x40 | (addr << 1), type, pdata,
-                                           addr);
-               if (status < 0)
-                       goto fail;
-
-               if (pdata->base != -1)
-                       pdata->base += data->mcp[addr]->chip.ngpio;
-               ngpio += data->mcp[addr]->chip.ngpio;
-       }
-       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;
-
-fail:
-       for (addr = 0; addr < ARRAY_SIZE(data->mcp); addr++) {
-
-               if (!data->mcp[addr])
-                       continue;
-               gpiochip_remove(&data->mcp[addr]->chip);
-       }
-       return status;
-}
-
-static int mcp23s08_remove(struct spi_device *spi)
-{
-       struct mcp23s08_driver_data     *data = spi_get_drvdata(spi);
-       unsigned                        addr;
-
-       for (addr = 0; addr < ARRAY_SIZE(data->mcp); addr++) {
-
-               if (!data->mcp[addr])
-                       continue;
-
-               gpiochip_remove(&data->mcp[addr]->chip);
-       }
-
-       return 0;
-}
-
-static const struct spi_device_id mcp23s08_ids[] = {
-       { "mcp23s08", MCP_TYPE_S08 },
-       { "mcp23s17", MCP_TYPE_S17 },
-       { "mcp23s18", MCP_TYPE_S18 },
-       { },
-};
-MODULE_DEVICE_TABLE(spi, mcp23s08_ids);
-
-static struct spi_driver mcp23s08_driver = {
-       .probe          = mcp23s08_probe,
-       .remove         = mcp23s08_remove,
-       .id_table       = mcp23s08_ids,
-       .driver = {
-               .name   = "mcp23s08",
-               .of_match_table = of_match_ptr(mcp23s08_spi_of_match),
-       },
-};
-
-static int __init mcp23s08_spi_init(void)
-{
-       return spi_register_driver(&mcp23s08_driver);
-}
-
-static void mcp23s08_spi_exit(void)
-{
-       spi_unregister_driver(&mcp23s08_driver);
-}
-
-#else
-
-static int __init mcp23s08_spi_init(void) { return 0; }
-static void mcp23s08_spi_exit(void) { }
-
-#endif /* CONFIG_SPI_MASTER */
-
-/*----------------------------------------------------------------------*/
-
-static int __init mcp23s08_init(void)
-{
-       int ret;
-
-       ret = mcp23s08_spi_init();
-       if (ret)
-               goto spi_fail;
-
-       ret = mcp23s08_i2c_init();
-       if (ret)
-               goto i2c_fail;
-
-       return 0;
-
- i2c_fail:
-       mcp23s08_spi_exit();
- spi_fail:
-       return ret;
-}
-/* register after spi/i2c postcore initcall and before
- * subsys initcalls that may rely on these GPIOs
- */
-subsys_initcall(mcp23s08_init);
-
-static void __exit mcp23s08_exit(void)
-{
-       mcp23s08_spi_exit();
-       mcp23s08_i2c_exit();
-}
-module_exit(mcp23s08_exit);
-
-MODULE_LICENSE("GPL");
index 37af5e3029d54bbbd1ae7d27a970d6b3eb81407d..b5aa50c5163358e5938e0858fb4c024410b7c95c 100644 (file)
@@ -146,6 +146,19 @@ config PINCTRL_FALCON
        depends on SOC_FALCON
        depends on PINCTRL_LANTIQ
 
+config PINCTRL_MCP23S08
+       tristate "Microchip MCP23xxx I/O expander"
+       depends on OF_GPIO
+       depends on SPI_MASTER || I2C
+       select GPIOLIB_IRQCHIP
+       select REGMAP_I2C if I2C
+       select REGMAP_SPI if SPI_MASTER
+       help
+         SPI/I2C driver for Microchip MCP23S08/MCP23S17/MCP23008/MCP23017
+         I/O expanders.
+         This provides a GPIO interface supporting inputs and outputs.
+         The I2C versions of the chips can be used as interrupt-controller.
+
 config PINCTRL_MESON
        bool
        depends on OF
index 0e9b2226a7c2e62100dfa346a7ba8289e9989b72..59d793aa3db3ce8bbc6e4f686c1b6f37973f466e 100644 (file)
@@ -19,6 +19,7 @@ obj-$(CONFIG_PINCTRL_DA850_PUPD) += pinctrl-da850-pupd.o
 obj-$(CONFIG_PINCTRL_DIGICOLOR)        += pinctrl-digicolor.o
 obj-$(CONFIG_PINCTRL_FALCON)   += pinctrl-falcon.o
 obj-$(CONFIG_PINCTRL_MAX77620) += pinctrl-max77620.o
+obj-$(CONFIG_PINCTRL_MCP23S08) += pinctrl-mcp23s08.o
 obj-$(CONFIG_PINCTRL_MESON)    += meson/
 obj-$(CONFIG_PINCTRL_OXNAS)    += pinctrl-oxnas.o
 obj-$(CONFIG_PINCTRL_PALMAS)   += pinctrl-palmas.o
diff --git a/drivers/pinctrl/pinctrl-mcp23s08.c b/drivers/pinctrl/pinctrl-mcp23s08.c
new file mode 100644 (file)
index 0000000..2a57d02
--- /dev/null
@@ -0,0 +1,1047 @@
+/*
+ * 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.
+ */
+
+#include <linux/kernel.h>
+#include <linux/device.h>
+#include <linux/mutex.h>
+#include <linux/module.h>
+#include <linux/gpio.h>
+#include <linux/i2c.h>
+#include <linux/spi/spi.h>
+#include <linux/spi/mcp23s08.h>
+#include <linux/slab.h>
+#include <asm/byteorder.h>
+#include <linux/interrupt.h>
+#include <linux/of_irq.h>
+#include <linux/of_device.h>
+#include <linux/regmap.h>
+
+/**
+ * MCP types supported by driver
+ */
+#define MCP_TYPE_S08   0
+#define MCP_TYPE_S17   1
+#define MCP_TYPE_008   2
+#define MCP_TYPE_017   3
+#define MCP_TYPE_S18    4
+
+/* Registers are all 8 bits wide.
+ *
+ * The mcp23s17 has twice as many bits, and can be configured to work
+ * with either 16 bit registers or with two adjacent 8 bit banks.
+ */
+#define MCP_IODIR      0x00            /* init/reset:  all ones */
+#define MCP_IPOL       0x01
+#define MCP_GPINTEN    0x02
+#define MCP_DEFVAL     0x03
+#define MCP_INTCON     0x04
+#define MCP_IOCON      0x05
+#      define IOCON_MIRROR     (1 << 6)
+#      define IOCON_SEQOP      (1 << 5)
+#      define IOCON_HAEN       (1 << 3)
+#      define IOCON_ODR        (1 << 2)
+#      define IOCON_INTPOL     (1 << 1)
+#      define IOCON_INTCC      (1)
+#define MCP_GPPU       0x06
+#define MCP_INTF       0x07
+#define MCP_INTCAP     0x08
+#define MCP_GPIO       0x09
+#define MCP_OLAT       0x0a
+
+struct mcp23s08;
+
+struct mcp23s08 {
+       u8                      addr;
+       bool                    irq_active_high;
+       bool                    reg_shift;
+
+       u16                     cache[11];
+       u16                     irq_rise;
+       u16                     irq_fall;
+       int                     irq;
+       bool                    irq_controller;
+       /* lock protects the cached values */
+       struct mutex            lock;
+       struct mutex            irq_lock;
+
+       struct gpio_chip        chip;
+
+       struct regmap           *regmap;
+       struct device           *dev;
+};
+
+static const struct regmap_config mcp23x08_regmap = {
+       .reg_bits = 8,
+       .val_bits = 8,
+
+       .reg_stride = 1,
+       .max_register = MCP_OLAT,
+};
+
+static const struct regmap_config mcp23x17_regmap = {
+       .reg_bits = 8,
+       .val_bits = 16,
+
+       .reg_stride = 2,
+       .max_register = MCP_OLAT << 1,
+       .val_format_endian = REGMAP_ENDIAN_LITTLE,
+};
+
+/*----------------------------------------------------------------------*/
+
+#ifdef CONFIG_SPI_MASTER
+
+static int mcp23sxx_spi_write(void *context, const void *data, size_t count)
+{
+       struct mcp23s08 *mcp = context;
+       struct spi_device *spi = to_spi_device(mcp->dev);
+       struct spi_message m;
+       struct spi_transfer t[2] = { { .tx_buf = &mcp->addr, .len = 1, },
+                                    { .tx_buf = data, .len = count, }, };
+
+       spi_message_init(&m);
+       spi_message_add_tail(&t[0], &m);
+       spi_message_add_tail(&t[1], &m);
+
+       return spi_sync(spi, &m);
+}
+
+static int mcp23sxx_spi_gather_write(void *context,
+                               const void *reg, size_t reg_size,
+                               const void *val, size_t val_size)
+{
+       struct mcp23s08 *mcp = context;
+       struct spi_device *spi = to_spi_device(mcp->dev);
+       struct spi_message m;
+       struct spi_transfer t[3] = { { .tx_buf = &mcp->addr, .len = 1, },
+                                    { .tx_buf = reg, .len = reg_size, },
+                                    { .tx_buf = val, .len = val_size, }, };
+
+       spi_message_init(&m);
+       spi_message_add_tail(&t[0], &m);
+       spi_message_add_tail(&t[1], &m);
+       spi_message_add_tail(&t[2], &m);
+
+       return spi_sync(spi, &m);
+}
+
+static int mcp23sxx_spi_read(void *context, const void *reg, size_t reg_size,
+                               void *val, size_t val_size)
+{
+       struct mcp23s08 *mcp = context;
+       struct spi_device *spi = to_spi_device(mcp->dev);
+       u8 tx[2];
+
+       if (reg_size != 1)
+               return -EINVAL;
+
+       tx[0] = mcp->addr | 0x01;
+       tx[1] = *((u8 *) reg);
+
+       return spi_write_then_read(spi, tx, sizeof(tx), val, val_size);
+}
+
+static const struct regmap_bus mcp23sxx_spi_regmap = {
+       .write = mcp23sxx_spi_write,
+       .gather_write = mcp23sxx_spi_gather_write,
+       .read = mcp23sxx_spi_read,
+};
+
+#endif /* CONFIG_SPI_MASTER */
+
+static int mcp_read(struct mcp23s08 *mcp, unsigned int reg, unsigned int *val)
+{
+       return regmap_read(mcp->regmap, reg << mcp->reg_shift, val);
+}
+
+static int mcp_write(struct mcp23s08 *mcp, unsigned int reg, unsigned int val)
+{
+       return regmap_write(mcp->regmap, reg << mcp->reg_shift, val);
+}
+
+static int mcp_update_cache(struct mcp23s08 *mcp)
+{
+       int ret, reg, i;
+
+       for (i = 0; i < ARRAY_SIZE(mcp->cache); i++) {
+               ret = mcp_read(mcp, i, &reg);
+               if (ret < 0)
+                       return ret;
+               mcp->cache[i] = reg;
+       }
+
+       return 0;
+}
+
+/*----------------------------------------------------------------------*/
+
+/* A given spi_device can represent up to eight mcp23sxx chips
+ * sharing the same chipselect but using different addresses
+ * (e.g. chips #0 and #3 might be populated, but not #1 or $2).
+ * Driver data holds all the per-chip data.
+ */
+struct mcp23s08_driver_data {
+       unsigned                ngpio;
+       struct mcp23s08         *mcp[8];
+       struct mcp23s08         chip[];
+};
+
+
+static int mcp23s08_direction_input(struct gpio_chip *chip, unsigned offset)
+{
+       struct mcp23s08 *mcp = gpiochip_get_data(chip);
+       int status;
+
+       mutex_lock(&mcp->lock);
+       mcp->cache[MCP_IODIR] |= (1 << offset);
+       status = mcp_write(mcp, MCP_IODIR, mcp->cache[MCP_IODIR]);
+       mutex_unlock(&mcp->lock);
+       return status;
+}
+
+static int mcp23s08_get(struct gpio_chip *chip, unsigned offset)
+{
+       struct mcp23s08 *mcp = gpiochip_get_data(chip);
+       int status, ret;
+
+       mutex_lock(&mcp->lock);
+
+       /* REVISIT reading this clears any IRQ ... */
+       ret = mcp_read(mcp, MCP_GPIO, &status);
+       if (ret < 0)
+               status = 0;
+       else {
+               mcp->cache[MCP_GPIO] = status;
+               status = !!(status & (1 << offset));
+       }
+       mutex_unlock(&mcp->lock);
+       return status;
+}
+
+static int __mcp23s08_set(struct mcp23s08 *mcp, unsigned mask, int value)
+{
+       unsigned olat = mcp->cache[MCP_OLAT];
+
+       if (value)
+               olat |= mask;
+       else
+               olat &= ~mask;
+       mcp->cache[MCP_OLAT] = olat;
+       return mcp_write(mcp, MCP_OLAT, olat);
+}
+
+static void mcp23s08_set(struct gpio_chip *chip, unsigned offset, int value)
+{
+       struct mcp23s08 *mcp = gpiochip_get_data(chip);
+       unsigned mask = 1 << offset;
+
+       mutex_lock(&mcp->lock);
+       __mcp23s08_set(mcp, mask, value);
+       mutex_unlock(&mcp->lock);
+}
+
+static int
+mcp23s08_direction_output(struct gpio_chip *chip, unsigned offset, int value)
+{
+       struct mcp23s08 *mcp = gpiochip_get_data(chip);
+       unsigned mask = 1 << offset;
+       int status;
+
+       mutex_lock(&mcp->lock);
+       status = __mcp23s08_set(mcp, mask, value);
+       if (status == 0) {
+               mcp->cache[MCP_IODIR] &= ~mask;
+               status = mcp_write(mcp, MCP_IODIR, mcp->cache[MCP_IODIR]);
+       }
+       mutex_unlock(&mcp->lock);
+       return status;
+}
+
+/*----------------------------------------------------------------------*/
+static irqreturn_t mcp23s08_irq(int irq, void *data)
+{
+       struct mcp23s08 *mcp = data;
+       int intcap, intf, i, gpio, gpio_orig, intcap_mask;
+       unsigned int child_irq;
+       bool intf_set, intcap_changed, gpio_bit_changed,
+               defval_changed, gpio_set;
+
+       mutex_lock(&mcp->lock);
+       if (mcp_read(mcp, MCP_INTF, &intf) < 0) {
+               mutex_unlock(&mcp->lock);
+               return IRQ_HANDLED;
+       }
+
+       mcp->cache[MCP_INTF] = intf;
+
+       if (mcp_read(mcp, MCP_INTCAP, &intcap) < 0) {
+               mutex_unlock(&mcp->lock);
+               return IRQ_HANDLED;
+       }
+
+       mcp->cache[MCP_INTCAP] = intcap;
+
+       /* This clears the interrupt(configurable on S18) */
+       if (mcp_read(mcp, MCP_GPIO, &gpio) < 0) {
+               mutex_unlock(&mcp->lock);
+               return IRQ_HANDLED;
+       }
+       gpio_orig = mcp->cache[MCP_GPIO];
+       mcp->cache[MCP_GPIO] = gpio;
+       mutex_unlock(&mcp->lock);
+
+       if (mcp->cache[MCP_INTF] == 0) {
+               /* There is no interrupt pending */
+               return IRQ_HANDLED;
+       }
+
+       dev_dbg(mcp->chip.parent,
+               "intcap 0x%04X intf 0x%04X gpio_orig 0x%04X gpio 0x%04X\n",
+               intcap, intf, gpio_orig, gpio);
+
+       for (i = 0; i < mcp->chip.ngpio; i++) {
+               /* We must check all of the inputs on the chip,
+                * otherwise we may not notice a change on >=2 pins.
+                *
+                * On at least the mcp23s17, INTCAP is only updated
+                * one byte at a time(INTCAPA and INTCAPB are
+                * not written to at the same time - only on a per-bank
+                * basis).
+                *
+                * INTF only contains the single bit that caused the
+                * interrupt per-bank.  On the mcp23s17, there is
+                * INTFA and INTFB.  If two pins are changed on the A
+                * side at the same time, INTF will only have one bit
+                * set.  If one pin on the A side and one pin on the B
+                * side are changed at the same time, INTF will have
+                * two bits set.  Thus, INTF can't be the only check
+                * to see if the input has changed.
+                */
+
+               intf_set = BIT(i) & mcp->cache[MCP_INTF];
+               if (i < 8 && intf_set)
+                       intcap_mask = 0x00FF;
+               else if (i >= 8 && intf_set)
+                       intcap_mask = 0xFF00;
+               else
+                       intcap_mask = 0x00;
+
+               intcap_changed = (intcap_mask &
+                       (BIT(i) & mcp->cache[MCP_INTCAP])) !=
+                       (intcap_mask & (BIT(i) & gpio_orig));
+               gpio_set = BIT(i) & mcp->cache[MCP_GPIO];
+               gpio_bit_changed = (BIT(i) & gpio_orig) !=
+                       (BIT(i) & mcp->cache[MCP_GPIO]);
+               defval_changed = (BIT(i) & mcp->cache[MCP_INTCON]) &&
+                       ((BIT(i) & mcp->cache[MCP_GPIO]) !=
+                       (BIT(i) & mcp->cache[MCP_DEFVAL]));
+
+               if (((gpio_bit_changed || intcap_changed) &&
+                       (BIT(i) & mcp->irq_rise) && gpio_set) ||
+                   ((gpio_bit_changed || intcap_changed) &&
+                       (BIT(i) & mcp->irq_fall) && !gpio_set) ||
+                   defval_changed) {
+                       child_irq = irq_find_mapping(mcp->chip.irqdomain, i);
+                       handle_nested_irq(child_irq);
+               }
+       }
+
+       return IRQ_HANDLED;
+}
+
+static void mcp23s08_irq_mask(struct irq_data *data)
+{
+       struct gpio_chip *gc = irq_data_get_irq_chip_data(data);
+       struct mcp23s08 *mcp = gpiochip_get_data(gc);
+       unsigned int pos = data->hwirq;
+
+       mcp->cache[MCP_GPINTEN] &= ~BIT(pos);
+}
+
+static void mcp23s08_irq_unmask(struct irq_data *data)
+{
+       struct gpio_chip *gc = irq_data_get_irq_chip_data(data);
+       struct mcp23s08 *mcp = gpiochip_get_data(gc);
+       unsigned int pos = data->hwirq;
+
+       mcp->cache[MCP_GPINTEN] |= BIT(pos);
+}
+
+static int mcp23s08_irq_set_type(struct irq_data *data, unsigned int type)
+{
+       struct gpio_chip *gc = irq_data_get_irq_chip_data(data);
+       struct mcp23s08 *mcp = gpiochip_get_data(gc);
+       unsigned int pos = data->hwirq;
+       int status = 0;
+
+       if ((type & IRQ_TYPE_EDGE_BOTH) == IRQ_TYPE_EDGE_BOTH) {
+               mcp->cache[MCP_INTCON] &= ~BIT(pos);
+               mcp->irq_rise |= BIT(pos);
+               mcp->irq_fall |= BIT(pos);
+       } else if (type & IRQ_TYPE_EDGE_RISING) {
+               mcp->cache[MCP_INTCON] &= ~BIT(pos);
+               mcp->irq_rise |= BIT(pos);
+               mcp->irq_fall &= ~BIT(pos);
+       } else if (type & IRQ_TYPE_EDGE_FALLING) {
+               mcp->cache[MCP_INTCON] &= ~BIT(pos);
+               mcp->irq_rise &= ~BIT(pos);
+               mcp->irq_fall |= BIT(pos);
+       } else if (type & IRQ_TYPE_LEVEL_HIGH) {
+               mcp->cache[MCP_INTCON] |= BIT(pos);
+               mcp->cache[MCP_DEFVAL] &= ~BIT(pos);
+       } else if (type & IRQ_TYPE_LEVEL_LOW) {
+               mcp->cache[MCP_INTCON] |= BIT(pos);
+               mcp->cache[MCP_DEFVAL] |= BIT(pos);
+       } else
+               return -EINVAL;
+
+       return status;
+}
+
+static void mcp23s08_irq_bus_lock(struct irq_data *data)
+{
+       struct gpio_chip *gc = irq_data_get_irq_chip_data(data);
+       struct mcp23s08 *mcp = gpiochip_get_data(gc);
+
+       mutex_lock(&mcp->irq_lock);
+}
+
+static void mcp23s08_irq_bus_unlock(struct irq_data *data)
+{
+       struct gpio_chip *gc = irq_data_get_irq_chip_data(data);
+       struct mcp23s08 *mcp = gpiochip_get_data(gc);
+
+       mutex_lock(&mcp->lock);
+       mcp_write(mcp, MCP_GPINTEN, mcp->cache[MCP_GPINTEN]);
+       mcp_write(mcp, MCP_DEFVAL, mcp->cache[MCP_DEFVAL]);
+       mcp_write(mcp, MCP_INTCON, mcp->cache[MCP_INTCON]);
+       mutex_unlock(&mcp->lock);
+       mutex_unlock(&mcp->irq_lock);
+}
+
+static struct irq_chip mcp23s08_irq_chip = {
+       .name = "gpio-mcp23xxx",
+       .irq_mask = mcp23s08_irq_mask,
+       .irq_unmask = mcp23s08_irq_unmask,
+       .irq_set_type = mcp23s08_irq_set_type,
+       .irq_bus_lock = mcp23s08_irq_bus_lock,
+       .irq_bus_sync_unlock = mcp23s08_irq_bus_unlock,
+};
+
+static int mcp23s08_irq_setup(struct mcp23s08 *mcp)
+{
+       struct gpio_chip *chip = &mcp->chip;
+       int err;
+       unsigned long irqflags = IRQF_ONESHOT | IRQF_SHARED;
+
+       mutex_init(&mcp->irq_lock);
+
+       if (mcp->irq_active_high)
+               irqflags |= IRQF_TRIGGER_HIGH;
+       else
+               irqflags |= IRQF_TRIGGER_LOW;
+
+       err = devm_request_threaded_irq(chip->parent, mcp->irq, NULL,
+                                       mcp23s08_irq,
+                                       irqflags, dev_name(chip->parent), mcp);
+       if (err != 0) {
+               dev_err(chip->parent, "unable to request IRQ#%d: %d\n",
+                       mcp->irq, err);
+               return err;
+       }
+
+       err =  gpiochip_irqchip_add_nested(chip,
+                                          &mcp23s08_irq_chip,
+                                          0,
+                                          handle_simple_irq,
+                                          IRQ_TYPE_NONE);
+       if (err) {
+               dev_err(chip->parent,
+                       "could not connect irqchip to gpiochip: %d\n", err);
+               return err;
+       }
+
+       gpiochip_set_nested_irqchip(chip,
+                                   &mcp23s08_irq_chip,
+                                   mcp->irq);
+
+       return 0;
+}
+
+/*----------------------------------------------------------------------*/
+
+#ifdef CONFIG_DEBUG_FS
+
+#include <linux/seq_file.h>
+
+/*
+ * This shows more info than the generic gpio dump code:
+ * pullups, deglitching, open drain drive.
+ */
+static void mcp23s08_dbg_show(struct seq_file *s, struct gpio_chip *chip)
+{
+       struct mcp23s08 *mcp;
+       char            bank;
+       int             t;
+       unsigned        mask;
+
+       mcp = gpiochip_get_data(chip);
+
+       /* NOTE: we only handle one bank for now ... */
+       bank = '0' + ((mcp->addr >> 1) & 0x7);
+
+       mutex_lock(&mcp->lock);
+       t = mcp_update_cache(mcp);
+       if (t < 0) {
+               seq_printf(s, " I/O ERROR %d\n", t);
+               goto done;
+       }
+
+       for (t = 0, mask = 1; t < chip->ngpio; t++, mask <<= 1) {
+               const char      *label;
+
+               label = gpiochip_is_requested(chip, t);
+               if (!label)
+                       continue;
+
+               seq_printf(s, " gpio-%-3d P%c.%d (%-12s) %s %s %s",
+                       chip->base + t, bank, t, label,
+                       (mcp->cache[MCP_IODIR] & mask) ? "in " : "out",
+                       (mcp->cache[MCP_GPIO] & mask) ? "hi" : "lo",
+                       (mcp->cache[MCP_GPPU] & mask) ? "up" : "  ");
+               /* NOTE:  ignoring the irq-related registers */
+               seq_puts(s, "\n");
+       }
+done:
+       mutex_unlock(&mcp->lock);
+}
+
+#else
+#define mcp23s08_dbg_show      NULL
+#endif
+
+/*----------------------------------------------------------------------*/
+
+static int mcp23s08_probe_one(struct mcp23s08 *mcp, struct device *dev,
+                             void *data, unsigned addr, unsigned type,
+                             struct mcp23s08_platform_data *pdata, int cs)
+{
+       int status, ret;
+       bool mirror = false;
+
+       mutex_init(&mcp->lock);
+
+       mcp->dev = dev;
+       mcp->addr = addr;
+       mcp->irq_active_high = false;
+
+       mcp->chip.direction_input = mcp23s08_direction_input;
+       mcp->chip.get = mcp23s08_get;
+       mcp->chip.direction_output = mcp23s08_direction_output;
+       mcp->chip.set = mcp23s08_set;
+       mcp->chip.dbg_show = mcp23s08_dbg_show;
+#ifdef CONFIG_OF_GPIO
+       mcp->chip.of_gpio_n_cells = 2;
+       mcp->chip.of_node = dev->of_node;
+#endif
+
+       switch (type) {
+#ifdef CONFIG_SPI_MASTER
+       case MCP_TYPE_S08:
+               mcp->regmap = devm_regmap_init(dev, &mcp23sxx_spi_regmap, mcp,
+                                              &mcp23x08_regmap);
+               mcp->reg_shift = 0;
+               mcp->chip.ngpio = 8;
+               mcp->chip.label = "mcp23s08";
+               break;
+
+       case MCP_TYPE_S17:
+               mcp->regmap = devm_regmap_init(dev, &mcp23sxx_spi_regmap, mcp,
+                                              &mcp23x17_regmap);
+               mcp->reg_shift = 1;
+               mcp->chip.ngpio = 16;
+               mcp->chip.label = "mcp23s17";
+               break;
+
+       case MCP_TYPE_S18:
+               mcp->regmap = devm_regmap_init(dev, &mcp23sxx_spi_regmap, mcp,
+                                              &mcp23x17_regmap);
+               mcp->reg_shift = 1;
+               mcp->chip.ngpio = 16;
+               mcp->chip.label = "mcp23s18";
+               break;
+#endif /* CONFIG_SPI_MASTER */
+
+#if IS_ENABLED(CONFIG_I2C)
+       case MCP_TYPE_008:
+               mcp->regmap = devm_regmap_init_i2c(data, &mcp23x08_regmap);
+               mcp->reg_shift = 0;
+               mcp->chip.ngpio = 8;
+               mcp->chip.label = "mcp23008";
+               break;
+
+       case MCP_TYPE_017:
+               mcp->regmap = devm_regmap_init_i2c(data, &mcp23x17_regmap);
+               mcp->reg_shift = 1;
+               mcp->chip.ngpio = 16;
+               mcp->chip.label = "mcp23017";
+               break;
+#endif /* CONFIG_I2C */
+
+       default:
+               dev_err(dev, "invalid device type (%d)\n", type);
+               return -EINVAL;
+       }
+
+       if (IS_ERR(mcp->regmap))
+               return PTR_ERR(mcp->regmap);
+
+       mcp->chip.base = pdata->base;
+       mcp->chip.can_sleep = true;
+       mcp->chip.parent = dev;
+       mcp->chip.owner = THIS_MODULE;
+
+       /* verify MCP_IOCON.SEQOP = 0, so sequential reads work,
+        * and MCP_IOCON.HAEN = 1, so we work with all chips.
+        */
+
+       ret = mcp_read(mcp, MCP_IOCON, &status);
+       if (ret < 0)
+               goto fail;
+
+       mcp->irq_controller = pdata->irq_controller;
+       if (mcp->irq && mcp->irq_controller) {
+               mcp->irq_active_high =
+                       of_property_read_bool(mcp->chip.parent->of_node,
+                                             "microchip,irq-active-high");
+
+               mirror = pdata->mirror;
+       }
+
+       if ((status & IOCON_SEQOP) || !(status & IOCON_HAEN) || mirror ||
+            mcp->irq_active_high) {
+               /* mcp23s17 has IOCON twice, make sure they are in sync */
+               status &= ~(IOCON_SEQOP | (IOCON_SEQOP << 8));
+               status |= IOCON_HAEN | (IOCON_HAEN << 8);
+               if (mcp->irq_active_high)
+                       status |= IOCON_INTPOL | (IOCON_INTPOL << 8);
+               else
+                       status &= ~(IOCON_INTPOL | (IOCON_INTPOL << 8));
+
+               if (mirror)
+                       status |= IOCON_MIRROR | (IOCON_MIRROR << 8);
+
+               if (type == MCP_TYPE_S18)
+                       status |= IOCON_INTCC | (IOCON_INTCC << 8);
+
+               ret = mcp_write(mcp, MCP_IOCON, status);
+               if (ret < 0)
+                       goto fail;
+       }
+
+       /* configure ~100K pullups */
+       ret = mcp_write(mcp, MCP_GPPU, pdata->chip[cs].pullups);
+       if (ret < 0)
+               goto fail;
+
+       ret = mcp_update_cache(mcp);
+       if (ret < 0)
+               goto fail;
+
+       /* disable inverter on input */
+       if (mcp->cache[MCP_IPOL] != 0) {
+               mcp->cache[MCP_IPOL] = 0;
+               ret = mcp_write(mcp, MCP_IPOL, 0);
+               if (ret < 0)
+                       goto fail;
+       }
+
+       /* disable irqs */
+       if (mcp->cache[MCP_GPINTEN] != 0) {
+               mcp->cache[MCP_GPINTEN] = 0;
+               ret = mcp_write(mcp, MCP_GPINTEN, 0);
+               if (ret < 0)
+                       goto fail;
+       }
+
+       ret = gpiochip_add_data(&mcp->chip, mcp);
+       if (ret < 0)
+               goto fail;
+
+       if (mcp->irq && mcp->irq_controller) {
+               ret = mcp23s08_irq_setup(mcp);
+               if (ret)
+                       goto fail;
+       }
+fail:
+       if (ret < 0)
+               dev_dbg(dev, "can't setup chip %d, --> %d\n", addr, ret);
+       return ret;
+}
+
+/*----------------------------------------------------------------------*/
+
+#ifdef CONFIG_OF
+#ifdef CONFIG_SPI_MASTER
+static const struct of_device_id mcp23s08_spi_of_match[] = {
+       {
+               .compatible = "microchip,mcp23s08",
+               .data = (void *) MCP_TYPE_S08,
+       },
+       {
+               .compatible = "microchip,mcp23s17",
+               .data = (void *) MCP_TYPE_S17,
+       },
+       {
+               .compatible = "microchip,mcp23s18",
+               .data = (void *) MCP_TYPE_S18,
+       },
+/* NOTE: The use of the mcp prefix is deprecated and will be removed. */
+       {
+               .compatible = "mcp,mcp23s08",
+               .data = (void *) MCP_TYPE_S08,
+       },
+       {
+               .compatible = "mcp,mcp23s17",
+               .data = (void *) MCP_TYPE_S17,
+       },
+       { },
+};
+MODULE_DEVICE_TABLE(of, mcp23s08_spi_of_match);
+#endif
+
+#if IS_ENABLED(CONFIG_I2C)
+static const struct of_device_id mcp23s08_i2c_of_match[] = {
+       {
+               .compatible = "microchip,mcp23008",
+               .data = (void *) MCP_TYPE_008,
+       },
+       {
+               .compatible = "microchip,mcp23017",
+               .data = (void *) MCP_TYPE_017,
+       },
+/* NOTE: The use of the mcp prefix is deprecated and will be removed. */
+       {
+               .compatible = "mcp,mcp23008",
+               .data = (void *) MCP_TYPE_008,
+       },
+       {
+               .compatible = "mcp,mcp23017",
+               .data = (void *) MCP_TYPE_017,
+       },
+       { },
+};
+MODULE_DEVICE_TABLE(of, mcp23s08_i2c_of_match);
+#endif
+#endif /* CONFIG_OF */
+
+
+#if IS_ENABLED(CONFIG_I2C)
+
+static int mcp230xx_probe(struct i2c_client *client,
+                                   const struct i2c_device_id *id)
+{
+       struct mcp23s08_platform_data *pdata, local_pdata;
+       struct mcp23s08 *mcp;
+       int status;
+       const struct of_device_id *match;
+
+       match = of_match_device(of_match_ptr(mcp23s08_i2c_of_match),
+                                       &client->dev);
+       if (match) {
+               pdata = &local_pdata;
+               pdata->base = -1;
+               pdata->chip[0].pullups = 0;
+               pdata->irq_controller = of_property_read_bool(
+                                       client->dev.of_node,
+                                       "interrupt-controller");
+               pdata->mirror = of_property_read_bool(client->dev.of_node,
+                                                     "microchip,irq-mirror");
+               client->irq = irq_of_parse_and_map(client->dev.of_node, 0);
+       } else {
+               pdata = dev_get_platdata(&client->dev);
+               if (!pdata) {
+                       pdata = devm_kzalloc(&client->dev,
+                                       sizeof(struct mcp23s08_platform_data),
+                                       GFP_KERNEL);
+                       if (!pdata)
+                               return -ENOMEM;
+                       pdata->base = -1;
+               }
+       }
+
+       mcp = kzalloc(sizeof(*mcp), GFP_KERNEL);
+       if (!mcp)
+               return -ENOMEM;
+
+       mcp->irq = client->irq;
+       status = mcp23s08_probe_one(mcp, &client->dev, client, client->addr,
+                                   id->driver_data, pdata, 0);
+       if (status)
+               goto fail;
+
+       i2c_set_clientdata(client, mcp);
+
+       return 0;
+
+fail:
+       kfree(mcp);
+
+       return status;
+}
+
+static int mcp230xx_remove(struct i2c_client *client)
+{
+       struct mcp23s08 *mcp = i2c_get_clientdata(client);
+
+       gpiochip_remove(&mcp->chip);
+       kfree(mcp);
+
+       return 0;
+}
+
+static const struct i2c_device_id mcp230xx_id[] = {
+       { "mcp23008", MCP_TYPE_008 },
+       { "mcp23017", MCP_TYPE_017 },
+       { },
+};
+MODULE_DEVICE_TABLE(i2c, mcp230xx_id);
+
+static struct i2c_driver mcp230xx_driver = {
+       .driver = {
+               .name   = "mcp230xx",
+               .of_match_table = of_match_ptr(mcp23s08_i2c_of_match),
+       },
+       .probe          = mcp230xx_probe,
+       .remove         = mcp230xx_remove,
+       .id_table       = mcp230xx_id,
+};
+
+static int __init mcp23s08_i2c_init(void)
+{
+       return i2c_add_driver(&mcp230xx_driver);
+}
+
+static void mcp23s08_i2c_exit(void)
+{
+       i2c_del_driver(&mcp230xx_driver);
+}
+
+#else
+
+static int __init mcp23s08_i2c_init(void) { return 0; }
+static void mcp23s08_i2c_exit(void) { }
+
+#endif /* CONFIG_I2C */
+
+/*----------------------------------------------------------------------*/
+
+#ifdef CONFIG_SPI_MASTER
+
+static int mcp23s08_probe(struct spi_device *spi)
+{
+       struct mcp23s08_platform_data   *pdata, local_pdata;
+       unsigned                        addr;
+       int                             chips = 0;
+       struct mcp23s08_driver_data     *data;
+       int                             status, type;
+       unsigned                        ngpio = 0;
+       const struct                    of_device_id *match;
+       u32                             spi_present_mask = 0;
+
+       match = of_match_device(of_match_ptr(mcp23s08_spi_of_match), &spi->dev);
+       if (match) {
+               type = (int)(uintptr_t)match->data;
+               status = of_property_read_u32(spi->dev.of_node,
+                           "microchip,spi-present-mask", &spi_present_mask);
+               if (status) {
+                       status = of_property_read_u32(spi->dev.of_node,
+                                   "mcp,spi-present-mask", &spi_present_mask);
+                       if (status) {
+                               dev_err(&spi->dev,
+                                       "DT has no spi-present-mask\n");
+                               return -ENODEV;
+                       }
+               }
+               if ((spi_present_mask <= 0) || (spi_present_mask >= 256)) {
+                       dev_err(&spi->dev, "invalid spi-present-mask\n");
+                       return -ENODEV;
+               }
+
+               pdata = &local_pdata;
+               pdata->base = -1;
+               for (addr = 0; addr < ARRAY_SIZE(pdata->chip); addr++) {
+                       pdata->chip[addr].pullups = 0;
+                       if (spi_present_mask & (1 << addr))
+                               chips++;
+               }
+               pdata->irq_controller = of_property_read_bool(
+                                       spi->dev.of_node,
+                                       "interrupt-controller");
+               pdata->mirror = of_property_read_bool(spi->dev.of_node,
+                                                     "microchip,irq-mirror");
+       } else {
+               type = spi_get_device_id(spi)->driver_data;
+               pdata = dev_get_platdata(&spi->dev);
+               if (!pdata) {
+                       pdata = devm_kzalloc(&spi->dev,
+                                       sizeof(struct mcp23s08_platform_data),
+                                       GFP_KERNEL);
+                       pdata->base = -1;
+               }
+
+               for (addr = 0; addr < ARRAY_SIZE(pdata->chip); addr++) {
+                       if (!pdata->chip[addr].is_present)
+                               continue;
+                       chips++;
+                       if ((type == MCP_TYPE_S08) && (addr > 3)) {
+                               dev_err(&spi->dev,
+                                       "mcp23s08 only supports address 0..3\n");
+                               return -EINVAL;
+                       }
+                       spi_present_mask |= 1 << addr;
+               }
+       }
+
+       if (!chips)
+               return -ENODEV;
+
+       data = devm_kzalloc(&spi->dev,
+                           sizeof(*data) + chips * sizeof(struct mcp23s08),
+                           GFP_KERNEL);
+       if (!data)
+               return -ENOMEM;
+
+       spi_set_drvdata(spi, data);
+
+       spi->irq = irq_of_parse_and_map(spi->dev.of_node, 0);
+
+       for (addr = 0; addr < ARRAY_SIZE(pdata->chip); addr++) {
+               if (!(spi_present_mask & (1 << addr)))
+                       continue;
+               chips--;
+               data->mcp[addr] = &data->chip[chips];
+               data->mcp[addr]->irq = spi->irq;
+               status = mcp23s08_probe_one(data->mcp[addr], &spi->dev, spi,
+                                           0x40 | (addr << 1), type, pdata,
+                                           addr);
+               if (status < 0)
+                       goto fail;
+
+               if (pdata->base != -1)
+                       pdata->base += data->mcp[addr]->chip.ngpio;
+               ngpio += data->mcp[addr]->chip.ngpio;
+       }
+       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;
+
+fail:
+       for (addr = 0; addr < ARRAY_SIZE(data->mcp); addr++) {
+
+               if (!data->mcp[addr])
+                       continue;
+               gpiochip_remove(&data->mcp[addr]->chip);
+       }
+       return status;
+}
+
+static int mcp23s08_remove(struct spi_device *spi)
+{
+       struct mcp23s08_driver_data     *data = spi_get_drvdata(spi);
+       unsigned                        addr;
+
+       for (addr = 0; addr < ARRAY_SIZE(data->mcp); addr++) {
+
+               if (!data->mcp[addr])
+                       continue;
+
+               gpiochip_remove(&data->mcp[addr]->chip);
+       }
+
+       return 0;
+}
+
+static const struct spi_device_id mcp23s08_ids[] = {
+       { "mcp23s08", MCP_TYPE_S08 },
+       { "mcp23s17", MCP_TYPE_S17 },
+       { "mcp23s18", MCP_TYPE_S18 },
+       { },
+};
+MODULE_DEVICE_TABLE(spi, mcp23s08_ids);
+
+static struct spi_driver mcp23s08_driver = {
+       .probe          = mcp23s08_probe,
+       .remove         = mcp23s08_remove,
+       .id_table       = mcp23s08_ids,
+       .driver = {
+               .name   = "mcp23s08",
+               .of_match_table = of_match_ptr(mcp23s08_spi_of_match),
+       },
+};
+
+static int __init mcp23s08_spi_init(void)
+{
+       return spi_register_driver(&mcp23s08_driver);
+}
+
+static void mcp23s08_spi_exit(void)
+{
+       spi_unregister_driver(&mcp23s08_driver);
+}
+
+#else
+
+static int __init mcp23s08_spi_init(void) { return 0; }
+static void mcp23s08_spi_exit(void) { }
+
+#endif /* CONFIG_SPI_MASTER */
+
+/*----------------------------------------------------------------------*/
+
+static int __init mcp23s08_init(void)
+{
+       int ret;
+
+       ret = mcp23s08_spi_init();
+       if (ret)
+               goto spi_fail;
+
+       ret = mcp23s08_i2c_init();
+       if (ret)
+               goto i2c_fail;
+
+       return 0;
+
+ i2c_fail:
+       mcp23s08_spi_exit();
+ spi_fail:
+       return ret;
+}
+/* register after spi/i2c postcore initcall and before
+ * subsys initcalls that may rely on these GPIOs
+ */
+subsys_initcall(mcp23s08_init);
+
+static void __exit mcp23s08_exit(void)
+{
+       mcp23s08_spi_exit();
+       mcp23s08_i2c_exit();
+}
+module_exit(mcp23s08_exit);
+
+MODULE_LICENSE("GPL");