cortexm: bump default stack size to 2048 bytes

aykevl · deadprogram · commit 1913cb76a5c6 · 2021-06-18T13:23:50.000+02:00
Previously it was 1024 bytes, which occasionally ran into a stack
overflow. I hope that 2048 bytes will be enough for most purposes.

I've also removed some 2048-byte stack size settings in JSON files,
which are unnecessary now that the parent (cortex-m.json) sets them.
diff --git a/targets/cortex-m.json b/targets/cortex-m.json
@@ -8,7 +8,7 @@
 	"rtlib": "compiler-rt",
 	"libc": "picolibc",
 	"automatic-stack-size": true,
-	"default-stack-size": 1024,
+	"default-stack-size": 2048,
 	"cflags": [
 		"-Oz",
 		"-mthumb",
diff --git a/targets/feather-stm32f405.json b/targets/feather-stm32f405.json
@@ -2,7 +2,6 @@
   "inherits": ["cortex-m4"],
   "build-tags": ["feather_stm32f405", "stm32f405", "stm32f4", "stm32"],
   "automatic-stack-size": false,
-  "default-stack-size": 1024,
   "linkerscript": "targets/stm32f405.ld",
   "extra-files": [
     "src/device/stm32/stm32f405.s"
diff --git a/targets/grandcentral-m4.json b/targets/grandcentral-m4.json
@@ -5,6 +5,5 @@
     "flash-method": "msd",
     "msd-volume-name": "GCM4BOOT",
     "msd-firmware-name": "firmware.uf2",
-    "openocd-interface": "jlink",
-    "default-stack-size": 2048
+    "openocd-interface": "jlink"
 }
diff --git a/targets/p1am-100.json b/targets/p1am-100.json
@@ -2,6 +2,5 @@
     "inherits": ["atsamd21g18a"],
     "build-tags": ["sam", "atsamd21g18a", "p1am_100"],
     "flash-command": "bossac -d -i -e -w -v -R --port={port} --offset=0x2000 {bin}",
-    "flash-1200-bps-reset": "true",
-    "default-stack-size": 2048
+    "flash-1200-bps-reset": "true"
 }
diff --git a/targets/pygamer.json b/targets/pygamer.json
@@ -4,6 +4,5 @@
     "flash-1200-bps-reset": "true",
     "flash-method": "msd",
     "msd-volume-name": "PYGAMERBOOT",
-    "msd-firmware-name": "arcade.uf2",
-    "default-stack-size": 2048
+    "msd-firmware-name": "arcade.uf2"
 }
diff --git a/targets/pyportal.json b/targets/pyportal.json
@@ -4,6 +4,5 @@
     "flash-1200-bps-reset": "true",
     "flash-method": "msd",
     "msd-volume-name": "PORTALBOOT",
-    "msd-firmware-name": "firmware.uf2",
-    "default-stack-size": 2048
+    "msd-firmware-name": "firmware.uf2"
 }
diff --git a/targets/wioterminal.json b/targets/wioterminal.json
@@ -4,6 +4,5 @@
     "flash-1200-bps-reset": "true",
     "flash-method": "msd",
     "msd-volume-name": "Arduino",
-    "msd-firmware-name": "firmware.uf2",
-    "default-stack-size": 2048
+    "msd-firmware-name": "firmware.uf2"
 }

Original file line number	Diff line number	Diff line change
`@@ -5,6 +5,5 @@`
`5`	`5`	`"flash-method": "msd",`
`6`	`6`	`"msd-volume-name": "GCM4BOOT",`
`7`	`7`	`"msd-firmware-name": "firmware.uf2",`
`8`		`- "openocd-interface": "jlink",`
`9`		`- "default-stack-size": 2048`
	`8`	`+ "openocd-interface": "jlink"`
`10`	`9`	`}`
Original file line number	Diff line number	Diff line change
`@@ -2,6 +2,5 @@`
`2`	`2`	`"inherits": ["atsamd21g18a"],`
`3`	`3`	`"build-tags": ["sam", "atsamd21g18a", "p1am_100"],`
`4`	`4`	`"flash-command": "bossac -d -i -e -w -v -R --port={port} --offset=0x2000 {bin}",`
`5`		`- "flash-1200-bps-reset": "true",`
`6`		`- "default-stack-size": 2048`
	`5`	`+ "flash-1200-bps-reset": "true"`
`7`	`6`	`}`
Original file line number	Diff line number	Diff line change
`@@ -4,6 +4,5 @@`
`4`	`4`	`"flash-1200-bps-reset": "true",`
`5`	`5`	`"flash-method": "msd",`
`6`	`6`	`"msd-volume-name": "PYGAMERBOOT",`
`7`		`- "msd-firmware-name": "arcade.uf2",`
`8`		`- "default-stack-size": 2048`
	`7`	`+ "msd-firmware-name": "arcade.uf2"`
`9`	`8`	`}`