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

Axel Lin axel.lin at ingics.com
Thu Jun 20 03:03:09 CEST 2013


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 at 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





More information about the U-Boot mailing list