From: Keith Packard Date: Sat, 29 Mar 2014 06:33:25 +0000 (-0700) Subject: altos: HAS_LED is useless; remove it X-Git-Tag: 1.3.2.2~159 X-Git-Url: https://git.gag.com/?p=fw%2Faltos;a=commitdiff_plain;h=ff4deb417a460d96645fb6934890c2c195142be3 altos: HAS_LED is useless; remove it ao_flight was trying to decide whether to turn off the red LED by checking whether HAS_LED was defined. And yet, none of the flight firmware defines that anymore, except for easymini which defines it to zero. Remove all uses and defines of this value, substituting AO_LED_RED in ao_flight.c, which has to be defined for the ao_led_off call to work. Signed-off-by: Keith Packard --- diff --git a/src/core/ao_flight.c b/src/core/ao_flight.c index 702c3403..24099347 100644 --- a/src/core/ao_flight.c +++ b/src/core/ao_flight.c @@ -143,7 +143,7 @@ ao_flight(void) ao_rdf_set(1); ao_telemetry_set_interval(AO_TELEMETRY_INTERVAL_PAD); #endif -#if HAS_LED +#if AO_LED_RED /* signal successful initialization by turning off the LED */ ao_led_off(AO_LED_RED); #endif @@ -160,7 +160,7 @@ ao_flight(void) ao_packet_slave_start(); #endif -#if HAS_LED +#if AO_LED_RED /* signal successful initialization by turning off the LED */ ao_led_off(AO_LED_RED); #endif diff --git a/src/easymini-v1.0/ao_pins.h b/src/easymini-v1.0/ao_pins.h index e721030d..6757047f 100644 --- a/src/easymini-v1.0/ao_pins.h +++ b/src/easymini-v1.0/ao_pins.h @@ -16,7 +16,6 @@ */ #define HAS_BEEP 1 -#define HAS_LED 0 #define AO_STACK_SIZE 384 diff --git a/src/lpcxpresso/ao_pins.h b/src/lpcxpresso/ao_pins.h index c0074ce2..0ffc2fad 100644 --- a/src/lpcxpresso/ao_pins.h +++ b/src/lpcxpresso/ao_pins.h @@ -16,7 +16,6 @@ */ #define HAS_BEEP 0 -#define HAS_LED 1 /* Crystal on the board */ #define AO_LPC_CLKIN 12000000 diff --git a/src/usbtrng/ao_pins.h b/src/usbtrng/ao_pins.h index 9c8fa831..b1fa6eb9 100644 --- a/src/usbtrng/ao_pins.h +++ b/src/usbtrng/ao_pins.h @@ -16,7 +16,6 @@ */ #define HAS_BEEP 0 -#define HAS_LED 1 #define AO_STACK_SIZE 384