Develop/nn by 00lllooolll00 · Pull Request #3 · EmbeddedKitOrg/EmbeddedKit

00lllooolll00 · 2026-01-31T07:24:40Z

No description provided.

Copilot

Pull request overview

This PR backports / refactors the LVGL integration towards the 8.x API style, simplifies the build integration, and extends the HAL and memory utilities.

Changes:

Reworks LVGL core headers and sources (events, display, input, groups, styles, drawing) to a different API surface and build layout.
Adjusts build and packaging infrastructure (CMake, Zephyr, ESP, Micropython, GitHub Actions) to the new LVGL structure.
Extends and refactors local utilities and HAL (TLSF-backed ek_mem wrappers, new I2C HAL, minor SPI updates) and adds LVGL documentation in multiple languages.

Reviewed changes

Copilot reviewed 61 out of 961 changed files in this pull request and generated 23 comments.

Show a summary per file

File	Description
L3_Middlewares/LVGL/src/core/lv_obj_style.h	Refactors style-related types/APIs, adds internal structs and helpers.
L3_Middlewares/LVGL/src/core/lv_obj_scroll.h	Updates scroll enums, types, and prototypes to new object forward-decl pattern.
L3_Middlewares/LVGL/src/core/lv_obj_pos.h	Changes layout/position APIs and introduces layout registration helpers.
L3_Middlewares/LVGL/src/core/lv_obj_draw.h	Consolidates draw includes, defines draw-part descriptor, and exposes ext-draw helpers.
L3_Middlewares/LVGL/src/core/lv_indev_scroll.h	Switches scroll helpers to new indev proc type and include path.
L3_Middlewares/LVGL/src/core/lv_indev.h	Introduces a new public input-device API surface.
L3_Middlewares/LVGL/src/core/lv_event.h / lv_event.c	Reimplements the event system around `lv_event_t` and `lv_event_send`.
L3_Middlewares/LVGL/src/core/lv_disp.h / lv_disp.c	Adds display API (screen management, background, inactivity, invalidation).
L3_Middlewares/LVGL/src/core/lv_group.h / lv_group.c	Reworks focus group implementation and API names.
L3_Middlewares/LVGL/src/core/lv_core.mk	Adds a core source list for LVGL builds.
L3_Middlewares/LVGL/lvgl.h	Reorganizes top-level LVGL includes and version macros.
L3_Middlewares/LVGL/env_support/cmake/*.cmake	Simplifies version handling and platform integration (Zephyr, Micropython, ESP, custom).
L3_Middlewares/LVGL/doc/README*.md	Adds/updates multi-language LVGL README documentation.
L2_Core/utils/src/ek_mem.c / inc/ek_mem.h	Wraps TLSF allocators with overridable APIs and adjusts heap macros.
L2_Core/hal/src/ek_hal_spi.c	Minor cleanup and comments in SPI HAL stub.
L2_Core/hal/src/ek_hal_i2c.c / inc/ek_hal_i2c.h	Introduces a new I2C HAL abstraction with locking and list registration.
.github/workflows/build.yml	Simplifies CI matrix by removing LVGL ThorVG combinations.
L3_Middlewares/CMakeLists.txt	Removes ThorVG option and links LVGL more simply.
L3_Middlewares/LVGL/CMakeLists.txt	Simplifies LVGL top-level CMake config and removes version.cmake usage.

Comments suppressed due to low confidence (2)

L3_Middlewares/LVGL/src/core/lv_indev_scroll.h:1

Both _lv_indev_scroll_handler and _lv_indev_scroll_throw_handler use the _lv_indev_proc_t type, but this header neither declares that struct nor includes a header that does, which will lead to an unknown type error when compiling any user of these functions. Either forward-declare the type (e.g. struct _lv_indev_proc_t;) above these prototypes or include the appropriate private indev header where _lv_indev_proc_t is defined.

/**

L3_Middlewares/LVGL/src/core/lv_obj_style.h:1

This header now omits the lv_style.h and lv_types.h includes while still using lv_style_t, lv_style_selector_t, lv_style_prop_t, lv_style_value_t, lv_anim_path_cb_t, etc., which will cause undeclared-type errors unless every includer happens to pull in those headers beforehand. Restore the #include "../misc/lv_style.h" (and lv_types.h if needed) or add equivalent forward declarations/typedefs so that all style-related types are visible from lv_obj_style.h alone.

/**

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Copilot · 2026-01-31T07:26:58Z

+/*********************
+ *      INCLUDES
+ *********************/
+#include <stdbool.h>


