Browse Source

companion: ui: Respect LED_STATE_ON for status LED

The current logic only works for active high LEDs. Some boards need
an active low level control and therefore they set LED_STATE_ON to 0.
Take this into account and use the correct LED pattern for both cases.

Signed-off-by: Frieder Schrempf <[email protected]>
pull/1122/head
Frieder Schrempf 7 months ago
parent
commit
048bd268a1
No known key found for this signature in database GPG Key ID: E7DD51F45F833802
  1. 2
      examples/companion_radio/ui-new/UITask.cpp
  2. 2
      examples/companion_radio/ui-orig/UITask.cpp

2
examples/companion_radio/ui-new/UITask.cpp

@ -618,7 +618,7 @@ void UITask::userLedHandler() {
led_state = 0;
next_led_change = cur_time + LED_CYCLE_MILLIS - last_led_increment;
}
digitalWrite(PIN_STATUS_LED, led_state);
digitalWrite(PIN_STATUS_LED, led_state == LED_STATE_ON);
}
#endif
}

2
examples/companion_radio/ui-orig/UITask.cpp

@ -269,7 +269,7 @@ void UITask::userLedHandler() {
state = 0;
next_change = cur_time + LED_CYCLE_MILLIS - last_increment;
}
digitalWrite(PIN_STATUS_LED, state);
digitalWrite(PIN_STATUS_LED, state == LED_STATE_ON);
}
#endif
}

Loading…
Cancel
Save