extmod/utime_mphal: Don't exit/enter the GIL in generic sleep functions.

dpgeorge · dpgeorge · commit 2e3fc778094c · 2017-03-22T12:49:21.000+11:00
GIL behaviour should be handled by the port.  And ports probably want to
define sleep_us so that it doesn't release the GIL, to improve timing
accuracy.
diff --git a/extmod/utime_mphal.c b/extmod/utime_mphal.c
@@ -37,23 +37,19 @@
 #include "extmod/utime_mphal.h"
 
 STATIC mp_obj_t time_sleep(mp_obj_t seconds_o) {
-    MP_THREAD_GIL_EXIT();
     #if MICROPY_PY_BUILTINS_FLOAT
     mp_hal_delay_ms(1000 * mp_obj_get_float(seconds_o));
     #else
     mp_hal_delay_ms(1000 * mp_obj_get_int(seconds_o));
     #endif
-    MP_THREAD_GIL_ENTER();
     return mp_const_none;
 }
 MP_DEFINE_CONST_FUN_OBJ_1(mp_utime_sleep_obj, time_sleep);
 
 STATIC mp_obj_t time_sleep_ms(mp_obj_t arg) {
     mp_int_t ms = mp_obj_get_int(arg);
     if (ms > 0) {
-        MP_THREAD_GIL_EXIT();
         mp_hal_delay_ms(ms);
-        MP_THREAD_GIL_ENTER();
     }
     return mp_const_none;
 }
@@ -62,9 +58,7 @@ MP_DEFINE_CONST_FUN_OBJ_1(mp_utime_sleep_ms_obj, time_sleep_ms);
 STATIC mp_obj_t time_sleep_us(mp_obj_t arg) {
     mp_int_t us = mp_obj_get_int(arg);
     if (us > 0) {
-        MP_THREAD_GIL_EXIT();
         mp_hal_delay_us(us);
-        MP_THREAD_GIL_ENTER();
     }
     return mp_const_none;
 }
