modules: lvgl: allow usage of zephyr memory-region for mem / vdb

Allow selection of a custom section for memory-poll and / or VDB
based on zephyr memory-regions.
This takes advantages that zephyr,memory-region automatically have
sections being created, hence allowing to easily indicate into which
memory-region to store data.

Signed-off-by: Alain Volmat <[email protected]>

Author: Alain Volmat

modules/lvgl/Kconfig.memory

@@ -52,6 +52,22 @@ config LV_Z_MEMORY_POOL_CUSTOM_SECTION
 	  memory pool to a custom location, such as tightly coupled or
 	  external memory.
 
+config LV_Z_MEMORY_POOL_CUSTOM_GENERIC_SECTION
+	bool "Link memory pool to custom generic section"
+	depends on LV_Z_MEM_POOL_SYS_HEAP
+	help
+	  Place LVGL memory pool in custom generic section automatically
+	  created by Zephyr when compatible zephyr,memory-region is used.
+	  LV_Z_MEMORY_POOL_CUSTOM_SECTION must be disabled in order to use
+	  generic section.
+
+config LV_Z_MEMORY_POOL_CUSTOM_GENERIC_SECTION_NAME
+	string "Name of the memory pool custom generic section"
+	depends on LV_Z_MEMORY_POOL_CUSTOM_GENERIC_SECTION
+	help
+	  Name of the generic section to be used to store memory-pool.
+	  The name must match with the name of a zephyr,memory-region.
+
 config LV_Z_VDB_SIZE
 	int "Rendering buffer size"
 	default 100 if LV_Z_FULL_REFRESH
@@ -92,6 +108,22 @@ config LV_Z_VDB_CUSTOM_SECTION
 	  rendering buffers to a custom location, such as tightly coupled or
 	  external memory.
 
+config LV_Z_VDB_CUSTOM_GENERIC_SECTION
+	bool "Link rendering buffers to custom generic section"
+	depends on LV_Z_BUFFER_ALLOC_STATIC
+	help
+	  Place LVGL rendering buffers in custom generic section automatically
+	  created by Zephyr when compatible zephyr,memory-region is used.
+	  LV_Z_MEMORY_POOL_CUSTOM_SECTION must be disabled in order to use
+	  generic section.
+
+config LV_Z_VDB_CUSTOM_GENERIC_SECTION_NAME
+	string "Link rendering buffers to custom generic section"
+	depends on LV_Z_VDB_CUSTOM_GENERIC_SECTION
+	help
+	  Name of the generic section to be used to store rendering buffers.
+	  The name must match with the name of a zephyr,memory-region.
+
 config LV_Z_MONOCHROME_CONVERSION_BUFFER
 	bool "Use intermediate conversion buffer for monochrome displays"
 	default y

modules/lvgl/lvgl.c

@@ -78,22 +78,27 @@ static uint8_t *mono_vtile_buf_p[DT_ZEPHYR_DISPLAYS_COUNT] = {NULL};
 /* uint16_t * or uint32_t *, therefore buffer needs to be aligned accordingly to */
 /* prevent unaligned memory accesses. */
 
+#if defined(CONFIG_LV_Z_VDB_CUSTOM_SECTION)
+#define LV_BUF_SECTION	Z_GENERIC_SECTION(.lvgl_buf)
+#elif defined(CONFIG_LV_Z_VDB_CUSTOM_GENERIC_SECTION)
+#define LV_BUF_SECTION	Z_GENERIC_SECTION(CONFIG_LV_Z_VDB_CUSTOM_GENERIC_SECTION_NAME)
+#else
+#define LV_BUF_SECTION
+#endif
+
 /* clang-format off */
 #define LV_BUFFERS_DEFINE(n)									\
 	static DISPLAY_BUFFER_ALIGN(LV_DRAW_BUF_ALIGN) uint8_t buf0_##n[BUFFER_SIZE(n)]		\
-	IF_ENABLED(CONFIG_LV_Z_VDB_CUSTOM_SECTION, (Z_GENERIC_SECTION(.lvgl_buf)))		\
-						       __aligned(CONFIG_LV_Z_VDB_ALIGN);	\
+	LV_BUF_SECTION __aligned(CONFIG_LV_Z_VDB_ALIGN);					\
 												\
 	IF_ENABLED(CONFIG_LV_Z_DOUBLE_VDB, (							\
 	static DISPLAY_BUFFER_ALIGN(LV_DRAW_BUF_ALIGN) uint8_t buf1_##n[BUFFER_SIZE(n)]		\
-	IF_ENABLED(CONFIG_LV_Z_VDB_CUSTOM_SECTION, (Z_GENERIC_SECTION(.lvgl_buf)))		\
-			__aligned(CONFIG_LV_Z_VDB_ALIGN);					\
+	LV_BUF_SECTION __aligned(CONFIG_LV_Z_VDB_ALIGN);					\
 	))											\
 												\
 	IF_ENABLED(ALLOC_MONOCHROME_CONV_BUFFER, (						\
 	static uint8_t mono_vtile_buf_##n[BUFFER_SIZE(n)]					\
-	IF_ENABLED(CONFIG_LV_Z_VDB_CUSTOM_SECTION, (Z_GENERIC_SECTION(.lvgl_buf)))		\
-			__aligned(CONFIG_LV_Z_VDB_ALIGN);					\
+	LV_BUF_SECTION __aligned(CONFIG_LV_Z_VDB_ALIGN);					\
 	))
 
 FOR_EACH(LV_BUFFERS_DEFINE, (), LV_DISPLAYS_IDX_LIST);

modules/lvgl/lvgl_mem.c

@@ -15,6 +15,9 @@ LOG_MODULE_DECLARE(lvgl, CONFIG_LV_Z_LOG_LEVEL);
 
 #ifdef CONFIG_LV_Z_MEMORY_POOL_CUSTOM_SECTION
 #define HEAP_MEM_ATTRIBUTES Z_GENERIC_SECTION(.lvgl_heap) __aligned(8)
+#elif defined(CONFIG_LV_Z_MEMORY_POOL_CUSTOM_GENERIC_SECTION)
+#define HEAP_MEM_ATTRIBUTES Z_GENERIC_SECTION(CONFIG_LV_Z_MEMORY_POOL_CUSTOM_GENERIC_SECTION_NAME) \
+			    __aligned(8)
 #else
 #define HEAP_MEM_ATTRIBUTES __aligned(8)
 #endif /* CONFIG_LV_Z_MEMORY_POOL_CUSTOM_SECTION */