From 69639a5736aa968cf69509d8ed0adf86f4605a11 Mon Sep 17 00:00:00 2001
From: Matheus Afonso Martins Moreira <matheus.a.m.moreira@gmail.com>
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(-)
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