Merge branch 'iop-cleanup' into devel
authorLinus Walleij <linus.walleij@linaro.org>
Mon, 23 Sep 2013 10:47:14 +0000 (12:47 +0200)
committerLinus Walleij <linus.walleij@linaro.org>
Mon, 23 Sep 2013 10:47:14 +0000 (12:47 +0200)
Documentation/devicetree/bindings/gpio/gpio-bcm-kona.txt [new file with mode: 0644]
drivers/gpio/Kconfig
drivers/gpio/Makefile
drivers/gpio/gpio-arizona.c
drivers/gpio/gpio-bcm-kona.c [new file with mode: 0644]
drivers/gpio/gpio-bt8xx.c
drivers/gpio/gpio-em.c
drivers/gpio/gpio-palmas.c
drivers/gpio/gpio-pcf857x.c
drivers/gpio/gpiolib-acpi.c
drivers/gpio/gpiolib.c

diff --git a/Documentation/devicetree/bindings/gpio/gpio-bcm-kona.txt b/Documentation/devicetree/bindings/gpio/gpio-bcm-kona.txt
new file mode 100644 (file)
index 0000000..4a63bc9
--- /dev/null
@@ -0,0 +1,52 @@
+Broadcom Kona Family GPIO
+=========================
+
+This GPIO driver is used in the following Broadcom SoCs:
+  BCM11130, BCM11140, BCM11351, BCM28145, BCM28155
+
+The Broadcom GPIO Controller IP can be configured prior to synthesis to
+support up to 8 banks of 32 GPIOs where each bank has its own IRQ. The
+GPIO controller only supports edge, not level, triggering of interrupts.
+
+Required properties
+-------------------
+
+- compatible: "brcm,bcm11351-gpio", "brcm,kona-gpio"
+- reg: Physical base address and length of the controller's registers.
+- interrupts: The interrupt outputs from the controller. There is one GPIO
+  interrupt per GPIO bank. The number of interrupts listed depends on the
+  number of GPIO banks on the SoC. The interrupts must be ordered by bank,
+  starting with bank 0. There is always a 1:1 mapping between banks and
+  IRQs.
+- #gpio-cells: Should be <2>. The first cell is the pin number, the second
+  cell is used to specify optional parameters:
+  - bit 0 specifies polarity (0 for normal, 1 for inverted)
+  See also "gpio-specifier" in .../devicetree/bindings/gpio/gpio.txt.
+- #interrupt-cells: Should be <2>. The first cell is the GPIO number. The
+  second cell is used to specify flags. The following subset of flags is
+  supported:
+  - trigger type (bits[1:0]):
+      1 = low-to-high edge triggered.
+      2 = high-to-low edge triggered.
+      3 = low-to-high or high-to-low edge triggered
+      Valid values are 1, 2, 3
+  See also .../devicetree/bindings/interrupt-controller/interrupts.txt.
+- gpio-controller: Marks the device node as a GPIO controller.
+- interrupt-controller: Marks the device node as an interrupt controller.
+
+Example:
+       gpio: gpio@35003000 {
+               compatible = "brcm,bcm11351-gpio", "brcm,kona-gpio";
+               reg = <0x35003000 0x800>;
+               interrupts =
+                      <GIC_SPI 106 IRQ_TYPE_LEVEL_HIGH
+                       GIC_SPI 115 IRQ_TYPE_LEVEL_HIGH
+                       GIC_SPI 114 IRQ_TYPE_LEVEL_HIGH
+                       GIC_SPI 113 IRQ_TYPE_LEVEL_HIGH
+                       GIC_SPI 112 IRQ_TYPE_LEVEL_HIGH
+                       GIC_SPI 111 IRQ_TYPE_LEVEL_HIGH>;
+               #gpio-cells = <2>;
+               #interrupt-cells = <2>;
+               gpio-controller;
+               interrupt-controller;
+       };
index cc30426..c8b02a5 100644 (file)
@@ -129,7 +129,7 @@ config GPIO_IT8761E
 
 config GPIO_EM
        tristate "Emma Mobile GPIO"
-       depends on ARM
+       depends on ARM && OF_GPIO
        help
          Say yes here to support GPIO on Renesas Emma Mobile SoCs.
 
@@ -712,7 +712,7 @@ config GPIO_74X164
 comment "AC97 GPIO expanders:"
 
 config GPIO_UCB1400
-       bool "Philips UCB1400 GPIO"
+       tristate "Philips UCB1400 GPIO"
        depends on UCB1400_CORE
        help
          This enables support for the Philips UCB1400 GPIO pins.
@@ -768,6 +768,12 @@ config GPIO_MSIC
          Enable support for GPIO on intel MSIC controllers found in
          intel MID devices
 
+config GPIO_BCM_KONA
+       bool "Broadcom Kona GPIO"
+       depends on OF_GPIO
+       help
+         Turn on GPIO support for Broadcom "Kona" chips.
+
 comment "USB GPIO expanders:"
 
 config GPIO_VIPERBOARD
index 06e5662..5c353df 100644 (file)
@@ -16,6 +16,7 @@ obj-$(CONFIG_GPIO_ADP5520)    += gpio-adp5520.o
 obj-$(CONFIG_GPIO_ADP5588)     += gpio-adp5588.o
 obj-$(CONFIG_GPIO_AMD8111)     += gpio-amd8111.o
 obj-$(CONFIG_GPIO_ARIZONA)     += gpio-arizona.o
+obj-$(CONFIG_GPIO_BCM_KONA)    += gpio-bcm-kona.o
 obj-$(CONFIG_GPIO_BT8XX)       += gpio-bt8xx.o
 obj-$(CONFIG_GPIO_CLPS711X)    += gpio-clps711x.o
 obj-$(CONFIG_GPIO_CS5535)      += gpio-cs5535.o
