TonBERRY/micropython
3
0
Fork 0
Code Pull Requests Activity

samd/mphalport: Simplify mp_hal_delay_ms().

Browse Source 
Do NOT use `mp_hal_delay_us()` for short delays.  This was initially done
to make short delays precise, but it does not allow for scheduling.  Leave
using `mp_hal_delay_us()` to user code if needed.

Signed-off-by: robert-hh <robert@hammelrath.com>
This commit is contained in:
robert-hh
2024-08-17 17:07:44 +02:00
committed by Damien George
parent ed86fdbdf6
commit 1a6279ba37
2 changed files with 5 additions and 7 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
docs/samd/quickref.rst
View File

@@ -65,6 +65,8 @@ Use the :mod:`time <time>` module::
start = time.ticks_ms() # get millisecond counter start = time.ticks_ms() # get millisecond counter
delta = time.ticks_diff(time.ticks_ms(), start) # compute time difference delta = time.ticks_diff(time.ticks_ms(), start) # compute time difference
Note that :func:`time.sleep_us()` delays by busy waiting. During that time, other tasks are
not scheduled.
Clock and time Clock and time
-------------- --------------

10
ports/samd/mphalport.c
View File

@@ -69,13 +69,9 @@ void mp_hal_clr_pin_mux(mp_hal_pin_obj_t pin) {
} }
void mp_hal_delay_ms(mp_uint_t ms) { void mp_hal_delay_ms(mp_uint_t ms) {
if (ms > 10) { uint32_t t0 = systick_ms;
uint32_t t0 = systick_ms; while (systick_ms - t0 < ms) {
while (systick_ms - t0 < ms) { MICROPY_EVENT_POLL_HOOK
MICROPY_EVENT_POLL_HOOK
}
} else {
mp_hal_delay_us(ms * 1000);
} }
} }