diff options
| author | Michele Bini <michele.bini@gmail.com> | 2022-04-07 19:09:09 (GMT) |
|---|---|---|
| committer | Michele Bini <michele.bini@gmail.com> | 2022-04-07 19:09:09 (GMT) |
| commit | 0c08a87a68498f7f724a65bdc03547768049283e (patch) | |
| tree | efe29641eb3302314c12096b10cf1087eb8a75b0 | |
| parent | 1659bdad58734504858cdf75129e76945c12aa07 (diff) | |
Don't keep subsecond ticks
| -rw-r--r-- | src/components/datetime/DateTimeController.cpp | 14 | ||||
| -rw-r--r-- | src/components/datetime/DateTimeController.h | 1 |
2 files changed, 3 insertions, 12 deletions
diff --git a/src/components/datetime/DateTimeController.cpp b/src/components/datetime/DateTimeController.cpp index 90ce3fe..43f445c 100644 --- a/src/components/datetime/DateTimeController.cpp +++ b/src/components/datetime/DateTimeController.cpp @@ -45,17 +45,9 @@ void DateTime::SetTime( void DateTime::UpdateTime(uint32_t systickCounter) { // Handle systick counter overflow - uint32_t systickDelta = (systickCounter - previousSystickCounter) & 0xffffff; - previousSystickCounter = systickCounter; - - /* - * 1000 ms = 1024 ticks - */ - // auto newSeconds = systickDelta >> 10; - // auto rest = systickDelta & ((1 << 10)-1); - subsecondTicks += systickDelta & ((1 << 10)-1); - systickDelta = (systickDelta >> 10) + (subsecondTicks >> 10); - subsecondTicks &= ((1 << 10)-1); + uint32_t systickDelta = ( systickCounter - previousSystickCounter ) & 0xffffff; + previousSystickCounter = ( systickCounter - (systickDelta & ((1<<10)-1)) ) & 0xffffff; + systickDelta >>= 10; currentDateTime += std::chrono::seconds(systickDelta); uptime += std::chrono::seconds(systickDelta); diff --git a/src/components/datetime/DateTimeController.h b/src/components/datetime/DateTimeController.h index ca2a0fe..28bf0e5 100644 --- a/src/components/datetime/DateTimeController.h +++ b/src/components/datetime/DateTimeController.h @@ -84,7 +84,6 @@ namespace Pinetime { uint8_t hour = 0; uint8_t minute = 0; uint8_t second = 0; - uint16_t subsecondTicks = 0; uint32_t previousSystickCounter = 0; // FIXME: This could probably be 16 bits? std::chrono::time_point<std::chrono::system_clock, std::chrono::seconds> currentDateTime; |
