gpio: omap: simplify get() method
authorRussell King <rmk+kernel@armlinux.org.uk>
Mon, 10 Jun 2019 17:10:51 +0000 (20:10 +0300)
committerLinus Walleij <linus.walleij@linaro.org>
Wed, 12 Jun 2019 09:13:47 +0000 (11:13 +0200)
omap_gpio_get() calls omap_get_gpio_datain() or omap_get_gpio_dataout()
to read the GPIO state. These two functions are only called from this
method, so they don't add much value.  Move their contents into
omap_gpio_get() method and simplify.

Signed-off-by: Russell King <rmk+kernel@armlinux.org.uk>
Signed-off-by: Grygorii Strashko <grygorii.strashko@ti.com>
Tested-by: Tony Lindgren <tony@atomide.com>
Signed-off-by: Linus Walleij <linus.walleij@linaro.org>
drivers/gpio/gpio-omap.c

index 44a4287cce9e166942eaf493a9c4219a3178e9ce..bf1e6f1d0de7986276f1f906659760a9aff81b4c 100644 (file)
@@ -146,20 +146,6 @@ static void omap_set_gpio_dataout_mask(struct gpio_bank *bank, unsigned offset,
        bank->context.dataout = l;
 }
 
-static int omap_get_gpio_datain(struct gpio_bank *bank, int offset)
-{
-       void __iomem *reg = bank->base + bank->regs->datain;
-
-       return (readl_relaxed(reg) & (BIT(offset))) != 0;
-}
-
-static int omap_get_gpio_dataout(struct gpio_bank *bank, int offset)
-{
-       void __iomem *reg = bank->base + bank->regs->dataout;
-
-       return (readl_relaxed(reg) & (BIT(offset))) != 0;
-}
-
 /* set multiple data out values using dedicate set/clear register */
 static void omap_set_gpio_dataout_reg_multiple(struct gpio_bank *bank,
                                               unsigned long *mask,
@@ -973,14 +959,15 @@ static int omap_gpio_input(struct gpio_chip *chip, unsigned offset)
 
 static int omap_gpio_get(struct gpio_chip *chip, unsigned offset)
 {
-       struct gpio_bank *bank;
-
-       bank = gpiochip_get_data(chip);
+       struct gpio_bank *bank = gpiochip_get_data(chip);
+       void __iomem *reg;
 
        if (omap_gpio_is_input(bank, offset))
-               return omap_get_gpio_datain(bank, offset);
+               reg = bank->base + bank->regs->datain;
        else
-               return omap_get_gpio_dataout(bank, offset);
+               reg = bank->base + bank->regs->dataout;
+
+       return (readl_relaxed(reg) & BIT(offset)) != 0;
 }
 
 static int omap_gpio_output(struct gpio_chip *chip, unsigned offset, int value)