From 69639a5736aa968cf69509d8ed0adf86f4605a11 Mon Sep 17 00:00:00 2001 From: Matheus Afonso Martins Moreira Date: Sun, 25 Feb 2024 14:14:26 -0300 Subject: faces/clock: indicate low power only when needed There is no need to set the indicator on every clock tick. Indicate only when the battery is checked. --- movement/watch_faces/clock/clock_face.c | 11 ++++++----- 1 file changed, 6 insertions(+), 5 deletions(-) (limited to 'movement') diff --git a/movement/watch_faces/clock/clock_face.c b/movement/watch_faces/clock/clock_face.c index 8a396569..27d9acc0 100644 --- a/movement/watch_faces/clock/clock_face.c +++ b/movement/watch_faces/clock/clock_face.c @@ -73,6 +73,11 @@ static void clock_indicate_pm(movement_settings_t *settings, watch_date_time dat clock_indicate(WATCH_INDICATOR_PM, clock_is_pm(date_time)); } +static void clock_indicate_low_available_power(clock_state_t *clock) { + // Set the LAP indicator if battery power is low + clock_indicate(WATCH_INDICATOR_LAP, clock->battery_low); +} + static watch_date_time clock_24h_to_12h(watch_date_time date_time) { date_time.unit.hour %= 12; @@ -94,11 +99,8 @@ static void clock_check_battery_periodically(clock_state_t *clock, watch_date_ti watch_disable_adc(); clock->battery_low = voltage < CLOCK_FACE_LOW_BATTERY_VOLTAGE_THRESHOLD; -} -static void clock_indicate_low_available_power(clock_state_t *clock) { - // Set the LAP indicator if battery power is low - clock_indicate(WATCH_INDICATOR_LAP, clock->battery_low); + clock_indicate_low_available_power(clock); } static void clock_toggle_time_signal(clock_state_t *clock) { @@ -237,7 +239,6 @@ bool clock_face_loop(movement_event_t event, movement_settings_t *settings, void clock_display_clock(settings, state, current); clock_check_battery_periodically(state, current); - clock_indicate_low_available_power(state); state->date_time.previous = current; -- cgit v1.2.3