index fa8b6a7..a2bec3c 100644 (file)
@@ -113,6 +113,7 @@ static int arizona_gpio_probe(struct platform_device *pdev)
        switch (arizona->type) {
        case WM5102:
        case WM5110:
+       case WM8997:
                arizona_gpio->gpio_chip.ngpio = 5;
                break;
        default:
diff --git a/drivers/gpio/gpio-bcm-kona.c b/drivers/gpio/gpio-bcm-kona.c
new file mode 100644 (file)
index 0000000..c0751a8
--- /dev/null
@@ -0,0 +1,640 @@
+/*
+ * Copyright (C) 2012-2013 Broadcom Corporation
+ *
+ * This program is free software; you can redistribute it and/or
+ * modify it under the terms of the GNU General Public License as
+ * published by the Free Software Foundation version 2.
+ *
+ * This program is distributed "as is" WITHOUT ANY WARRANTY of any
+ * kind, whether express or implied; without even the implied warranty
+ * of MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+ * GNU General Public License for more details.
+ */
+
+#include <linux/bitops.h>
+#include <linux/err.h>
+#include <linux/io.h>
+#include <linux/gpio.h>
+#include <linux/of_device.h>
+#include <linux/of_irq.h>
+#include <linux/module.h>
+#include <linux/irqdomain.h>
+#include <linux/irqchip/chained_irq.h>
+
+#define BCM_GPIO_PASSWD                                0x00a5a501
+#define GPIO_PER_BANK                          32
+#define GPIO_MAX_BANK_NUM                      8
+
+#define GPIO_BANK(gpio)                                ((gpio) >> 5)
+#define GPIO_BIT(gpio)                         ((gpio) & (GPIO_PER_BANK - 1))
+
+#define GPIO_OUT_STATUS(bank)                  (0x00000000 + ((bank) << 2))
+#define GPIO_IN_STATUS(bank)                   (0x00000020 + ((bank) << 2))
+#define GPIO_OUT_SET(bank)                     (0x00000040 + ((bank) << 2))
+#define GPIO_OUT_CLEAR(bank)                   (0x00000060 + ((bank) << 2))
+#define GPIO_INT_STATUS(bank)                  (0x00000080 + ((bank) << 2))
+#define GPIO_INT_MASK(bank)                    (0x000000a0 + ((bank) << 2))
+#define GPIO_INT_MSKCLR(bank)                  (0x000000c0 + ((bank) << 2))
+#define GPIO_CONTROL(bank)                     (0x00000100 + ((bank) << 2))
+#define GPIO_PWD_STATUS(bank)                  (0x00000500 + ((bank) << 2))
+
+#define GPIO_GPPWR_OFFSET                      0x00000520
+
+#define GPIO_GPCTR0_DBR_SHIFT                  5
+#define GPIO_GPCTR0_DBR_MASK                   0x000001e0
+
+#define GPIO_GPCTR0_ITR_SHIFT                  3
+#define GPIO_GPCTR0_ITR_MASK                   0x00000018
+#define GPIO_GPCTR0_ITR_CMD_RISING_EDGE                0x00000001
+#define GPIO_GPCTR0_ITR_CMD_FALLING_EDGE       0x00000002
+#define GPIO_GPCTR0_ITR_CMD_BOTH_EDGE          0x00000003
+
+#define GPIO_GPCTR0_IOTR_MASK                  0x00000001
+#define GPIO_GPCTR0_IOTR_CMD_0UTPUT            0x00000000
+#define GPIO_GPCTR0_IOTR_CMD_INPUT             0x00000001
+
+#define GPIO_GPCTR0_DB_ENABLE_MASK             0x00000100
+
+#define LOCK_CODE                              0xffffffff
+#define UNLOCK_CODE                            0x00000000
+
+struct bcm_kona_gpio {
+       void __iomem *reg_base;
+       int num_bank;
+       spinlock_t lock;
+       struct gpio_chip gpio_chip;
+       struct irq_domain *irq_domain;
+       struct bcm_kona_gpio_bank *banks;
+       struct platform_device *pdev;
+};
+
+struct bcm_kona_gpio_bank {
+       int id;
+       int irq;
+       /* Used in the interrupt handler */
+       struct bcm_kona_gpio *kona_gpio;
+};
+
+static inline struct bcm_kona_gpio *to_kona_gpio(struct gpio_chip *chip)
+{
+       return container_of(chip, struct bcm_kona_gpio, gpio_chip);
+}
+
+static void bcm_kona_gpio_set_lockcode_bank(void __iomem *reg_base,
+                                           int bank_id, int lockcode)
+{
+       writel(BCM_GPIO_PASSWD, reg_base + GPIO_GPPWR_OFFSET);
+       writel(lockcode, reg_base + GPIO_PWD_STATUS(bank_id));
+}
+
+static inline void bcm_kona_gpio_lock_bank(void __iomem *reg_base, int bank_id)
+{
+       bcm_kona_gpio_set_lockcode_bank(reg_base, bank_id, LOCK_CODE);
+}
+
+static inline void bcm_kona_gpio_unlock_bank(void __iomem *reg_base,
+                                            int bank_id)
+{
+       bcm_kona_gpio_set_lockcode_bank(reg_base, bank_id, UNLOCK_CODE);
+}
+
+static void bcm_kona_gpio_set(struct gpio_chip *chip, unsigned gpio, int value)
+{
+       struct bcm_kona_gpio *kona_gpio;
+       void __iomem *reg_base;
+       int bank_id = GPIO_BANK(gpio);
+       int bit = GPIO_BIT(gpio);
+       u32 val, reg_offset;
+       unsigned long flags;
+
+       kona_gpio = to_kona_gpio(chip);
+       reg_base = kona_gpio->reg_base;
+       spin_lock_irqsave(&kona_gpio->lock, flags);
+       bcm_kona_gpio_unlock_bank(reg_base, bank_id);
+
+       /* determine the GPIO pin direction */
+       val = readl(reg_base + GPIO_CONTROL(gpio));
+       val &= GPIO_GPCTR0_IOTR_MASK;
+
+       /* this function only applies to output pin */
+       if (GPIO_GPCTR0_IOTR_CMD_INPUT == val)
+               goto out;
+
+       reg_offset = value ? GPIO_OUT_SET(bank_id) : GPIO_OUT_CLEAR(bank_id);
+
+       val = readl(reg_base + reg_offset);
+       val |= BIT(bit);
+       writel(val, reg_base + reg_offset);
+
+out:
+       bcm_kona_gpio_lock_bank(reg_base, bank_id);
+       spin_unlock_irqrestore(&kona_gpio->lock, flags);
+}
+
+static int bcm_kona_gpio_get(struct gpio_chip *chip, unsigned gpio)
+{
+       struct bcm_kona_gpio *kona_gpio;
+       void __iomem *reg_base;
+       int bank_id = GPIO_BANK(gpio);
+       int bit = GPIO_BIT(gpio);
+       u32 val, reg_offset;
+       unsigned long flags;
+
+       kona_gpio = to_kona_gpio(chip);
+       reg_base = kona_gpio->reg_base;
+       spin_lock_irqsave(&kona_gpio->lock, flags);
+       bcm_kona_gpio_unlock_bank(reg_base, bank_id);
+
+       /* determine the GPIO pin direction */
+       val = readl(reg_base + GPIO_CONTROL(gpio));
+       val &= GPIO_GPCTR0_IOTR_MASK;
+
+       /* read the GPIO bank status */
+       reg_offset = (GPIO_GPCTR0_IOTR_CMD_INPUT == val) ?
+           GPIO_IN_STATUS(bank_id) : GPIO_OUT_STATUS(bank_id);
+       val = readl(reg_base + reg_offset);
+
+       bcm_kona_gpio_lock_bank(reg_base, bank_id);
+       spin_unlock_irqrestore(&kona_gpio->lock, flags);
+
+       /* return the specified bit status */
+       return !!(val & bit);
+}
+
+static int bcm_kona_gpio_direction_input(struct gpio_chip *chip, unsigned gpio)
+{
+       struct bcm_kona_gpio *kona_gpio;
+       void __iomem *reg_base;
+       u32 val;
+       unsigned long flags;
+       int bank_id = GPIO_BANK(gpio);
+
+       kona_gpio = to_kona_gpio(chip);
+       reg_base = kona_gpio->reg_base;
+       spin_lock_irqsave(&kona_gpio->lock, flags);
+       bcm_kona_gpio_unlock_bank(reg_base, bank_id);
+
+       val = readl(reg_base + GPIO_CONTROL(gpio));
+       val &= ~GPIO_GPCTR0_IOTR_MASK;
+       val |= GPIO_GPCTR0_IOTR_CMD_INPUT;
+       writel(val, reg_base + GPIO_CONTROL(gpio));
+
+       bcm_kona_gpio_lock_bank(reg_base, bank_id);
+       spin_unlock_irqrestore(&kona_gpio->lock, flags);
+
+       return 0;
+}
+
+static int bcm_kona_gpio_direction_output(struct gpio_chip *chip,
+                                         unsigned gpio, int value)
+{
+       struct bcm_kona_gpio *kona_gpio;
+       void __iomem *reg_base;
+       int bank_id = GPIO_BANK(gpio);
+       int bit = GPIO_BIT(gpio);
+       u32 val, reg_offset;
+       unsigned long flags;
+
+       kona_gpio = to_kona_gpio(chip);
+       reg_base = kona_gpio->reg_base;
+       spin_lock_irqsave(&kona_gpio->lock, flags);
+       bcm_kona_gpio_unlock_bank(reg_base, bank_id);
+
+       val = readl(reg_base + GPIO_CONTROL(gpio));
+       val &= ~GPIO_GPCTR0_IOTR_MASK;
+       val |= GPIO_GPCTR0_IOTR_CMD_0UTPUT;
+       writel(val, reg_base + GPIO_CONTROL(gpio));
+       reg_offset = value ? GPIO_OUT_SET(bank_id) : GPIO_OUT_CLEAR(bank_id);
+
+       val = readl(reg_base + reg_offset);
+       val |= BIT(bit);
+       writel(val, reg_base + reg_offset);
+
+       bcm_kona_gpio_lock_bank(reg_base, bank_id);
+       spin_unlock_irqrestore(&kona_gpio->lock, flags);
+
+       return 0;
+}
+
+static int bcm_kona_gpio_to_irq(struct gpio_chip *chip, unsigned gpio)
+{
+       struct bcm_kona_gpio *kona_gpio;
+
+       kona_gpio = to_kona_gpio(chip);
+       if (gpio >= kona_gpio->gpio_chip.ngpio)
+               return -ENXIO;
+       return irq_create_mapping(kona_gpio->irq_domain, gpio);
+}
+
+static int bcm_kona_gpio_set_debounce(struct gpio_chip *chip, unsigned gpio,
+                                     unsigned debounce)
+{
+       struct bcm_kona_gpio *kona_gpio;
+       void __iomem *reg_base;
+       u32 val, res;
+       unsigned long flags;
+       int bank_id = GPIO_BANK(gpio);
+
+       kona_gpio = to_kona_gpio(chip);
+       reg_base = kona_gpio->reg_base;
+       /* debounce must be 1-128ms (or 0) */
+       if ((debounce > 0 && debounce < 1000) || debounce > 128000) {
+               dev_err(chip->dev, "Debounce value %u not in range\n",
+                       debounce);
+               return -EINVAL;
+       }
+
+       /* calculate debounce bit value */
+       if (debounce != 0) {
+               /* Convert to ms */
+               debounce /= 1000;
+               /* find the MSB */
+               res = fls(debounce) - 1;
+               /* Check if MSB-1 is set (round up or down) */
+               if (res > 0 && (debounce & BIT(res - 1)))
+                       res++;
+       }
+
+       /* spin lock for read-modify-write of the GPIO register */
+       spin_lock_irqsave(&kona_gpio->lock, flags);
+       bcm_kona_gpio_unlock_bank(reg_base, bank_id);
+
+       val = readl(reg_base + GPIO_CONTROL(gpio));
+       val &= ~GPIO_GPCTR0_DBR_MASK;
+
+       if (debounce == 0) {
+               /* disable debounce */
+               val &= ~GPIO_GPCTR0_DB_ENABLE_MASK;
+       } else {
+               val |= GPIO_GPCTR0_DB_ENABLE_MASK |
+                   (res << GPIO_GPCTR0_DBR_SHIFT);
+       }
+
+       writel(val, reg_base + GPIO_CONTROL(gpio));
+
+       bcm_kona_gpio_lock_bank(reg_base, bank_id);
+       spin_unlock_irqrestore(&kona_gpio->lock, flags);
+
+       return 0;
+}
+
+static struct gpio_chip template_chip = {
+       .label = "bcm-kona-gpio",
+       .direction_input = bcm_kona_gpio_direction_input,
+       .get = bcm_kona_gpio_get,
+       .direction_output = bcm_kona_gpio_direction_output,
+       .set = bcm_kona_gpio_set,
+       .set_debounce = bcm_kona_gpio_set_debounce,
+       .to_irq = bcm_kona_gpio_to_irq,
+       .base = 0,
+};
+
+static void bcm_kona_gpio_irq_ack(struct irq_data *d)
+{
+       struct bcm_kona_gpio *kona_gpio;
+       void __iomem *reg_base;
+       int gpio = d->hwirq;
+       int bank_id = GPIO_BANK(gpio);
+       int bit = GPIO_BIT(gpio);
+       u32 val;
+       unsigned long flags;
+
+       kona_gpio = irq_data_get_irq_chip_data(d);
+       reg_base = kona_gpio->reg_base;
+       spin_lock_irqsave(&kona_gpio->lock, flags);
+       bcm_kona_gpio_unlock_bank(reg_base, bank_id);
+
+       val = readl(reg_base + GPIO_INT_STATUS(bank_id));
+       val |= BIT(bit);
+       writel(val, reg_base + GPIO_INT_STATUS(bank_id));
+
+       bcm_kona_gpio_lock_bank(reg_base, bank_id);
+       spin_unlock_irqrestore(&kona_gpio->lock, flags);
+}
+
+static void bcm_kona_gpio_irq_mask(struct irq_data *d)
+{
+       struct bcm_kona_gpio *kona_gpio;
+       void __iomem *reg_base;
+       int gpio = d->hwirq;
+       int bank_id = GPIO_BANK(gpio);
+       int bit = GPIO_BIT(gpio);
+       u32 val;
+       unsigned long flags;
+
+       kona_gpio = irq_data_get_irq_chip_data(d);
+       reg_base = kona_gpio->reg_base;
+       spin_lock_irqsave(&kona_gpio->lock, flags);
+       bcm_kona_gpio_unlock_bank(reg_base, bank_id);
+
+       val = readl(reg_base + GPIO_INT_MASK(bank_id));
+       val |= BIT(bit);
+       writel(val, reg_base + GPIO_INT_MASK(bank_id));
+
+       bcm_kona_gpio_lock_bank(reg_base, bank_id);
+       spin_unlock_irqrestore(&kona_gpio->lock, flags);
+}
+
+static void bcm_kona_gpio_irq_unmask(struct irq_data *d)
+{
+       struct bcm_kona_gpio *kona_gpio;
+       void __iomem *reg_base;
+       int gpio = d->hwirq;
+       int bank_id = GPIO_BANK(gpio);
+       int bit = GPIO_BIT(gpio);
+       u32 val;
+       unsigned long flags;
+
+       kona_gpio = irq_data_get_irq_chip_data(d);
+       reg_base = kona_gpio->reg_base;
+       spin_lock_irqsave(&kona_gpio->lock, flags);
+       bcm_kona_gpio_unlock_bank(reg_base, bank_id);
+
+       val = readl(reg_base + GPIO_INT_MSKCLR(bank_id));
+       val |= BIT(bit);
+       writel(val, reg_base + GPIO_INT_MSKCLR(bank_id));
+
+       bcm_kona_gpio_lock_bank(reg_base, bank_id);
+       spin_unlock_irqrestore(&kona_gpio->lock, flags);
+}
+
+static int bcm_kona_gpio_irq_set_type(struct irq_data *d, unsigned int type)
+{
+       struct bcm_kona_gpio *kona_gpio;
+       void __iomem *reg_base;
+       int gpio = d->hwirq;
+       u32 lvl_type;
+       u32 val;
+       unsigned long flags;
+       int bank_id = GPIO_BANK(gpio);
+
+       kona_gpio = irq_data_get_irq_chip_data(d);
+       reg_base = kona_gpio->reg_base;
+       switch (type & IRQ_TYPE_SENSE_MASK) {
+       case IRQ_TYPE_EDGE_RISING:
+               lvl_type = GPIO_GPCTR0_ITR_CMD_RISING_EDGE;
+               break;
+
+       case IRQ_TYPE_EDGE_FALLING:
+               lvl_type = GPIO_GPCTR0_ITR_CMD_FALLING_EDGE;
+               break;
+
+       case IRQ_TYPE_EDGE_BOTH:
+               lvl_type = GPIO_GPCTR0_ITR_CMD_BOTH_EDGE;
+               break;
+
+       case IRQ_TYPE_LEVEL_HIGH:
+       case IRQ_TYPE_LEVEL_LOW:
+               /* BCM GPIO doesn't support level triggering */
+       default:
+               dev_err(kona_gpio->gpio_chip.dev,
+                       "Invalid BCM GPIO irq type 0x%x\n", type);
+               return -EINVAL;
+       }
+
+       spin_lock_irqsave(&kona_gpio->lock, flags);
+       bcm_kona_gpio_unlock_bank(reg_base, bank_id);
+
+       val = readl(reg_base + GPIO_CONTROL(gpio));
+       val &= ~GPIO_GPCTR0_ITR_MASK;
+       val |= lvl_type << GPIO_GPCTR0_ITR_SHIFT;
+       writel(val, reg_base + GPIO_CONTROL(gpio));
+
+       bcm_kona_gpio_lock_bank(reg_base, bank_id);
+       spin_unlock_irqrestore(&kona_gpio->lock, flags);
+
+       return 0;
+}
+
+static void bcm_kona_gpio_irq_handler(unsigned int irq, struct irq_desc *desc)
+{
+       void __iomem *reg_base;
+       int bit, bank_id;
+       unsigned long sta;
+       struct bcm_kona_gpio_bank *bank = irq_get_handler_data(irq);
+       struct irq_chip *chip = irq_desc_get_chip(desc);
+
+       chained_irq_enter(chip, desc);
+
+       /*
+        * For bank interrupts, we can't use chip_data to store the kona_gpio
+        * pointer, since GIC needs it for its own purposes. Therefore, we get
+        * our pointer from the bank structure.
+        */
+       reg_base = bank->kona_gpio->reg_base;
+       bank_id = bank->id;
+       bcm_kona_gpio_unlock_bank(reg_base, bank_id);
+
+       while ((sta = readl(reg_base + GPIO_INT_STATUS(bank_id)) &
+                   (~(readl(reg_base + GPIO_INT_MASK(bank_id)))))) {
+               for_each_set_bit(bit, &sta, 32) {
+                       int gpio = GPIO_PER_BANK * bank_id + bit;
+                       int virq = irq_find_mapping(bank->kona_gpio->irq_domain,
+                                                   gpio);
+                       /*
+                        * Clear interrupt before handler is called so we don't
+                        * miss any interrupt occurred during executing them.
+                        */
+                       writel(readl(reg_base + GPIO_INT_STATUS(bank_id)) |
+                              BIT(bit), reg_base + GPIO_INT_STATUS(bank_id));
+                       /* Invoke interrupt handler */
+                       generic_handle_irq(virq);
+               }
+       }
+
+       bcm_kona_gpio_lock_bank(reg_base, bank_id);
+
+       chained_irq_exit(chip, desc);
+}
+
+static struct irq_chip bcm_gpio_irq_chip = {
+       .name = "bcm-kona-gpio",
+       .irq_ack = bcm_kona_gpio_irq_ack,
+       .irq_mask = bcm_kona_gpio_irq_mask,
+       .irq_unmask = bcm_kona_gpio_irq_unmask,
+       .irq_set_type = bcm_kona_gpio_irq_set_type,
+};
+
+static struct __initconst of_device_id bcm_kona_gpio_of_match[] = {
+       { .compatible = "brcm,kona-gpio" },
+       {}
+};
+
+MODULE_DEVICE_TABLE(of, bcm_kona_gpio_of_match);
+
+/*
+ * This lock class tells lockdep that GPIO irqs are in a different
+ * category than their parents, so it won't report false recursion.
+ */
+static struct lock_class_key gpio_lock_class;
+
+static int bcm_kona_gpio_irq_map(struct irq_domain *d, unsigned int irq,
+                                irq_hw_number_t hwirq)
+{
+       int ret;
+
+       ret = irq_set_chip_data(irq, d->host_data);
+       if (ret < 0)
+               return ret;
+       irq_set_lockdep_class(irq, &gpio_lock_class);
+       irq_set_chip_and_handler(irq, &bcm_gpio_irq_chip, handle_simple_irq);
+       irq_set_nested_thread(irq, 1);
+#ifdef CONFIG_ARM
+       set_irq_flags(irq, IRQF_VALID);
+#else
+       irq_set_noprobe(irq);
+#endif
+
+       return 0;
+}
+
+static void bcm_kona_gpio_irq_unmap(struct irq_domain *d, unsigned int virq)
+{
+       irq_set_chip_and_handler(virq, NULL, NULL);
+       irq_set_chip_data(virq, NULL);
+}
+
+static struct irq_domain_ops bcm_kona_irq_ops = {
+       .map = bcm_kona_gpio_irq_map,
+       .unmap = bcm_kona_gpio_irq_unmap,
+       .xlate = irq_domain_xlate_twocell,
+};
+
+static void bcm_kona_gpio_reset(struct bcm_kona_gpio *kona_gpio)
+{
+       void __iomem *reg_base;
+       int i;
+
+       reg_base = kona_gpio->reg_base;
+       /* disable interrupts and clear status */
+       for (i = 0; i < kona_gpio->num_bank; i++) {
+               bcm_kona_gpio_unlock_bank(reg_base, i);
+               writel(0xffffffff, reg_base + GPIO_INT_MASK(i));
+               writel(0xffffffff, reg_base + GPIO_INT_STATUS(i));
+               bcm_kona_gpio_lock_bank(reg_base, i);
+       }
+}
+
+static int bcm_kona_gpio_probe(struct platform_device *pdev)
+{
+       struct device *dev = &pdev->dev;
+       const struct of_device_id *match;
+       struct resource *res;
+       struct bcm_kona_gpio_bank *bank;
+       struct bcm_kona_gpio *kona_gpio;
+       struct gpio_chip *chip;
+       int ret;
+       int i;
+
+       match = of_match_device(bcm_kona_gpio_of_match, dev);
+       if (!match) {
+               dev_err(dev, "Failed to find gpio controller\n");
+               return -ENODEV;
+       }
+
+       kona_gpio = devm_kzalloc(dev, sizeof(*kona_gpio), GFP_KERNEL);
+       if (!kona_gpio)
+               return -ENOMEM;
+
+       kona_gpio->gpio_chip = template_chip;
+       chip = &kona_gpio->gpio_chip;
+       kona_gpio->num_bank = of_irq_count(dev->of_node);
+       if (kona_gpio->num_bank == 0) {
+               dev_err(dev, "Couldn't determine # GPIO banks\n");
+               return -ENOENT;
+       }
+       if (kona_gpio->num_bank > GPIO_MAX_BANK_NUM) {
+               dev_err(dev, "Too many GPIO banks configured (max=%d)\n",
+                       GPIO_MAX_BANK_NUM);
+               return -ENXIO;
+       }
+       kona_gpio->banks = devm_kzalloc(dev,
+                                       kona_gpio->num_bank *
+                                       sizeof(*kona_gpio->banks), GFP_KERNEL);
+       if (!kona_gpio->banks)
+               return -ENOMEM;
+
+       kona_gpio->pdev = pdev;
+       platform_set_drvdata(pdev, kona_gpio);
+       chip->of_node = dev->of_node;
+       chip->ngpio = kona_gpio->num_bank * GPIO_PER_BANK;
+
+       kona_gpio->irq_domain = irq_domain_add_linear(dev->of_node,
+                                                     chip->ngpio,
+                                                     &bcm_kona_irq_ops,
+                                                     kona_gpio);
+       if (!kona_gpio->irq_domain) {
+               dev_err(dev, "Couldn't allocate IRQ domain\n");
+               return -ENXIO;
+       }
+
+       res = platform_get_resource(pdev, IORESOURCE_MEM, 0);
+       kona_gpio->reg_base = devm_ioremap_resource(dev, res);
+       if (IS_ERR(kona_gpio->reg_base)) {
+               ret = -ENXIO;
+               goto err_irq_domain;
+       }
+
+       for (i = 0; i < kona_gpio->num_bank; i++) {
+               bank = &kona_gpio->banks[i];
+               bank->id = i;
+               bank->irq = platform_get_irq(pdev, i);
+               bank->kona_gpio = kona_gpio;
+               if (bank->irq < 0) {
+                       dev_err(dev, "Couldn't get IRQ for bank %d", i);
+                       ret = -ENOENT;
+                       goto err_irq_domain;
+               }
+       }
+
+       dev_info(&pdev->dev, "Setting up Kona GPIO at 0x%p (phys %#x)\n",
+                kona_gpio->reg_base, res->start);
+
+       bcm_kona_gpio_reset(kona_gpio);
+
+       ret = gpiochip_add(chip);
+       if (ret < 0) {
+               dev_err(dev, "Couldn't add GPIO chip -- %d\n", ret);
+               goto err_irq_domain;
+       }
+       for (i = 0; i < chip->ngpio; i++) {
+               int irq = bcm_kona_gpio_to_irq(chip, i);
+               irq_set_lockdep_class(irq, &gpio_lock_class);
+               irq_set_chip_and_handler(irq, &bcm_gpio_irq_chip,
+                                        handle_simple_irq);
+#ifdef CONFIG_ARM
+               set_irq_flags(irq, IRQF_VALID);
+#else
+               irq_set_noprobe(irq);
+#endif
+       }
+       for (i = 0; i < kona_gpio->num_bank; i++) {
+               bank = &kona_gpio->banks[i];
+               irq_set_chained_handler(bank->irq, bcm_kona_gpio_irq_handler);
+               irq_set_handler_data(bank->irq, bank);
+       }
+
+       spin_lock_init(&kona_gpio->lock);
+
+       return 0;
+
+err_irq_domain:
+       irq_domain_remove(kona_gpio->irq_domain);
+
+       return ret;
+}
+
+static struct platform_driver bcm_kona_gpio_driver = {
+       .driver = {
+                       .name = "bcm-kona-gpio",
+                       .owner = THIS_MODULE,
+                       .of_match_table = bcm_kona_gpio_of_match,
+       },
+       .probe = bcm_kona_gpio_probe,
+};
+
+module_platform_driver(bcm_kona_gpio_driver);
+
+MODULE_AUTHOR("Broadcom");
+MODULE_DESCRIPTION("Broadcom Kona GPIO Driver");
+MODULE_LICENSE("GPL v2");
index 8369e71..9dfe36f 100644 (file)
@@ -228,7 +228,6 @@ static int bt8xxgpio_probe(struct pci_dev *dev,
 err_release_mem:
        release_mem_region(pci_resource_start(dev, 0),
                           pci_resource_len(dev, 0));
-       pci_set_drvdata(dev, NULL);
 err_disable:
        pci_disable_device(dev);
 err_freebg:
@@ -252,7 +251,6 @@ static void bt8xxgpio_remove(struct pci_dev *pdev)
                           pci_resource_len(pdev, 0));
        pci_disable_device(pdev);
 
-       pci_set_drvdata(pdev, NULL);
        kfree(bg);
 }
 
