Diff: hal/mbed_ticker_api.c

Revision:

181:57724642e740

Parent:

180:96ed750bd169

Child:

182:a56a73fd2a6f

--- a/hal/mbed_ticker_api.c	Wed Jan 17 15:23:54 2018 +0000
+++ b/hal/mbed_ticker_api.c	Fri Feb 16 16:09:33 2018 +0000
@@ -110,7 +110,6 @@
  */
 static void update_present_time(const ticker_data_t *const ticker)
 {
-
     ticker_event_queue_t *queue = ticker->queue;
     uint32_t ticker_time = ticker->interface->read();
     if (ticker_time == ticker->queue->tick_last_read) {
@@ -252,11 +251,16 @@
 void ticker_set_handler(const ticker_data_t *const ticker, ticker_event_handler handler)
 {
     initialize(ticker);
+
+    core_util_critical_section_enter();
     set_handler(ticker, handler);
+    core_util_critical_section_exit();
 }
 
 void ticker_irq_handler(const ticker_data_t *const ticker)
 {
+    core_util_critical_section_enter();
+
     ticker->interface->clear_interrupt();
 
     /* Go through all the pending TimerEvents */
@@ -284,6 +288,8 @@
     }
 
     schedule_interrupt(ticker);
+
+    core_util_critical_section_exit();
 }
 
 void ticker_insert_event(const ticker_data_t *const ticker, ticker_event_t *obj, timestamp_t timestamp, uint32_t id)
@@ -296,13 +302,14 @@
         ticker->queue->present_time, 
         timestamp
     );
-    core_util_critical_section_exit();
 
     // defer to ticker_insert_event_us
     ticker_insert_event_us(
         ticker, 
         obj, absolute_timestamp, id
     );
+
+    core_util_critical_section_exit();
 }
 
 void ticker_insert_event_us(const ticker_data_t *const ticker, ticker_event_t *obj, us_timestamp_t timestamp, uint32_t id)
@@ -343,6 +350,7 @@
     schedule_interrupt(ticker);
 
     core_util_critical_section_exit();
+
 }
 
 void ticker_remove_event(const ticker_data_t *const ticker, ticker_event_t *obj)
@@ -377,7 +385,11 @@
 us_timestamp_t ticker_read_us(const ticker_data_t *const ticker)
 {
     initialize(ticker);
+
+    core_util_critical_section_enter();
     update_present_time(ticker);
+    core_util_critical_section_exit();
+
     return ticker->queue->present_time;
 }