codal_app/microbithal: Add microbit_hal_power_clear_wake_sources().

To clear all the wake up sources on every call to power.deep_sleep().

See #60 (comment)

Author: microbit-carlos

src/codal_app/microbithal.cpp

@@ -28,6 +28,8 @@
 #include "MicroBitDevice.h"
 #include "neopixel.h"
 
+#define HAL_ARRAY_SIZE(a) (sizeof(a) / sizeof((a)[0]))
+
 NRF52Pin *const pin_obj[] = {
     &uBit.io.P0,
     &uBit.io.P1,
@@ -104,12 +106,22 @@ int microbit_hal_temperature(void) {
     return uBit.thermometer.getTemperature();
 }
 
-void microbit_hal_power_wake_on_button(int button) {
-    button_obj[button]->wakeOnActive(true);
+void microbit_hal_power_clear_wake_sources(void) {
+    for (size_t i = 0; i < HAL_ARRAY_SIZE(pin_obj); ++i) {
+        microbit_hal_power_wake_on_pin(i, false);
+    }
+    for (size_t i = 0; i < HAL_ARRAY_SIZE(button_obj); ++i) {
+        microbit_hal_power_wake_on_button(i, false);
+    }
+    // TODO: Clear the run_every wake up source when implemented
+}
+
+void microbit_hal_power_wake_on_button(int button, bool wake_on_active) {
+    button_obj[button]->wakeOnActive(wake_on_active);
 }
 
-void microbit_hal_power_wake_on_pin(int pin) {
-    pin_obj[pin]->wakeOnActive(true);
+void microbit_hal_power_wake_on_pin(int pin, bool wake_on_active) {
+    pin_obj[pin]->wakeOnActive(wake_on_active);
 }
 
 void microbit_hal_power_off(void) {

src/codal_app/microbithal.h

@@ -103,8 +103,9 @@ void microbit_hal_reset(void);
 void microbit_hal_panic(int);
 int microbit_hal_temperature(void);
 
-void microbit_hal_power_wake_on_button(int button);
-void microbit_hal_power_wake_on_pin(int pin);
+void microbit_hal_power_clear_wake_sources(void);
+void microbit_hal_power_wake_on_button(int button, bool wake_on_active);
+void microbit_hal_power_wake_on_pin(int pin, bool wake_on_active);
 void microbit_hal_power_off(void);
 void microbit_hal_power_deep_sleep(bool wake_on_ms, uint32_t ms);
 

src/codal_port/modpower.c

@@ -62,11 +62,13 @@ STATIC mp_obj_t power_deep_sleep(size_t n_args, const mp_obj_t *pos_args, mp_map
     mp_arg_val_t args[MP_ARRAY_SIZE(allowed_args)];
     mp_arg_parse_all(n_args, pos_args, kw_args, MP_ARRAY_SIZE(allowed_args), allowed_args, args);
 
+    microbit_hal_power_clear_wake_sources();
+
     if (args[ARG_pins].u_obj != mp_const_none) {
         mp_obj_t *items;
         size_t len = get_array(&args[ARG_pins].u_obj, &items);
         for (size_t i = 0; i < len; ++i) {
-            microbit_hal_power_wake_on_pin(microbit_obj_get_pin_name(items[i]));
+            microbit_hal_power_wake_on_pin(microbit_obj_get_pin_name(items[i]), true);
         }
     }
 
@@ -75,9 +77,9 @@ STATIC mp_obj_t power_deep_sleep(size_t n_args, const mp_obj_t *pos_args, mp_map
         size_t len = get_array(&args[ARG_buttons].u_obj, &items);
         for (size_t i = 0; i < len; ++i) {
             if (items[i] == MP_OBJ_FROM_PTR(&microbit_button_a_obj)) {
-                microbit_hal_power_wake_on_button(0);
+                microbit_hal_power_wake_on_button(0, true);
             } else if (items[i] == MP_OBJ_FROM_PTR(&microbit_button_b_obj)) {
-                microbit_hal_power_wake_on_button(1);
+                microbit_hal_power_wake_on_button(1, true);
             } else {
                 mp_raise_ValueError(MP_ERROR_TEXT("expecting a button"));
             }