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
