Merge pull request adafruit#4779 from tannewt/fix_sm_rp2040

dhalbert · web-flow · commit 1d39be4954eb · 2021-05-19T14:21:08.000-04:00
Fix safe mode on rp2040
diff --git a/.github/workflows/build.yml b/.github/workflows/build.yml
@@ -482,7 +482,7 @@ jobs:
       id: idf-cache
       with:
         path: ${{ github.workspace }}/.idf_tools
-        key: ${{ runner.os }}-idf-tools-${{ hashFiles('.git/modules/ports/esp32s2/esp-idf/HEAD') }}-20210422
+        key: ${{ runner.os }}-idf-tools-${{ hashFiles('.git/modules/ports/esp32s2/esp-idf/HEAD') }}-20210518
     - name: Clone IDF submodules
       run: |
         (cd $IDF_PATH && git submodule update --init)
diff --git a/ports/raspberrypi/supervisor/port.c b/ports/raspberrypi/supervisor/port.c
@@ -196,13 +196,13 @@ uint32_t *port_heap_get_top(void) {
     return port_stack_get_top();
 }
 
+extern uint32_t __scratch_x_start__;
 void port_set_saved_word(uint32_t value) {
-    // NOTE: This doesn't survive pressing the reset button (aka toggling RUN).
-    watchdog_hw->scratch[0] = value;
+    __scratch_x_start__ = value;
 }
 
 uint32_t port_get_saved_word(void) {
-    return watchdog_hw->scratch[0];
+    return __scratch_x_start__;
 }
 
 uint64_t port_get_raw_ticks(uint8_t *subticks) {
diff --git a/supervisor/shared/safe_mode.c b/supervisor/shared/safe_mode.c
@@ -78,7 +78,7 @@ safe_mode_t wait_for_safe_mode_reset(void) {
     #endif
     uint64_t start_ticks = supervisor_ticks_ms64();
     uint64_t diff = 0;
-    while (diff < 700) {
+    while (diff < 1000) {
         #ifdef MICROPY_HW_LED_STATUS
         // Blink on for 100, off for 100, on for 100, off for 100 and on for 200
         common_hal_digitalio_digitalinout_set_value(&status_led, diff > 100 && diff / 100 != 2 && diff / 100 != 4);

Original file line number	Diff line number	Diff line change
`@@ -196,13 +196,13 @@ uint32_t *port_heap_get_top(void) {`
`196`	`196`	`return port_stack_get_top();`
`197`	`197`	`}`
`198`	`198`
	`199`	`+extern uint32_t __scratch_x_start__;`
`199`	`200`	`void port_set_saved_word(uint32_t value) {`
`200`		`- // NOTE: This doesn't survive pressing the reset button (aka toggling RUN).`
`201`		`- watchdog_hw->scratch[0] = value;`
	`201`	`+ __scratch_x_start__ = value;`
`202`	`202`	`}`
`203`	`203`
`204`	`204`	`uint32_t port_get_saved_word(void) {`
`205`		`- return watchdog_hw->scratch[0];`
	`205`	`+ return __scratch_x_start__;`
`206`	`206`	`}`
`207`	`207`
`208`	`208`	`uint64_t port_get_raw_ticks(uint8_t *subticks) {`