[01/13] gpio: dln2: Include proper header

Message ID	20180319005056.7380-1-linus.walleij@linaro.org
State	Accepted
Commit	1cb6610aec83a592ca04d92d6a4fadaee5611e44
Headers	show Delivered-To: patch@linaro.org Received-SPF: pass (google.com: best guess record for domain of linux-gpio-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) client-ip=209.132.180.67; From: Linus Walleij <linus.walleij@linaro.org> To: linux-gpio@vger.kernel.org Cc: Linus Walleij <linus.walleij@linaro.org> Subject: [PATCH 01/13] gpio: dln2: Include proper header Date: Mon, 19 Mar 2018 01:50:44 +0100 Message-Id: <20180319005056.7380-1-linus.walleij@linaro.org> Sender: linux-gpio-owner@vger.kernel.org Precedence: bulk
Series	[01/13] gpio: dln2: Include proper header \| expand [01/13] gpio: dln2: Include proper header [02/13] gpio: ge: Drop of_gpio.h include [03/13] gpio: em: Use the right include [04/13] gpio: ftgpio010: Drop of_gpio.h include [05/13] gpio: grgpio: Include the right header [06/13] gpio: htc-gpio: Include the right header [07/13] gpio: ich: Include the right header [08/13] gpio: ich: Use BIT() macro [09/13] gpio: it87: Include the right header [10/13] gpio: janz-ttl: Include the right header [11/13] gpio: janz-ttl: Use BIT() macro [12/13] gpio: kempld: Include the right header [13/13] gpio: ks8695: Include the right header

Message ID

20180319005056.7380-1-linus.walleij@linaro.org

State

Accepted

Commit

1cb6610aec83a592ca04d92d6a4fadaee5611e44

Headers

Received-SPF: pass (google.com: best guess record for domain of
	linux-gpio-owner@vger.kernel.org designates 209.132.180.67 as
	permitted sender) client-ip=209.132.180.67; 
From: Linus Walleij <linus.walleij@linaro.org>
To: linux-gpio@vger.kernel.org
Cc: Linus Walleij <linus.walleij@linaro.org>
Subject: [PATCH 01/13] gpio: dln2: Include proper header
Date: Mon, 19 Mar 2018 01:50:44 +0100
Message-Id: <20180319005056.7380-1-linus.walleij@linaro.org>
Sender: linux-gpio-owner@vger.kernel.org
Precedence: bulk

Series

[01/13] gpio: dln2: Include proper header | expand

Commit Message

Linus Walleij March 19, 2018, 12:50 a.m. UTC

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 | 7 +++----
 1 file changed, 3 insertions(+), 4 deletions(-)

-- 
2.14.3

--
To unsubscribe from this list: send the line "unsubscribe linux-gpio" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

diff --git a/drivers/gpio/gpio-dln2.c b/drivers/gpio/gpio-dln2.c
index 1dada68b9a27..c4e7953c093e 100644
--- a/drivers/gpio/gpio-dln2.c
+++ b/drivers/gpio/gpio-dln2.c
@@ -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);

[01/13] gpio: dln2: Include proper header

Commit Message

Patch