index c6e1f08..160d759 100644 (file)
@@ -319,6 +319,7 @@ static int em_gio_probe(struct platform_device *pdev)
        }
 
        gpio_chip = &p->gpio_chip;
+       gpio_chip->of_node = pdev->dev.of_node;
        gpio_chip->direction_input = em_gio_direction_input;
        gpio_chip->get = em_gio_get;
        gpio_chip->direction_output = em_gio_direction_output;
index 8588af0..11801e9 100644 (file)
@@ -31,6 +31,10 @@ struct palmas_gpio {
        struct palmas *palmas;
 };
 
+struct palmas_device_data {
+       int ngpio;
+};
+
 static inline struct palmas_gpio *to_palmas_gpio(struct gpio_chip *chip)
 {
        return container_of(chip, struct palmas_gpio, gpio_chip);
@@ -42,23 +46,26 @@ static int palmas_gpio_get(struct gpio_chip *gc, unsigned offset)
        struct palmas *palmas = pg->palmas;
        unsigned int val;
        int ret;
+       unsigned int reg;
+       int gpio16 = (offset/8);
+
+       offset %= 8;
+       reg = (gpio16) ? PALMAS_GPIO_DATA_DIR2 : PALMAS_GPIO_DATA_DIR;
 
-       ret = palmas_read(palmas, PALMAS_GPIO_BASE, PALMAS_GPIO_DATA_DIR, &val);
+       ret = palmas_read(palmas, PALMAS_GPIO_BASE, reg, &val);
        if (ret < 0) {
-               dev_err(gc->dev, "GPIO_DATA_DIR read failed, err = %d\n", ret);
+               dev_err(gc->dev, "Reg 0x%02x read failed, %d\n", reg, ret);
                return ret;
        }
 
-       if (val & (1 << offset)) {
-               ret = palmas_read(palmas, PALMAS_GPIO_BASE,
-                                 PALMAS_GPIO_DATA_OUT, &val);
-       } else {
-               ret = palmas_read(palmas, PALMAS_GPIO_BASE,
-                                 PALMAS_GPIO_DATA_IN, &val);
-       }
+       if (val & BIT(offset))
+               reg = (gpio16) ? PALMAS_GPIO_DATA_OUT2 : PALMAS_GPIO_DATA_OUT;
+       else
+               reg = (gpio16) ? PALMAS_GPIO_DATA_IN2 : PALMAS_GPIO_DATA_IN;
+
+       ret = palmas_read(palmas, PALMAS_GPIO_BASE, reg, &val);
        if (ret < 0) {
-               dev_err(gc->dev, "GPIO_DATA_IN/OUT read failed, err = %d\n",
-                       ret);
+               dev_err(gc->dev, "Reg 0x%02x read failed, %d\n", reg, ret);
                return ret;
        }
        return !!(val & BIT(offset));
@@ -70,17 +77,20 @@ static void palmas_gpio_set(struct gpio_chip *gc, unsigned offset,
        struct palmas_gpio *pg = to_palmas_gpio(gc);
        struct palmas *palmas = pg->palmas;
        int ret;
+       unsigned int reg;
+       int gpio16 = (offset/8);
 
-       if (value)
-               ret = palmas_write(palmas, PALMAS_GPIO_BASE,
-                               PALMAS_GPIO_SET_DATA_OUT, BIT(offset));
+       offset %= 8;
+       if (gpio16)
+               reg = (value) ?
+                       PALMAS_GPIO_SET_DATA_OUT2 : PALMAS_GPIO_CLEAR_DATA_OUT2;
        else
-               ret = palmas_write(palmas, PALMAS_GPIO_BASE,
-                               PALMAS_GPIO_CLEAR_DATA_OUT, BIT(offset));
+               reg = (value) ?
+                       PALMAS_GPIO_SET_DATA_OUT : PALMAS_GPIO_CLEAR_DATA_OUT;
+
+       ret = palmas_write(palmas, PALMAS_GPIO_BASE, reg, BIT(offset));
        if (ret < 0)
-               dev_err(gc->dev, "%s write failed, err = %d\n",
-                       (value) ? "GPIO_SET_DATA_OUT" : "GPIO_CLEAR_DATA_OUT",
-                       ret);
+               dev_err(gc->dev, "Reg 0x%02x write failed, %d\n", reg, ret);
 }
 
 static int palmas_gpio_output(struct gpio_chip *gc, unsigned offset,
@@ -89,14 +99,19 @@ static int palmas_gpio_output(struct gpio_chip *gc, unsigned offset,
        struct palmas_gpio *pg = to_palmas_gpio(gc);
        struct palmas *palmas = pg->palmas;
        int ret;
+       unsigned int reg;
+       int gpio16 = (offset/8);
+
+       offset %= 8;
+       reg = (gpio16) ? PALMAS_GPIO_DATA_DIR2 : PALMAS_GPIO_DATA_DIR;
 
        /* Set the initial value */
        palmas_gpio_set(gc, offset, value);
 
-       ret = palmas_update_bits(palmas, PALMAS_GPIO_BASE,
-               PALMAS_GPIO_DATA_DIR, BIT(offset), BIT(offset));
+       ret = palmas_update_bits(palmas, PALMAS_GPIO_BASE, reg,
+                               BIT(offset), BIT(offset));
        if (ret < 0)
-               dev_err(gc->dev, "GPIO_DATA_DIR write failed, err = %d\n", ret);
+               dev_err(gc->dev, "Reg 0x%02x update failed, %d\n", reg, ret);
        return ret;
 }
 
@@ -105,11 +120,15 @@ static int palmas_gpio_input(struct gpio_chip *gc, unsigned offset)
        struct palmas_gpio *pg = to_palmas_gpio(gc);
        struct palmas *palmas = pg->palmas;
        int ret;
+       unsigned int reg;
+       int gpio16 = (offset/8);
+
+       offset %= 8;
+       reg = (gpio16) ? PALMAS_GPIO_DATA_DIR2 : PALMAS_GPIO_DATA_DIR;
 
-       ret = palmas_update_bits(palmas, PALMAS_GPIO_BASE,
-               PALMAS_GPIO_DATA_DIR, BIT(offset), 0);
+       ret = palmas_update_bits(palmas, PALMAS_GPIO_BASE, reg, BIT(offset), 0);
        if (ret < 0)
-               dev_err(gc->dev, "GPIO_DATA_DIR write failed, err = %d\n", ret);
+               dev_err(gc->dev, "Reg 0x%02x update failed, %d\n", reg, ret);
        return ret;
 }
 
@@ -121,12 +140,36 @@ static int palmas_gpio_to_irq(struct gpio_chip *gc, unsigned offset)
        return palmas_irq_get_virq(palmas, PALMAS_GPIO_0_IRQ + offset);
 }
 
