Re: [PATCH v2] gpio: tegra: Include the right header

[Dmitry Osipenko <digetx@xxxxxxxxx>]

On Tuesday 07 August 2018 02:04:39 Linus Walleij wrote:
> This is a GPIO driver so include only <linux/gpio/driver.h>.
> Drop the use of GPIOF_* flags: these are for consumers, not
> drivers. Just return 0/1.
>
> Cc: Stefan Agner <stefan@xxxxxxxx>
> Cc: Dmitry Osipenko <digetx@xxxxxxxxx>
> Cc: Thierry Reding <treding@xxxxxxxxxx>
> Signed-off-by: Linus Walleij <linus.walleij@xxxxxxxxxx>
> ---
> ChangeLog v1->v2:
> - Also get rid of GPIOF_* usage
> ---
>  drivers/gpio/gpio-tegra.c | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/drivers/gpio/gpio-tegra.c b/drivers/gpio/gpio-tegra.c
> index 2d940785bad0..8e5f3150c6af 100644
> --- a/drivers/gpio/gpio-tegra.c
> +++ b/drivers/gpio/gpio-tegra.c
> @@ -22,7 +22,7 @@
>  #include <linux/irq.h>
>  #include <linux/interrupt.h>
>  #include <linux/io.h>
> -#include <linux/gpio.h>
> +#include <linux/gpio/driver.h>
>  #include <linux/of_device.h>
>  #include <linux/platform_device.h>
>  #include <linux/module.h>
> @@ -207,7 +207,7 @@ static int tegra_gpio_get_direction(struct gpio_chip
> *chip,
>
>  	oe = tegra_gpio_readl(tgi, GPIO_OE(tgi, offset));
>
> -	return (oe & pin_mask) ? GPIOF_DIR_OUT : GPIOF_DIR_IN;
> +	return !(oe & pin_mask);
>  }
>
>  static int tegra_gpio_set_debounce(struct gpio_chip *chip, unsigned int
> offset,

Reviewed-by: Dmitry Osipenko <digetx@xxxxxxxxx>
--
To unsubscribe from this list: send the line "unsubscribe linux-gpio" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html