samples: drivers: display: Modify heap size configs for RT595 & RT1170

ngphibang · fabiobaltieri · commit 5607a4465811 · 2024-03-07T10:57:57.000Z
The heap size for i.MX RT1170 is way too much while for i.MX RT595, it
is too small to afford for 4-bytes formats, e.g. ARGB8888.

Signed-off-by: Phi Bang Nguyen &lt;phibang.nguyen@nxp.com&gt;
diff --git a/samples/drivers/display/boards/mimxrt1170_evk_mimxrt1176_cm7_A.conf b/samples/drivers/display/boards/mimxrt1170_evk_mimxrt1176_cm7_A.conf
@@ -4,4 +4,7 @@
 # SPDX-License-Identifier: Apache-2.0
 #
 
-CONFIG_HEAP_MEM_POOL_SIZE=4194304
+# Sample will allocate buffer equal to: (panelwidth / 8) * (panelwidth / 4) * pixel depth. For a
+# 1280x720 display in a 32-bpp format (e.g. ARGB8888), this is (720 / 8) * (720 / 4) * 4 = 64800
+# bytes. We include 128 bytes of padding for kernel heap structures
+CONFIG_HEAP_MEM_POOL_SIZE=64928
diff --git a/samples/drivers/display/boards/mimxrt1170_evk_mimxrt1176_cm7_B.conf b/samples/drivers/display/boards/mimxrt1170_evk_mimxrt1176_cm7_B.conf
@@ -4,4 +4,4 @@
 # SPDX-License-Identifier: Apache-2.0
 #
 
-CONFIG_HEAP_MEM_POOL_SIZE=4194304
+CONFIG_HEAP_MEM_POOL_SIZE=64928
diff --git a/samples/drivers/display/boards/mimxrt1170_evkb_cm7.conf b/samples/drivers/display/boards/mimxrt1170_evkb_cm7.conf
@@ -4,4 +4,7 @@
 # SPDX-License-Identifier: Apache-2.0
 #
 
-CONFIG_HEAP_MEM_POOL_SIZE=4194304
+# Sample will allocate buffer equal to: (panelwidth / 8) * (panelwidth / 4) * pixel depth. For a
+# 1280x720 display in a 32-bpp format (e.g. ARGB8888), this is (720 / 8) * (720 / 4) * 4 = 64800
+# bytes. We include 128 bytes of padding for kernel heap structures
+CONFIG_HEAP_MEM_POOL_SIZE=64928
diff --git a/samples/drivers/display/boards/mimxrt595_evk_mimxrt595s_cm33.conf b/samples/drivers/display/boards/mimxrt595_evk_mimxrt595s_cm33.conf
@@ -4,7 +4,7 @@
 # SPDX-License-Identifier: Apache-2.0
 #
 
-# Sample will allocate buffer equal to: (height / 8) * (height / 4) * pixel depth
-# For the 1280x720 16 bit display, this is (720 / 8) * (720 / 4) * 2 = 32400 bytes
-# We include 128 bytes of padding for kernel heap structures
-CONFIG_HEAP_MEM_POOL_SIZE=32528
+# Sample will allocate buffer equal to: (panelwidth / 8) * (panelwidth / 4) * pixel depth. For a
+# 1280x720 display in a 32-bpp format (e.g. ARGB8888), this is (720 / 8) * (720 / 4) * 4 = 64800
+# bytes. We include 128 bytes of padding for kernel heap structures
+CONFIG_HEAP_MEM_POOL_SIZE=64928

Original file line number	Diff line number	Diff line change
`@@ -4,4 +4,7 @@`
`4`	`4`	`# SPDX-License-Identifier: Apache-2.0`
`5`	`5`	`#`
`6`	`6`
`7`		`-CONFIG_HEAP_MEM_POOL_SIZE=4194304`
	`7`	`+# Sample will allocate buffer equal to: (panelwidth / 8) * (panelwidth / 4) * pixel depth. For a`
	`8`	`+# 1280x720 display in a 32-bpp format (e.g. ARGB8888), this is (720 / 8) * (720 / 4) * 4 = 64800`
	`9`	`+# bytes. We include 128 bytes of padding for kernel heap structures`
	`10`	`+CONFIG_HEAP_MEM_POOL_SIZE=64928`
Original file line number	Diff line number	Diff line change
`@@ -4,4 +4,4 @@`
`4`	`4`	`# SPDX-License-Identifier: Apache-2.0`
`5`	`5`	`#`
`6`	`6`
`7`		`-CONFIG_HEAP_MEM_POOL_SIZE=4194304`
	`7`	`+CONFIG_HEAP_MEM_POOL_SIZE=64928`