gpio: ath79: Move to the generic GPIO driver
authorAlban Bedel <albeu@free.fr>
Thu, 28 Jan 2016 19:44:29 +0000 (20:44 +0100)
committerLinus Walleij <linus.walleij@linaro.org>
Wed, 10 Feb 2016 10:07:24 +0000 (11:07 +0100)
Drop most of the code in favor of the generic MMIO GPIO driver.
As the driver now depend on CONFIG_GPIO_GENERIC also add a Kconfig
entry to make the driver optional.

We leave the base pointer and lock in the data struct because they are
needed for the IRQ support.

Signed-off-by: Alban Bedel <albeu@free.fr>
Acked-by: Alexandre Courbot <acourbot@nvidia.com>
Signed-off-by: Linus Walleij <linus.walleij@linaro.org>
drivers/gpio/Kconfig
drivers/gpio/Makefile
drivers/gpio/gpio-ath79.c

index 77df86635480e690d118b3a7fe94773f9d3ab33d..020a44a5233161fe245a454b208ae08c36e90f0d 100644 (file)
@@ -126,6 +126,15 @@ config GPIO_AMDPT
          driver for GPIO functionality on Promontory IOHub
          Require ACPI ASL code to enumerate as a platform device.
 
+config GPIO_ATH79
+       tristate "Atheros AR71XX/AR724X/AR913X GPIO support"
+       default y if ATH79
+       depends on ATH79
+       select GPIO_GENERIC
+       help
+         Select this option to enable GPIO driver for
+         Atheros AR71XX/AR724X/AR913X SoC devices.
+
 config GPIO_BCM_KONA
        bool "Broadcom Kona GPIO"
        depends on OF_GPIO && (ARCH_BCM_MOBILE || COMPILE_TEST)
index 2f5852daab0d4634cabe73575c8984fd20ac077d..30ff7de0f6d5178f3b9b2d27f6319ecfecd5cc3d 100644 (file)
@@ -24,7 +24,7 @@ obj-$(CONFIG_GPIO_ALTERA)     += gpio-altera.o
 obj-$(CONFIG_GPIO_AMD8111)     += gpio-amd8111.o
 obj-$(CONFIG_GPIO_AMDPT)       += gpio-amdpt.o
 obj-$(CONFIG_GPIO_ARIZONA)     += gpio-arizona.o
-obj-$(CONFIG_ATH79)            += gpio-ath79.o
+obj-$(CONFIG_GPIO_ATH79)       += gpio-ath79.o
 obj-$(CONFIG_GPIO_BCM_KONA)    += gpio-bcm-kona.o
 obj-$(CONFIG_GPIO_BRCMSTB)     += gpio-brcmstb.o
 obj-$(CONFIG_GPIO_BT8XX)       += gpio-bt8xx.o
index d13dd133a907e64cbce5dab44e99a3cf88327e41..13d96480a3b2d399e98594bede2dbd21c83bb683 100644 (file)
 #include <asm/mach-ath79/ar71xx_regs.h>
 
 struct ath79_gpio_ctrl {
-       struct gpio_chip chip;
+       struct gpio_chip gc;
        void __iomem *base;
        spinlock_t lock;
 };
 
