embassy-rs / embassy

Modern embedded framework, using Rust and async.
https://embassy.dev
Apache License 2.0
5.64k stars 781 forks source link

Only access the current time once per polling #3550

Closed bugadani closed 5 days ago

bugadani commented 1 week ago

I'm not 100% sure this changes anything - now impls are relatively simple, and ESP32, where it is not, doesn't use the repoll-if-set-alarm-returns-false mechanism (but that may change and changing it makes sense).

bugadani commented 5 days ago

This is a net loss when there are no timers running.