The event header uses several LVGL core types (lv_res_t, lv_obj_class_t, lv_indev_t, lv_obj_draw_part_dsc_t, lv_draw_ctx_t, lv_area_t, lv_point_t, lv_anim_t, lv_coord_t, lv_cover_res_t) without including the headers that define them or forward-declaring the struct/typedefs, which will cause compilation errors when including lv_event.h in isolation. Add the appropriate includes (e.g. lv_obj.h, lv_obj_class.h, lv_indev.h, lv_obj_draw.h, lv_anim.h, lv_area.h, lv_types.h) or add the required forward declarations and typedefs so that all referenced types are visible to translation units that include this header.

Suggested change

#include <stdbool.h>

#include <stdbool.h>

#include "../misc/lv_types.h"

#include "../misc/lv_area.h"

#include "../misc/lv_anim.h"

#include "../indev/lv_indev.h"

#include "lv_obj_draw.h"

Copilot · 2026-01-31T07:26:58Z

+struct _lv_obj_t;
+struct _lv_event_dsc_t;


The event header uses several LVGL core types (lv_res_t, lv_obj_class_t, lv_indev_t, lv_obj_draw_part_dsc_t, lv_draw_ctx_t, lv_area_t, lv_point_t, lv_anim_t, lv_coord_t, lv_cover_res_t) without including the headers that define them or forward-declaring the struct/typedefs, which will cause compilation errors when including lv_event.h in isolation. Add the appropriate includes (e.g. lv_obj.h, lv_obj_class.h, lv_indev.h, lv_obj_draw.h, lv_anim.h, lv_area.h, lv_types.h) or add the required forward declarations and typedefs so that all referenced types are visible to translation units that include this header.

Copilot · 2026-01-31T07:26:58Z

