adds --task-stack-size build argument.

This sets the stack size for a FreeRTOS task. We ran into an issue with the HX8369 driver where there was not enough stack memory to pass some of the init parameters. This allows the user to adjust the stack size if they encounter this issue.

Author: kdschlosser

api_drivers/common_api_drivers/display/hx8369/_hx8369_init_type1.py

@@ -27,7 +27,7 @@ def init(self):
     param_buf[:19] = bytearray([
         0x01, 0x00, 0x34, 0x06, 0x00, 0x0f, 0x0f, 0x2a, 0x32, 0x3f,
         0x3f, 0x07, 0x23, 0x01, 0xe6, 0xe6, 0xe6, 0xe6, 0xe6])
-    
+
     self.set_params(0xB1, param_mv[:19])
 
     # SET Display  480x800
@@ -63,24 +63,21 @@ def init(self):
     self.set_params(0xE0, param_mv[:34])
 
     # Set DGC
-    param_buf[:64] = bytearray([
+    param_buf[:127] = bytearray([
         0x01, 0x02, 0x08, 0x12, 0x1a, 0x22, 0x2a, 0x31, 0x36,
         0x3f, 0x48, 0x51, 0x58, 0x60, 0x68, 0x70, 0x78, 0x80,
         0x88, 0x90, 0x98, 0xa0, 0xa7, 0xaf, 0xb6, 0xbe, 0xc7,
         0xce, 0xd6, 0xde, 0xe6, 0xef, 0xf5, 0xfb, 0xfc, 0xfe,
         0x8c, 0xa4, 0x19, 0xec, 0x1b, 0x4c, 0x40, 0x02, 0x08,
         0x12, 0x1a, 0x22, 0x2a, 0x31, 0x36, 0x3f, 0x48, 0x51,
         0x58, 0x60, 0x68, 0x70, 0x78, 0x80, 0x88, 0x90, 0x98,
-        0xa0])
-
-    param_buf[64:127] = bytearray([
-        0xa7, 0xaf, 0xb6, 0xbe, 0xc7, 0xce, 0xd6, 0xde, 0xe6,
-        0xef, 0xf5, 0xfb, 0xfc, 0xfe, 0x8c, 0xa4, 0x19, 0xec,
-        0x1b, 0x4c, 0x40, 0x02, 0x08, 0x12, 0x1a, 0x22, 0x2a,
-        0x31, 0x36, 0x3f, 0x48, 0x51, 0x58, 0x60, 0x68, 0x70,
-        0x78, 0x80, 0x88, 0x90, 0x98, 0xa0, 0xa7, 0xaf, 0xb6,
-        0xbe, 0xc7, 0xce, 0xd6, 0xde, 0xe6, 0xef, 0xf5, 0xfb,
-        0xfc, 0xfe, 0x8c, 0xa4, 0x19, 0xec, 0x1b, 0x4c, 0x40])
+        0xa0, 0xa7, 0xaf, 0xb6, 0xbe, 0xc7, 0xce, 0xd6, 0xde,
+        0xe6, 0xef, 0xf5, 0xfb, 0xfc, 0xfe, 0x8c, 0xa4, 0x19,
+        0xec, 0x1b, 0x4c, 0x40, 0x02, 0x08, 0x12, 0x1a, 0x22,
+        0x2a, 0x31, 0x36, 0x3f, 0x48, 0x51, 0x58, 0x60, 0x68,
+        0x70, 0x78, 0x80, 0x88, 0x90, 0x98, 0xa0, 0xa7, 0xaf,
+        0xb6, 0xbe, 0xc7, 0xce, 0xd6, 0xde, 0xe6, 0xef, 0xf5,
+        0xfb, 0xfc, 0xfe, 0x8c, 0xa4, 0x19, 0xec, 0x1b, 0x4c, 0x40])
 
     self.set_params(0xC1, param_mv[:127])
 

builder/esp32.py

@@ -162,6 +162,7 @@ def get_espidf():
 ota = False
 
 dual_core_threads = False
+task_stack_size = 16 * 1024
 
 
 def common_args(extra_args):
@@ -177,6 +178,7 @@ def common_args(extra_args):
     global optimize_size
     global ota
     global dual_core_threads
+    global task_stack_size
 
     if board == 'ARDUINO_NANO_ESP32':
         raise RuntimeError('Board is not currently supported')
@@ -269,6 +271,21 @@ def common_args(extra_args):
         action='store_true'
     )
 
+    def multiple_of_16(value):
+        value = int(value)
+        if value % 1024:
+            raise ValueError('task stack size must be a multiple of 1024')
+
+        return value
+
+    esp_argParser.add_argument(
+        '--task-stack-size',
+        dest='task_stack_size',
+        default=task_stack_size,
+        type=multiple_of_16,
+        action='store'
+    )
+
     esp_args, extra_args = esp_argParser.parse_known_args(extra_args)
 
     BAUD = esp_args.baud
@@ -282,6 +299,7 @@ def common_args(extra_args):
     optimize_size = esp_args.optimize_size
     ota = esp_args.ota
     dual_core_threads = esp_args.dual_core_threads
+    task_stack_size = esp_args.task_stack_size
 
     return extra_args
 
@@ -833,6 +851,24 @@ def set_thread_core():
             data = data.replace(pattern, text)
             break
 
+    data = data.split('\n')
+    for i, line in enumerate(data[:]):
+        if line.startswith('#ifndef MICROPY_CONFIG_ROM_LEVEL'):
+            last_line = data[i - 2]
+
+            if not last_line.startswith('#define MICROPY_TASK_STACK_SIZE'):
+                data.insert(i - 1, '')
+
+            multiple = int(task_stack_size / 1024)
+
+            data[i - 2] = (
+                f'#define MICROPY_TASK_STACK_SIZE           ({multiple} * 1024)'
+            )
+
+            break
+
+    data = '\n'.join(data)
+
     with open(MPCONFIGPORT_PATH, 'wb') as f:
         f.write(data.encode('utf-8'))