+static const struct palmas_device_data palmas_dev_data = {
+       .ngpio = 8,
+};
+
+static const struct palmas_device_data tps80036_dev_data = {
+       .ngpio = 16,
+};
+
+static struct of_device_id of_palmas_gpio_match[] = {
+       { .compatible = "ti,palmas-gpio", .data = &palmas_dev_data,},
+       { .compatible = "ti,tps65913-gpio", .data = &palmas_dev_data,},
+       { .compatible = "ti,tps65914-gpio", .data = &palmas_dev_data,},
+       { .compatible = "ti,tps80036-gpio", .data = &tps80036_dev_data,},
+       { },
+};
+MODULE_DEVICE_TABLE(of, of_palmas_gpio_match);
+
 static int palmas_gpio_probe(struct platform_device *pdev)
 {
        struct palmas *palmas = dev_get_drvdata(pdev->dev.parent);
        struct palmas_platform_data *palmas_pdata;
        struct palmas_gpio *palmas_gpio;
        int ret;
+       const struct of_device_id *match;
+       const struct palmas_device_data *dev_data;
+
+       match = of_match_device(of_palmas_gpio_match, &pdev->dev);
+       dev_data = match->data;
+       if (!dev_data)
+               dev_data = &palmas_dev_data;
 
        palmas_gpio = devm_kzalloc(&pdev->dev,
                                sizeof(*palmas_gpio), GFP_KERNEL);
@@ -138,7 +181,7 @@ static int palmas_gpio_probe(struct platform_device *pdev)
        palmas_gpio->palmas = palmas;
        palmas_gpio->gpio_chip.owner = THIS_MODULE;
        palmas_gpio->gpio_chip.label = dev_name(&pdev->dev);
-       palmas_gpio->gpio_chip.ngpio = 8;
+       palmas_gpio->gpio_chip.ngpio = dev_data->ngpio;
        palmas_gpio->gpio_chip.can_sleep = 1;
        palmas_gpio->gpio_chip.direction_input = palmas_gpio_input;
        palmas_gpio->gpio_chip.direction_output = palmas_gpio_output;
@@ -172,15 +215,6 @@ static int palmas_gpio_remove(struct platform_device *pdev)
        return gpiochip_remove(&palmas_gpio->gpio_chip);
 }
 