-static void ath79_gpio_set_value(struct gpio_chip *chip,
-                               unsigned gpio, int value)
-{
-       struct ath79_gpio_ctrl *ctrl = gpiochip_get_data(chip);
-
-       if (value)
-               __raw_writel(BIT(gpio), ctrl->base + AR71XX_GPIO_REG_SET);
-       else
-               __raw_writel(BIT(gpio), ctrl->base + AR71XX_GPIO_REG_CLEAR);
-}
-
-static int ath79_gpio_get_value(struct gpio_chip *chip, unsigned gpio)
-{
-       struct ath79_gpio_ctrl *ctrl = gpiochip_get_data(chip);
-
-       return (__raw_readl(ctrl->base + AR71XX_GPIO_REG_IN) >> gpio) & 1;
-}
-
-static int ath79_gpio_direction_input(struct gpio_chip *chip,
-                                      unsigned offset)
-{
-       struct ath79_gpio_ctrl *ctrl = gpiochip_get_data(chip);
-       unsigned long flags;
-
-       spin_lock_irqsave(&ctrl->lock, flags);
-
-       __raw_writel(
-               __raw_readl(ctrl->base + AR71XX_GPIO_REG_OE) & ~BIT(offset),
-               ctrl->base + AR71XX_GPIO_REG_OE);
-
-       spin_unlock_irqrestore(&ctrl->lock, flags);
-
-       return 0;
-}
-
-static int ath79_gpio_direction_output(struct gpio_chip *chip,
-                                       unsigned offset, int value)
-{
-       struct ath79_gpio_ctrl *ctrl = gpiochip_get_data(chip);
-       unsigned long flags;
-
-       spin_lock_irqsave(&ctrl->lock, flags);
-
-       if (value)
-               __raw_writel(BIT(offset), ctrl->base + AR71XX_GPIO_REG_SET);
-       else
-               __raw_writel(BIT(offset), ctrl->base + AR71XX_GPIO_REG_CLEAR);
-
-       __raw_writel(
-               __raw_readl(ctrl->base + AR71XX_GPIO_REG_OE) | BIT(offset),
-               ctrl->base + AR71XX_GPIO_REG_OE);
-
-       spin_unlock_irqrestore(&ctrl->lock, flags);
-
-       return 0;
-}
-
-static int ar934x_gpio_direction_input(struct gpio_chip *chip, unsigned offset)
-{
-       struct ath79_gpio_ctrl *ctrl = gpiochip_get_data(chip);
-       unsigned long flags;
-
-       spin_lock_irqsave(&ctrl->lock, flags);
-
-       __raw_writel(
-               __raw_readl(ctrl->base + AR71XX_GPIO_REG_OE) | BIT(offset),
-               ctrl->base + AR71XX_GPIO_REG_OE);
-
-       spin_unlock_irqrestore(&ctrl->lock, flags);
-
-       return 0;
-}
-
-static int ar934x_gpio_direction_output(struct gpio_chip *chip, unsigned offset,
-                                       int value)
-{
-       struct ath79_gpio_ctrl *ctrl = gpiochip_get_data(chip);
-       unsigned long flags;
-
-       spin_lock_irqsave(&ctrl->lock, flags);
-
-       if (value)
-               __raw_writel(BIT(offset), ctrl->base + AR71XX_GPIO_REG_SET);
-       else
-               __raw_writel(BIT(offset), ctrl->base + AR71XX_GPIO_REG_CLEAR);
-
-       __raw_writel(
-               __raw_readl(ctrl->base + AR71XX_GPIO_REG_OE) & ~BIT(offset),
-               ctrl->base + AR71XX_GPIO_REG_OE);
-
-       spin_unlock_irqrestore(&ctrl->lock, flags);
-
-       return 0;
-}
-
-static const struct gpio_chip ath79_gpio_chip = {
-       .label                  = "ath79",
-       .get                    = ath79_gpio_get_value,
-       .set                    = ath79_gpio_set_value,
-       .direction_input        = ath79_gpio_direction_input,
-       .direction_output       = ath79_gpio_direction_output,
-       .base                   = 0,
-};
-
 static const struct of_device_id ath79_gpio_of_match[] = {
        { .compatible = "qca,ar7100-gpio" },
        { .compatible = "qca,ar9340-gpio" },
@@ -174,15 +70,21 @@ static int ath79_gpio_probe(struct platform_device *pdev)
                return -ENOMEM;
 
        spin_lock_init(&ctrl->lock);
-       memcpy(&ctrl->chip, &ath79_gpio_chip, sizeof(ctrl->chip));
-       ctrl->chip.parent = &pdev->dev;
-       ctrl->chip.ngpio = ath79_gpio_count;
-       if (oe_inverted) {
-               ctrl->chip.direction_input = ar934x_gpio_direction_input;
-               ctrl->chip.direction_output = ar934x_gpio_direction_output;
+       err = bgpio_init(&ctrl->gc, &pdev->dev, 4,
+                       ctrl->base + AR71XX_GPIO_REG_IN,
+                       ctrl->base + AR71XX_GPIO_REG_SET,
+                       ctrl->base + AR71XX_GPIO_REG_CLEAR,
+                       oe_inverted ? NULL : ctrl->base + AR71XX_GPIO_REG_OE,
+                       oe_inverted ? ctrl->base + AR71XX_GPIO_REG_OE : NULL,
+                       0);
+       if (err) {
+               dev_err(&pdev->dev, "bgpio_init failed\n");
+               return err;
        }
+       /* Use base 0 to stay compatible with legacy platforms */
+       ctrl->gc.base = 0;
 
-       err = gpiochip_add_data(&ctrl->chip, ctrl);
+       err = gpiochip_add_data(&ctrl->gc, ctrl);
        if (err) {
                dev_err(&pdev->dev,
                        "cannot add AR71xx GPIO chip, error=%d", err);