gpio: dln2: Include proper header
authorLinus Walleij <linus.walleij@linaro.org>
Wed, 24 Jan 2018 12:55:22 +0000 (13:55 +0100)
committerLinus Walleij <linus.walleij@linaro.org>
Mon, 19 Mar 2018 00:50:07 +0000 (01:50 +0100)
This driver has no business including <linux/gpio.h>, it is a
driver so include <linux/gpio/driver.h>.

GPIOF_DIR_IN/GPIOF_DIR_OUT are for consumers and should not be
used in drivers to use just 1/0 instead.

Signed-off-by: Linus Walleij <linus.walleij@linaro.org>
drivers/gpio/gpio-dln2.c

index 1dada68b9a27ab7b022cb309f7fb5ba9ac0b4f41..c4e7953c093ed016b5429d2528781b03a5a92421 100644 (file)
@@ -15,7 +15,6 @@
 #include <linux/irqdomain.h>
 #include <linux/irq.h>
 #include <linux/irqchip/chained_irq.h>
-#include <linux/gpio.h>
 #include <linux/gpio/driver.h>
 #include <linux/platform_device.h>
 #include <linux/mfd/dln2.h>
@@ -204,9 +203,9 @@ static int dln2_gpio_get_direction(struct gpio_chip *chip, unsigned offset)
        struct dln2_gpio *dln2 = gpiochip_get_data(chip);
 
        if (test_bit(offset, dln2->output_enabled))
-               return GPIOF_DIR_OUT;
+               return 0;
 
-       return GPIOF_DIR_IN;
+       return 1;
 }
 
 static int dln2_gpio_get(struct gpio_chip *chip, unsigned int offset)
@@ -218,7 +217,7 @@ static int dln2_gpio_get(struct gpio_chip *chip, unsigned int offset)
        if (dir < 0)
                return dir;
 
-       if (dir == GPIOF_DIR_IN)
+       if (dir == 1)
                return dln2_gpio_pin_get_in_val(dln2, offset);
 
        return dln2_gpio_pin_get_out_val(dln2, offset);