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.lin@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;