[U-Boot] [PATCH] gpio: omap_gpio: Remove check_gpio() function

[Axel Lin]

check_gpio() and gpio_is_valid() are both used to check if a gpio is valid or
not. It looks pointless to have both function because we can just call
gpio_is_valid() instead of check_gpio(). Thus remove check_gpio() function.

Signed-off-by: Axel Lin <axel....@ingics.com>
---
 drivers/gpio/omap_gpio.c | 19 +++++--------------
 1 file changed, 5 insertions(+), 14 deletions(-)

diff --git a/drivers/gpio/omap_gpio.c b/drivers/gpio/omap_gpio.c
index a30d7f0..221acc6 100644
--- a/drivers/gpio/omap_gpio.c
+++ b/drivers/gpio/omap_gpio.c
@@ -58,15 +58,6 @@ int gpio_is_valid(int gpio)
        return (gpio >= 0) && (gpio < 192);
 }
 
-static int check_gpio(int gpio)
-{
-       if (!gpio_is_valid(gpio)) {
-               printf("ERROR : check_gpio: invalid GPIO %d\n", gpio);
-               return -1;
-       }
-       return 0;
-}
-
 static void _set_gpio_direction(const struct gpio_bank *bank, int gpio,
                                int is_input)
 {
@@ -142,7 +133,7 @@ int gpio_set_value(unsigned gpio, int value)
 {
        const struct gpio_bank *bank;
 
-       if (check_gpio(gpio) < 0)
+       if (!gpio_is_valid(gpio))
                return -1;
        bank = get_gpio_bank(gpio);
        _set_gpio_dataout(bank, get_gpio_index(gpio), value);
@@ -159,7 +150,7 @@ int gpio_get_value(unsigned gpio)
        void *reg;
        int input;
 
-       if (check_gpio(gpio) < 0)
+       if (!gpio_is_valid(gpio))
                return -1;
        bank = get_gpio_bank(gpio);
        reg = bank->base;
@@ -191,7 +182,7 @@ int gpio_direction_input(unsigned gpio)
 {
        const struct gpio_bank *bank;
 
-       if (check_gpio(gpio) < 0)
+       if (!gpio_is_valid(gpio))
                return -1;
 
        bank = get_gpio_bank(gpio);
@@ -207,7 +198,7 @@ int gpio_direction_output(unsigned gpio, int value)
 {
        const struct gpio_bank *bank;
 
-       if (check_gpio(gpio) < 0)
+       if (!gpio_is_valid(gpio))
                return -1;
 
        bank = get_gpio_bank(gpio);
@@ -224,7 +215,7 @@ int gpio_direction_output(unsigned gpio, int value)
  */
 int gpio_request(unsigned gpio, const char *label)
 {
-       if (check_gpio(gpio) < 0)
+       if (!gpio_is_valid(gpio))
                return -1;
 
        return 0;
-- 
1.8.1.2



_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot