Implement the `embedded_hal_async::delay::DelayUs` trait

esp-rs / esp-hal

no_std Hardware Abstraction Layers for ESP32 microcontrollers

https://docs.esp-rs.org/esp-hal/

Apache License 2.0

752 stars 210 forks source link

Implement the `embedded_hal_async::delay::DelayUs` trait #752

Closed jessebraham closed 5 months ago

jessebraham commented 1 year ago

This trait as added in one of the alpha releases, and we never implemented it. This trait will need to be implemented in order to use future versions of embassy.

[x] SYSTIMER (#812)
[ ] TIMG
[ ] Xtensa CPU Timers (?)

MabezDev commented 1 year ago

This trait has been around since the 0.1 release of the eha, but didn't really need to implement it as embassy-time offers an impl based on the time driver impls. If we were to implement this we should impl on the following

Systimer (alarms only)
TIMG timers
Xtensa internal CPU timers (maybe)
Anything else that has a configurable time period + interrupt? Maybe something in the RTC domain

jessebraham commented 9 months ago

Once https://github.com/esp-rs/esp-hal/issues/1063 is resolved we can revisit this issue.

jessebraham commented 5 months ago

This trait is implemented for SYSTIMER directly, and is additionally implemented for OneShotTimer; since the timer::timg module still does not contain an asynch module of its own, I'm just going to close this as "completed".