gpio: dwapb: fix bgpio usage
authorLinus Walleij <linus.walleij@linaro.org>
Fri, 20 Oct 2017 10:26:51 +0000 (12:26 +0200)
committerLinus Walleij <linus.walleij@linaro.org>
Wed, 25 Oct 2017 09:25:10 +0000 (11:25 +0200)
The DW APB GPIO driver uses the generic GPIO library gpio-mmio,
and initialize the flags as "false", which should be 0.

When no flags are given, the native endianness is used to access
the MMIO registers, and the pin2mask() call can simply be
converted to a BIT() call, as per the default pin2mask()
implementation in gpio-mmio.c.

Acked-by: Alan Tull <atull@kernel.org>
Acked-by: Hoan Tran <hotran@apm.com>
Signed-off-by: Linus Walleij <linus.walleij@linaro.org>
drivers/gpio/gpio-dwapb.c

index d782ad195c891f9a367630f0af4235b7b3a424c6..6730c6642ce315c3da38c13a10fcae1c01601f32 100644 (file)
@@ -320,7 +320,7 @@ static int dwapb_gpio_set_debounce(struct gpio_chip *gc,
        struct dwapb_gpio_port *port = gpiochip_get_data(gc);
        struct dwapb_gpio *gpio = port->gpio;
        unsigned long flags, val_deb;
-       unsigned long mask = gc->pin2mask(gc, offset);
+       unsigned long mask = BIT(offset);
 
        spin_lock_irqsave(&gc->bgpio_lock, flags);
 
@@ -482,7 +482,7 @@ static int dwapb_gpio_add_port(struct dwapb_gpio *gpio,
                (pp->idx * GPIO_SWPORT_DDR_SIZE);
 
        err = bgpio_init(&port->gc, gpio->dev, 4, dat, set, NULL, dirout,
-                        NULL, false);
+                        NULL, 0);
        if (err) {
                dev_err(gpio->dev, "failed to init gpio chip for port%d\n",
                        port->idx);