-static struct of_device_id of_palmas_gpio_match[] = {
-       { .compatible = "ti,palmas-gpio"},
-       { .compatible = "ti,tps65913-gpio"},
-       { .compatible = "ti,tps65914-gpio"},
-       { .compatible = "ti,tps80036-gpio"},
-       { },
-};
-MODULE_DEVICE_TABLE(of, of_palmas_gpio_match);
-
 static struct platform_driver palmas_gpio_driver = {
        .driver.name    = "palmas-gpio",
        .driver.owner   = THIS_MODULE,
index 9e61bb0..54725a6 100644 (file)
@@ -28,7 +28,6 @@
 #include <linux/module.h>
 #include <linux/slab.h>
 #include <linux/spinlock.h>
-#include <linux/workqueue.h>
 
 
 static const struct i2c_device_id pcf857x_id[] = {
@@ -66,12 +65,11 @@ struct pcf857x {
        struct gpio_chip        chip;
        struct i2c_client       *client;
        struct mutex            lock;           /* protect 'out' */
-       struct work_struct      work;           /* irq demux work */
        struct irq_domain       *irq_domain;    /* for irq demux  */
        spinlock_t              slock;          /* protect irq demux */
        unsigned                out;            /* software latch */
        unsigned                status;         /* current status */
-       int                     irq;            /* real irq number */
+       unsigned                irq_mapped;     /* mapped gpio irqs */
 
        int (*write)(struct i2c_client *client, unsigned data);
        int (*read)(struct i2c_client *client);
@@ -164,48 +162,54 @@ static void pcf857x_set(struct gpio_chip *chip, unsigned offset, int value)
 static int pcf857x_to_irq(struct gpio_chip *chip, unsigned offset)
 {
        struct pcf857x *gpio = container_of(chip, struct pcf857x, chip);
+       int ret;
 
-       return irq_create_mapping(gpio->irq_domain, offset);
+       ret = irq_create_mapping(gpio->irq_domain, offset);
+       if (ret > 0)
+               gpio->irq_mapped |= (1 << offset);
+
+       return ret;
 }
 
-static void pcf857x_irq_demux_work(struct work_struct *work)
+static irqreturn_t pcf857x_irq(int irq, void *data)
 {
-       struct pcf857x *gpio = container_of(work,
-                                              struct pcf857x,
-                                              work);
+       struct pcf857x  *gpio = data;
        unsigned long change, i, status, flags;
 
        status = gpio->read(gpio->client);
 
        spin_lock_irqsave(&gpio->slock, flags);
 
-       change = gpio->status ^ status;
+       /*
+        * call the interrupt handler iff gpio is used as
+        * interrupt source, just to avoid bad irqs
+        */
+
+       change = ((gpio->status ^ status) & gpio->irq_mapped);
        for_each_set_bit(i, &change, gpio->chip.ngpio)
                generic_handle_irq(irq_find_mapping(gpio->irq_domain, i));
        gpio->status = status;
 
        spin_unlock_irqrestore(&gpio->slock, flags);
-}
-
-static irqreturn_t pcf857x_irq_demux(int irq, void *data)
-{
-       struct pcf857x  *gpio = data;
-
-       /*
-        * pcf857x can't read/write data here,
-        * since i2c data access might go to sleep.
-        */
-       schedule_work(&gpio->work);
 
        return IRQ_HANDLED;
 }
 
-static int pcf857x_irq_domain_map(struct irq_domain *domain, unsigned int virq,
+static int pcf857x_irq_domain_map(struct irq_domain *domain, unsigned int irq,
                                 irq_hw_number_t hw)
 {
-       irq_set_chip_and_handler(virq,
+       struct pcf857x *gpio = domain->host_data;
+
+       irq_set_chip_and_handler(irq,
                                 &dummy_irq_chip,
                                 handle_level_irq);
+#ifdef CONFIG_ARM
+       set_irq_flags(irq, IRQF_VALID);
+#else
+       irq_set_noprobe(irq);
+#endif
+       gpio->irq_mapped |= (1 << hw);
+
        return 0;
 }
 
@@ -218,8 +222,6 @@ static void pcf857x_irq_domain_cleanup(struct pcf857x *gpio)
        if (gpio->irq_domain)
                irq_domain_remove(gpio->irq_domain);
 
-       if (gpio->irq)
-               free_irq(gpio->irq, gpio);
 }
 
 static int pcf857x_irq_domain_init(struct pcf857x *gpio,
@@ -230,20 +232,21 @@ static int pcf857x_irq_domain_init(struct pcf857x *gpio,
        gpio->irq_domain = irq_domain_add_linear(client->dev.of_node,
                                                 gpio->chip.ngpio,
                                                 &pcf857x_irq_domain_ops,
-                                                NULL);
+                                                gpio);
        if (!gpio->irq_domain)
                goto fail;
 
        /* enable real irq */
-       status = request_irq(client->irq, pcf857x_irq_demux, 0,
-                            dev_name(&client->dev), gpio);
+       status = devm_request_threaded_irq(&client->dev, client->irq,
+                               NULL, pcf857x_irq, IRQF_ONESHOT |
+                               IRQF_TRIGGER_FALLING,
+                               dev_name(&client->dev), gpio);
+
        if (status)
                goto fail;
 
        /* enable gpio_to_irq() */
-       INIT_WORK(&gpio->work, pcf857x_irq_demux_work);
        gpio->chip.to_irq       = pcf857x_to_irq;
-       gpio->irq               = client->irq;
 
        return 0;
 
index 5c1ef2b..f2beb72 100644 (file)
@@ -73,15 +73,8 @@ static irqreturn_t acpi_gpio_irq_handler(int irq, void *data)
 static irqreturn_t acpi_gpio_irq_handler_evt(int irq, void *data)
 {
        struct acpi_gpio_evt_pin *evt_pin = data;
-       struct acpi_object_list args;
-       union acpi_object arg;
 
-       arg.type = ACPI_TYPE_INTEGER;
-       arg.integer.value = evt_pin->pin;
-       args.count = 1;
-       args.pointer = &arg;
-
-       acpi_evaluate_object(evt_pin->evt_handle, NULL, &args, NULL);
+       acpi_execute_simple_method(evt_pin->evt_handle, NULL, evt_pin->pin);
 
        return IRQ_HANDLED;
 }
index 86ef346..4fc2860 100644 (file)
@@ -102,6 +102,39 @@ static int gpiod_export_link(struct device *dev, const char *name,
 static int gpiod_sysfs_set_active_low(struct gpio_desc *desc, int value);
 static void gpiod_unexport(struct gpio_desc *desc);
 
+#ifdef CONFIG_DEBUG_FS
+#define gpiod_emerg(desc, fmt, ...)                                    \
+       pr_emerg("gpio-%d (%s): " fmt, desc_to_gpio(desc), desc->label, \
+                 ##__VA_ARGS__)
+#define gpiod_crit(desc, fmt, ...)                                     \
+       pr_crit("gpio-%d (%s): " fmt, desc_to_gpio(desc), desc->label,  \
+                 ##__VA_ARGS__)
+#define gpiod_err(desc, fmt, ...)                                      \
+       pr_err("gpio-%d (%s): " fmt, desc_to_gpio(desc), desc->label,   \
+                 ##__VA_ARGS__)
+#define gpiod_warn(desc, fmt, ...)                                     \
+       pr_warn("gpio-%d (%s): " fmt, desc_to_gpio(desc), desc->label,  \
+                 ##__VA_ARGS__)
+#define gpiod_info(desc, fmt, ...)                                     \
+       pr_info("gpio-%d (%s): " fmt, desc_to_gpio(desc), desc->label,  \
+                ##__VA_ARGS__)
+#define gpiod_dbg(desc, fmt, ...)                                 \
+       pr_debug("gpio-%d (%s): " fmt, desc_to_gpio(desc), desc->label, \
+                 ##__VA_ARGS__)
+#else
+#define gpiod_emerg(desc, fmt, ...)                               \
+       pr_emerg("gpio-%d: " fmt, desc_to_gpio(desc), ##__VA_ARGS__)
+#define gpiod_crit(desc, fmt, ...)                                \
+       pr_crit("gpio-%d: " fmt, desc_to_gpio(desc), ##__VA_ARGS__)
+#define gpiod_err(desc, fmt, ...)                                 \
+       pr_err("gpio-%d: " fmt, desc_to_gpio(desc), ##__VA_ARGS__)
+#define gpiod_warn(desc, fmt, ...)                                \
+       pr_warn("gpio-%d: " fmt, desc_to_gpio(desc), ##__VA_ARGS__)
+#define gpiod_info(desc, fmt, ...)                                \
+       pr_info("gpio-%d: " fmt, desc_to_gpio(desc), ##__VA_ARGS__)
+#define gpiod_dbg(desc, fmt, ...)                                 \
+       pr_debug("gpio-%d: " fmt, desc_to_gpio(desc), ##__VA_ARGS__)
+#endif
 
 static inline void desc_set_label(struct gpio_desc *d, const char *label)
 {
@@ -1635,8 +1668,9 @@ static int gpiod_direction_input(struct gpio_desc *desc)
 
        chip = desc->chip;
        if (!chip->get || !chip->direction_input) {
-               pr_warn("%s: missing get() or direction_input() operations\n",
-                       __func__);
+               gpiod_warn(desc,
+                       "%s: missing get() or direction_input() operations\n",
+                        __func__);
                return -EIO;
        }
 
@@ -1656,8 +1690,7 @@ static int gpiod_direction_input(struct gpio_desc *desc)
        if (status) {
                status = chip->request(chip, offset);
                if (status < 0) {
-                       pr_debug("GPIO-%d: chip request fail, %d\n",
-                               desc_to_gpio(desc), status);
+                       gpiod_dbg(desc, "chip request fail, %d\n", status);
                        /* and it's not available to anyone else ...
                         * gpio_request() is the fully clean solution.
                         */
@@ -1675,8 +1708,7 @@ lose:
 fail:
        spin_unlock_irqrestore(&gpio_lock, flags);
        if (status)
-               pr_debug("%s: gpio-%d status %d\n", __func__,
-                        desc_to_gpio(desc), status);
+               gpiod_dbg(desc, "%s status %d\n", __func__, status);
        return status;
 }
 
@@ -1708,8 +1740,9 @@ static int gpiod_direction_output(struct gpio_desc *desc, int value)
 
        chip = desc->chip;
        if (!chip->set || !chip->direction_output) {
-               pr_warn("%s: missing set() or direction_output() operations\n",
-                       __func__);
+               gpiod_warn(desc,
+                      "%s: missing set() or direction_output() operations\n",
+                      __func__);
                return -EIO;
        }
 
@@ -1729,8 +1762,7 @@ static int gpiod_direction_output(struct gpio_desc *desc, int value)
        if (status) {
                status = chip->request(chip, offset);
                if (status < 0) {
-                       pr_debug("GPIO-%d: chip request fail, %d\n",
-                               desc_to_gpio(desc), status);
+                       gpiod_dbg(desc, "chip request fail, %d\n", status);
                        /* and it's not available to anyone else ...
                         * gpio_request() is the fully clean solution.
                         */
@@ -1748,8 +1780,7 @@ lose:
 fail:
        spin_unlock_irqrestore(&gpio_lock, flags);
        if (status)
-               pr_debug("%s: gpio-%d status %d\n", __func__,
-                        desc_to_gpio(desc), status);
+               gpiod_dbg(desc, "%s: gpio status %d\n", __func__, status);
        return status;
 }
 
@@ -1781,8 +1812,9 @@ static int gpiod_set_debounce(struct gpio_desc *desc, unsigned debounce)
 
        chip = desc->chip;
        if (!chip->set || !chip->set_debounce) {
-               pr_debug("%s: missing set() or set_debounce() operations\n",
-                       __func__);
+               gpiod_dbg(desc,
+                         "%s: missing set() or set_debounce() operations\n",
+                         __func__);
                return -ENOTSUPP;
        }
 
@@ -1804,8 +1836,7 @@ static int gpiod_set_debounce(struct gpio_desc *desc, unsigned debounce)
 fail:
        spin_unlock_irqrestore(&gpio_lock, flags);
        if (status)
-               pr_debug("%s: gpio-%d status %d\n", __func__,
-                        desc_to_gpio(desc), status);
+               gpiod_dbg(desc, "%s: status %d\n", __func__, status);
 
        return status;
 }
@@ -1838,6 +1869,19 @@ EXPORT_SYMBOL_GPL(gpio_set_debounce);
  * that the GPIO was actually requested.
  */
 
+static int _gpiod_get_value(const struct gpio_desc *desc)
+{
+       struct gpio_chip        *chip;
+       int value;
+       int offset;
+
+       chip = desc->chip;
+       offset = gpio_chip_hwgpio(desc);
+       value = chip->get ? chip->get(chip, offset) : 0;
+       trace_gpio_value(desc_to_gpio(desc), 1, value);
+       return value;
+}
+
 /**
  * __gpio_get_value() - return a gpio's value
  * @gpio: gpio whose value will be returned
@@ -1849,19 +1893,11 @@ EXPORT_SYMBOL_GPL(gpio_set_debounce);
  */
 static int gpiod_get_value(const struct gpio_desc *desc)
 {
-       struct gpio_chip        *chip;
-       int value;
-       int offset;
-
        if (!desc)
                return 0;
-       chip = desc->chip;
-       offset = gpio_chip_hwgpio(desc);
        /* Should be using gpio_get_value_cansleep() */
-       WARN_ON(chip->can_sleep);
-       value = chip->get ? chip->get(chip, offset) : 0;
-       trace_gpio_value(desc_to_gpio(desc), 1, value);
-       return value;
+       WARN_ON(desc->chip->can_sleep);
+       return _gpiod_get_value(desc);
 }
 
 int __gpio_get_value(unsigned gpio)
@@ -1893,8 +1929,9 @@ static void _gpio_set_open_drain_value(struct gpio_desc *desc, int value)
        }
        trace_gpio_direction(desc_to_gpio(desc), value, err);
        if (err < 0)
-               pr_err("%s: Error in set_value for open drain gpio%d err %d\n",
-                                       __func__, desc_to_gpio(desc), err);
+               gpiod_err(desc,
+                         "%s: Error in set_value for open drain err %d\n",
+                         __func__, err);
 }
 
 /*
@@ -1920,8 +1957,23 @@ static void _gpio_set_open_source_value(struct gpio_desc *desc, int value)
        }
        trace_gpio_direction(desc_to_gpio(desc), !value, err);
        if (err < 0)
-               pr_err("%s: Error in set_value for open source gpio%d err %d\n",
-                                       __func__, desc_to_gpio(desc), err);
+               gpiod_err(desc,
+                         "%s: Error in set_value for open source err %d\n",
+                         __func__, err);
+}
+
+static void _gpiod_set_value(struct gpio_desc *desc, int value)
+{
+       struct gpio_chip        *chip;
+
+       chip = desc->chip;
+       trace_gpio_value(desc_to_gpio(desc), 0, value);
+       if (test_bit(FLAG_OPEN_DRAIN, &desc->flags))
+               _gpio_set_open_drain_value(desc, value);
+       else if (test_bit(FLAG_OPEN_SOURCE, &desc->flags))
+               _gpio_set_open_source_value(desc, value);
+       else
+               chip->set(chip, gpio_chip_hwgpio(desc), value);
 }
 
 /**
@@ -1935,20 +1987,12 @@ static void _gpio_set_open_source_value(struct gpio_desc *desc, int value)
  */
 static void gpiod_set_value(struct gpio_desc *desc, int value)
 {
-       struct gpio_chip        *chip;
 
        if (!desc)
                return;
-       chip = desc->chip;
        /* Should be using gpio_set_value_cansleep() */
-       WARN_ON(chip->can_sleep);
-       trace_gpio_value(desc_to_gpio(desc), 0, value);
-       if (test_bit(FLAG_OPEN_DRAIN, &desc->flags))
-               _gpio_set_open_drain_value(desc, value);
-       else if (test_bit(FLAG_OPEN_SOURCE, &desc->flags))
-               _gpio_set_open_source_value(desc, value);
-       else
-               chip->set(chip, gpio_chip_hwgpio(desc), value);
+       WARN_ON(desc->chip->can_sleep);
+       _gpiod_set_value(desc, value);
 }
 
 void __gpio_set_value(unsigned gpio, int value)
@@ -2013,18 +2057,10 @@ EXPORT_SYMBOL_GPL(__gpio_to_irq);
 
 static int gpiod_get_value_cansleep(const struct gpio_desc *desc)
 {
-       struct gpio_chip        *chip;
-       int value;
-       int offset;
-
        might_sleep_if(extra_checks);
        if (!desc)
                return 0;
-       chip = desc->chip;
-       offset = gpio_chip_hwgpio(desc);
-       value = chip->get ? chip->get(chip, offset) : 0;
-       trace_gpio_value(desc_to_gpio(desc), 1, value);
-       return value;
+       return _gpiod_get_value(desc);
 }
 
 int gpio_get_value_cansleep(unsigned gpio)
@@ -2035,19 +2071,10 @@ EXPORT_SYMBOL_GPL(gpio_get_value_cansleep);
 
 static void gpiod_set_value_cansleep(struct gpio_desc *desc, int value)
 {
-       struct gpio_chip        *chip;
-
        might_sleep_if(extra_checks);
        if (!desc)
                return;
-       chip = desc->chip;
-       trace_gpio_value(desc_to_gpio(desc), 0, value);
-       if (test_bit(FLAG_OPEN_DRAIN,  &desc->flags))
-               _gpio_set_open_drain_value(desc, value);
-       else if (test_bit(FLAG_OPEN_SOURCE,  &desc->flags))
-               _gpio_set_open_source_value(desc, value);
-       else
-               chip->set(chip, gpio_chip_hwgpio(desc), value);
+       _gpiod_set_value(desc, value);
 }
 
 void gpio_set_value_cansleep(unsigned gpio, int value)