]> git.sur5r.net Git - u-boot/commitdiff
ARM: at91: gpio: fix at91_set_gpio_value() define
authorWenyou Yang <wenyou.yang@atmel.com>
Thu, 23 Mar 2017 04:55:20 +0000 (12:55 +0800)
committerSimon Glass <sjg@chromium.org>
Thu, 13 Apr 2017 20:44:49 +0000 (14:44 -0600)
When the CONFIG_ATMEL_LEGACY is undefined, according to the following
defines, at91_set_gpio_value() references to at91_set_pio_value(x, y)
with two parameters.
 #define at91_set_gpio_value(x, y)      at91_set_pio_value(x, y)
 #define at91_get_gpio_value(x)         at91_get_pio_value(x)

But there isn't the implementation of at91_set_pio_value(x, y) with
two parameters in U-Boot. This is an error.

Same as at91_get_gpio_value(x) define.

Signed-off-by: Wenyou Yang <wenyou.yang@atmel.com>
Reviewed-by: Simon Glass <sjg@chromium.org>
arch/arm/mach-at91/include/mach/gpio.h

index 5a32bdba8d83a4fdf747ec2a8639a5c2bf51147c..df0f71975ad85833e4cf2abbe171bf41b432a78f 100644 (file)
@@ -223,15 +223,13 @@ static inline unsigned pin_to_mask(unsigned pin)
        at91_set_pio_output((x - PIN_BASE) / 32,(x % 32), y)
 #define at91_set_gpio_input(x, y) \
        at91_set_pio_input((x - PIN_BASE) / 32,(x % 32), y)
-#define at91_set_gpio_value(x, y) \
-       at91_set_pio_value((x - PIN_BASE) / 32,(x % 32), y)
-#define at91_get_gpio_value(x) \
-       at91_get_pio_value((x - PIN_BASE) / 32,(x % 32))
-#else
-#define at91_set_gpio_value(x, y)      at91_set_pio_value(x, y)
-#define at91_get_gpio_value(x)         at91_get_pio_value(x)
 #endif
 
+#define at91_set_gpio_value(x, y) \
+       at91_set_pio_value((x / 32), (x % 32), y)
+#define at91_get_gpio_value(x) \
+       at91_get_pio_value((x / 32), (x % 32))
+
 #define GPIO_PIOA_BASE  (0)
 #define GPIO_PIOB_BASE  (GPIO_PIOA_BASE + 32)
 #define GPIO_PIOC_BASE  (GPIO_PIOB_BASE + 32)