stm32/mboot: Make _estack an array to avoid compiler warnings.

dpgeorge · dpgeorge · commit c8f19f13715c · 2019-07-09T11:33:57.000+10:00
The compiler can warn about out-of-bounds array access if _estack is just a
single uint8_t.
diff --git a/ports/stm32/mboot/mboot.h b/ports/stm32/mboot/mboot.h
@@ -30,8 +30,9 @@
 // Use this to tag global static data in RAM that doesn't need to be zeroed on startup
 #define SECTION_NOZERO_BSS __attribute__((section(".nozero_bss")))
 
-#define ELEM_DATA_START (&_estack)
-#define ELEM_DATA_MAX (ELEM_DATA_START + 1024)
+#define ELEM_DATA_SIZE (1024)
+#define ELEM_DATA_START (&_estack[0])
+#define ELEM_DATA_MAX (&_estack[ELEM_DATA_SIZE])
 
 enum {
     ELEM_TYPE_END = 1,
@@ -48,7 +49,7 @@ typedef struct _fsload_bdev_t {
     uint32_t byte_len;
 } fsload_bdev_t;
 
-extern uint8_t _estack;
+extern uint8_t _estack[ELEM_DATA_SIZE];
 
 uint32_t get_le32(const uint8_t *b);
 void led_state_all(unsigned int mask);
