displayio: Pause rgbmatrix during soft-reset

jepler · jepler · commit 0d7f257eae42 · 2021-05-26T11:18:02.000-05:00
We can't handle rgbmatrix's interrupts from here until the display is
reinitialized, so set the display as paused.

With this change, I can survive multiple cycles with wifi+rgbmatrix
on an esp32s2.  Before, it usually failed.
diff --git a/shared-module/displayio/__init__.c b/shared-module/displayio/__init__.c
@@ -185,6 +185,8 @@ void reset_displays(void) {
             rgbmatrix_rgbmatrix_obj_t *pm = &displays[i].rgbmatrix;
             if (!any_display_uses_this_framebuffer(&pm->base)) {
                 common_hal_rgbmatrix_rgbmatrix_deinit(pm);
+            } else {
+                common_hal_rgbmatrix_rgbmatrix_set_paused(pm, true);
             }
         #endif
         #if CIRCUITPY_SHARPDISPLAY

Original file line number	Diff line number	Diff line change
`@@ -185,6 +185,8 @@ void reset_displays(void) {`
`185`	`185`	`rgbmatrix_rgbmatrix_obj_t *pm = &displays[i].rgbmatrix;`
`186`	`186`	`if (!any_display_uses_this_framebuffer(&pm->base)) {`
`187`	`187`	`common_hal_rgbmatrix_rgbmatrix_deinit(pm);`
	`188`	`+ } else {`
	`189`	`+ common_hal_rgbmatrix_rgbmatrix_set_paused(pm, true);`
`188`	`190`	`}`
`189`	`191`	`#endif`
`190`	`192`	`#if CIRCUITPY_SHARPDISPLAY`