diff options
author | Lorenz Meier <lm@inf.ethz.ch> | 2014-04-30 15:33:47 +0200 |
---|---|---|
committer | Lorenz Meier <lm@inf.ethz.ch> | 2014-04-30 15:33:47 +0200 |
commit | 1dfa2f100e37d9798fe50538ed442283dd075aac (patch) | |
tree | bc0bc0273b9ead13315c357855595d12114f35a6 /src/modules/commander/commander_helper.cpp | |
parent | 2ee02e5e4b455ef8ce136d62f2985b84357ef19f (diff) | |
download | px4-firmware-1dfa2f100e37d9798fe50538ed442283dd075aac.tar.gz px4-firmware-1dfa2f100e37d9798fe50538ed442283dd075aac.tar.bz2 px4-firmware-1dfa2f100e37d9798fe50538ed442283dd075aac.zip |
commander: Stop mixing board support and high level code - just accept that non-mandatory leds may or may not be there
Diffstat (limited to 'src/modules/commander/commander_helper.cpp')
-rw-r--r-- | src/modules/commander/commander_helper.cpp | 16 |
1 files changed, 3 insertions, 13 deletions
diff --git a/src/modules/commander/commander_helper.cpp b/src/modules/commander/commander_helper.cpp index fe6c9bfaa..0fd3c9e9e 100644 --- a/src/modules/commander/commander_helper.cpp +++ b/src/modules/commander/commander_helper.cpp @@ -199,15 +199,9 @@ int led_init() } /* the blue LED is only available on FMUv1 but not FMUv2 */ -#ifdef CONFIG_ARCH_BOARD_PX4FMU_V1 - - if (ioctl(leds, LED_ON, LED_BLUE)) { - warnx("Blue LED: ioctl fail\n"); - return ERROR; - } - -#endif + (void)ioctl(leds, LED_ON, LED_BLUE); + /* we consider the amber led mandatory */ if (ioctl(leds, LED_ON, LED_AMBER)) { warnx("Amber LED: ioctl fail\n"); return ERROR; @@ -217,11 +211,7 @@ int led_init() rgbleds = open(RGBLED_DEVICE_PATH, 0); if (rgbleds == -1) { -#ifdef CONFIG_ARCH_BOARD_PX4FMU_V2 - errx(1, "Unable to open " RGBLED_DEVICE_PATH); -#else - warnx("No RGB LED found"); -#endif + warnx("No RGB LED found at " RGBLED_DEVICE_PATH); } return 0; |