+typedef struct _lv_event_t {
+    struct _lv_obj_t * target;
+    struct _lv_obj_t * current_target;
+    lv_event_code_t code;


The event header uses several LVGL core types (lv_res_t, lv_obj_class_t, lv_indev_t, lv_obj_draw_part_dsc_t, lv_draw_ctx_t, lv_area_t, lv_point_t, lv_anim_t, lv_coord_t, lv_cover_res_t) without including the headers that define them or forward-declaring the struct/typedefs, which will cause compilation errors when including lv_event.h in isolation. Add the appropriate includes (e.g. lv_obj.h, lv_obj_class.h, lv_indev.h, lv_obj_draw.h, lv_anim.h, lv_area.h, lv_types.h) or add the required forward declarations and typedefs so that all referenced types are visible to translation units that include this header.

Copilot · 2026-01-31T07:26:58Z

+typedef struct {
+    const lv_point_t * point;   /**< A point relative to screen to check if it can click the object or not*/
+    bool res;                   /**< true: `point` can click the object; false: it cannot*/
+} lv_hit_test_info_t;


The event header uses several LVGL core types (lv_res_t, lv_obj_class_t, lv_indev_t, lv_obj_draw_part_dsc_t, lv_draw_ctx_t, lv_area_t, lv_point_t, lv_anim_t, lv_coord_t, lv_cover_res_t) without including the headers that define them or forward-declaring the struct/typedefs, which will cause compilation errors when including lv_event.h in isolation. Add the appropriate includes (e.g. lv_obj.h, lv_obj_class.h, lv_indev.h, lv_obj_draw.h, lv_anim.h, lv_area.h, lv_types.h) or add the required forward declarations and typedefs so that all referenced types are visible to translation units that include this header.

Copilot · 2026-01-31T07:26:58Z

+typedef struct {
+    lv_cover_res_t res;
+    const lv_area_t * area;
+} lv_cover_check_info_t;


The event header uses several LVGL core types (lv_res_t, lv_obj_class_t, lv_indev_t, lv_obj_draw_part_dsc_t, lv_draw_ctx_t, lv_area_t, lv_point_t, lv_anim_t, lv_coord_t, lv_cover_res_t) without including the headers that define them or forward-declaring the struct/typedefs, which will cause compilation errors when including lv_event.h in isolation. Add the appropriate includes (e.g. lv_obj.h, lv_obj_class.h, lv_indev.h, lv_obj_draw.h, lv_anim.h, lv_area.h, lv_types.h) or add the required forward declarations and typedefs so that all referenced types are visible to translation units that include this header.

Copilot · 2026-01-31T07:27:01Z

+ * @param indev pointer to an input device
+ * @return pointer to the indev read refresher timer. (NULL on error)
+ */
+lv_timer_t * lv_indev_get_read_timer(lv_disp_t * indev);


The lv_indev_get_read_timer API takes an lv_disp_t * parameter while the documentation and semantics clearly refer to an input device; the type should be lv_indev_t * (or const lv_indev_t *) to match how this function is used elsewhere in LVGL. Update the function signature (and its implementation) to accept an input device pointer instead of a display pointer to prevent type mismatches and incorrect calls.

Suggested change

lv_timer_t * lv_indev_get_read_timer(lv_disp_t * indev);

lv_timer_t * lv_indev_get_read_timer(lv_indev_t * indev);

Copilot · 2026-01-31T07:27:02Z

+ * Add a style to an object.
 * @param obj       pointer to an object
 * @param style     pointer to a style to remove. Can be NULL to check only the selector
 * @param selector  OR-ed values of states and a part to remove only styles with matching selectors. LV_STATE_ANY and LV_PART_ANY can be used
- *
- * Examples:
- * @code
- * lv_obj_remove_style(obj, &style, LV_PART_ANY | LV_STATE_ANY); //Remove a specific style
- *
- * lv_obj_remove_style(obj, NULL, LV_PART_MAIN | LV_STATE_ANY); //Remove all styles from the main part
- *
- * lv_obj_remove_style(obj, NULL, LV_PART_ANY | LV_STATE_ANY); //Remove all styles
- * @endcode
+ * @example lv_obj_remove_style(obj, &style, LV_PART_ANY | LV_STATE_ANY); //Remove a specific style
+ * @example lv_obj_remove_style(obj, NULL, LV_PART_MAIN | LV_STATE_ANY); //Remove all styles from the main part
+ * @example lv_obj_remove_style(obj, NULL, LV_PART_ANY | LV_STATE_ANY); //Remove all styles
 */


The documentation for lv_obj_remove_style still says “Add a style to an object.” while the function removes styles, and the lv_obj_get_style_opa_recursive comment contains a typo (“teh MAIN part”). Update the comment header to describe removal (e.g. “Remove a style from an object.”) and fix the typo “teh” to “the” to keep the API documentation accurate and readable.

Copilot · 2026-01-31T07:27:02Z

 * Get the `opa` style property from all parents and multiply and `>> 8` them.
 * @param obj       the object whose opacity should be get
- * @param part      the part whose opacity should be get. Non-MAIN parts will consider the `opa` of the MAIN part too
+ * @param part      the part whose opacity should be get. Non-MAIN parts will consider the `opa` of teh MAIN part too


The documentation for lv_obj_remove_style still says “Add a style to an object.” while the function removes styles, and the lv_obj_get_style_opa_recursive comment contains a typo (“teh MAIN part”). Update the comment header to describe removal (e.g. “Remove a style from an object.”) and fix the typo “teh” to “the” to keep the API documentation accurate and readable.

Suggested change

* @param part the part whose opacity should be get. Non-MAIN parts will consider the `opa` of teh MAIN part too

* @param part the part whose opacity should be get. Non-MAIN parts will consider the `opa` of the MAIN part too

Copilot · 2026-01-31T07:27:02Z


 /**
- * Get the actually set x coordinate of object, i.e. the offset from the set alignment
+ * Get the actually set x coordinate of object, i.e. the offset form the set alignment


The word “form” in “the offset form the set alignment” is a typo and should be “from” in both comments to correctly describe the offset from the set alignment.

Copilot · 2026-01-31T07:27:03Z


 /**
- * Get the actually set y coordinate of object, i.e. the offset from the set alignment
+ * Get the actually set y coordinate of object, i.e. the offset form the set alignment


The word “form” in “the offset form the set alignment” is a typo and should be “from” in both comments to correctly describe the offset from the set alignment.

00lllooolll00 added 10 commits January 30, 2026 22:30

[update]添加了和模块

7b06e87

update

7babe1d

[update]优化FreeRTOS的 FreeRTOSConfig.h 的布局

d1c57f4

[update]优化FreeRTOS的 FreeRTOSConfig.h 的布局

2580184

[fix]修复FreeRTOS的配置文件问题

f29c6ae

[fix]修复FreeRTOS的配置文件问题

0d0170d

[fix]修改默认配置

36c3f0b

[update]更新模块，增加灵活性

048f3f1

[update]优化布局

5a61e3c

[update]更换lvgl版本为更通用的v8.3.11

2942d9f

Copilot AI review requested due to automatic review settings January 31, 2026 07:24

Copilot AI reviewed Jan 31, 2026

View reviewed changes

00lllooolll00 added 2 commits January 31, 2026 15:35

[update]修改gitaction的windows的unit test使用gcc

a1e7b5e

[update]fuck windows

c772b4b

00lllooolll00 merged commit 40e9993 into master Jan 31, 2026
8 checks passed

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

Develop/nn#3

Develop/nn#3
00lllooolll00 merged 12 commits into
masterfrom
develop/nn

00lllooolll00 commented Jan 31, 2026

Uh oh!

Copilot AI left a comment

Uh oh!

Copilot AI Jan 31, 2026

Uh oh!

Copilot AI Jan 31, 2026

Uh oh!

Copilot AI Jan 31, 2026

Uh oh!

Copilot AI Jan 31, 2026

Uh oh!

Copilot AI Jan 31, 2026

Uh oh!

Copilot AI Jan 31, 2026

Uh oh!

Copilot AI Jan 31, 2026

Uh oh!

Copilot AI Jan 31, 2026

Uh oh!

Copilot AI Jan 31, 2026

Uh oh!

Copilot AI Jan 31, 2026

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

-#include <stdbool.h>
+#include <stdbool.h>
+#include "../misc/lv_types.h"
+#include "../misc/lv_area.h"
+#include "../misc/lv_anim.h"
+#include "../indev/lv_indev.h"
+#include "lv_obj_draw.h"

	lv_timer_t * lv_indev_get_read_timer(lv_disp_t * indev);
	lv_timer_t * lv_indev_get_read_timer(lv_indev_t * indev);

	* @param part the part whose opacity should be get. Non-MAIN parts will consider the `opa` of teh MAIN part too
	* @param part the part whose opacity should be get. Non-MAIN parts will consider the `opa` of the MAIN part too

Uh oh!

Conversation

00lllooolll00 commented Jan 31, 2026

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Reviewed changes

Uh oh!

Copilot AI Jan 31, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Jan 31, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Jan 31, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Jan 31, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Jan 31, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Jan 31, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Jan 31, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Jan 31, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Jan 31, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Jan 31, 2026

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants