altos: Fix LPC LED driver
authorKeith Packard <keithp@keithp.com>
Mon, 10 Mar 2014 03:59:43 +0000 (20:59 -0700)
committerKeith Packard <keithp@keithp.com>
Mon, 10 Mar 2014 03:59:43 +0000 (20:59 -0700)
Was using wrong types

Signed-off-by: Keith Packard <keithp@keithp.com>
src/lpc/ao_arch.h
src/lpc/ao_led_lpc.c

index d04bf2c8b06d210a59b39ce39653fdcf14df0304..9dcfadc03031e162d699d9a4a6ce11d4b7b01ff3 100644 (file)
 #define AO_STACK_SIZE  512
 #endif
 
-#define AO_LED_TYPE    uint16_t
-
 #define AO_PORT_TYPE   uint32_t
 
+#define AO_LED_TYPE    AO_PORT_TYPE
+
 #ifndef AO_TICK_TYPE
 #define AO_TICK_TYPE   uint16_t
 #define AO_TICK_SIGNED int16_t
index 7bef51ba25cdce0e07b05b0bae002395fc940d66..d983437cd4c4549c0a64886b434d8e61f2aebe7b 100644 (file)
 
 #include <ao.h>
 
-__pdata uint16_t ao_led_enable;
+__pdata AO_PORT_TYPE ao_led_enable;
 
 void
-ao_led_on(uint16_t colors)
+ao_led_on(AO_PORT_TYPE colors)
 {
        lpc_gpio.pin[LED_PORT] |= colors;
 }
 
 void
-ao_led_off(uint16_t colors)
+ao_led_off(AO_PORT_TYPE colors)
 {
        lpc_gpio.pin[LED_PORT] &= ~colors;
 }
 
 void
-ao_led_set(uint16_t colors)
+ao_led_set(AO_PORT_TYPE colors)
 {
-       uint16_t        on = colors & ao_led_enable;
-       uint16_t        off = ~colors & ao_led_enable;
+       AO_PORT_TYPE    on = colors & ao_led_enable;
+       AO_PORT_TYPE    off = ~colors & ao_led_enable;
 
        ao_led_off(off);
        ao_led_on(on);
 }
 
 void
-ao_led_toggle(uint16_t colors)
+ao_led_toggle(AO_PORT_TYPE colors)
 {
        lpc_gpio.pin[LED_PORT] ^= colors;
 }
 
 void
-ao_led_for(uint16_t colors, uint16_t ticks) __reentrant
+ao_led_for(AO_PORT_TYPE colors, uint16_t ticks) __reentrant
 {
        ao_led_on(colors);
        ao_delay(ticks);
@@ -56,10 +56,8 @@ ao_led_for(uint16_t colors, uint16_t ticks) __reentrant
 }
 
 void
-ao_led_init(uint16_t enable)
+ao_led_init(AO_PORT_TYPE enable)
 {
-       int     bit;
-
        ao_led_enable = enable;
        lpc_scb.sysahbclkctrl |= (1 << LPC_SCB_SYSAHBCLKCTRL_GPIO);
        lpc_gpio.dir[LED_PORT] |= enable;