fix(lvgl_port): Improved and fixed deinit function (remove semaphore) #675

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Merged

espzav merged 1 commit into master from fix/lvgl_port_deinit_fix

Nov 13, 2025

Collaborator

espzav commented Nov 12, 2025 •

edited

Loading

ESP-BSP Pull Request checklist

Version of modified component bumped
CI passing

Change description

Improved and fixed deinit function (remove semaphore)
Closes esp_lvgl_port: Always having a mutex lock(task_init_mux) makes it impossible to restore task priority (BSP-739) #673

github-actions bot commented Nov 12, 2025 •

edited

Loading

Test Results

28 tests 28 ✅ 1m 32s ⏱️
5 suites 0 💤
5 files 0 ❌

Results for commit b1c0156.

♻️ This comment has been updated with latest results.

espzav force-pushed the fix/lvgl_port_deinit_fix branch from 4805d0f to ed24a46 Compare

November 12, 2025 13:56

espzav requested review from PetrESP and tore-espressif

November 12, 2025 14:41

Collaborator Author

espzav commented Nov 12, 2025

@PetrESP @tore-espressif PTAL

PetrESP approved these changes

View reviewed changes

Collaborator

PetrESP left a comment

To me this change makes sense and is cleaner. Great job.
Personally I would maybe separate the touch driver change to a separate commit because from this I don't know how it's connected to the LVGL task.

ThunderDai commented Nov 13, 2025

hi, LVGL v8 also be fixed: https://github.com/espressif/esp-bsp/blob/fix/lvgl_port_deinit_fix/components/esp_lvgl_port/src/lvgl8/esp_lvgl_port.c#L190


          fix(lvgl_port): Improved and fixed deinit function (remove semaphor)

b1c0156

espzav force-pushed the fix/lvgl_port_deinit_fix branch from ed24a46 to b1c0156 Compare

November 13, 2025 07:02

github-actions bot commented Nov 13, 2025

⚠️ Build failed for idf_ver=latest index=4

This failure was ignored (continue-on-error enabled).
See details: View build logs

github-actions bot commented Nov 13, 2025

⚠️ Build failed for idf_ver=latest index=5

This failure was ignored (continue-on-error enabled).
See details: View build logs

github-actions bot commented Nov 13, 2025

⚠️ Build failed for idf_ver=latest index=3

This failure was ignored (continue-on-error enabled).
See details: View build logs

github-actions bot commented Nov 13, 2025

⚠️ Build failed for idf_ver=latest index=2

This failure was ignored (continue-on-error enabled).
See details: View build logs

github-actions bot commented Nov 13, 2025

⚠️ Build failed for idf_ver=latest index=1

This failure was ignored (continue-on-error enabled).
See details: View build logs

espzav merged commit 21c6cd0 into master

69 checks passed

espzav deleted the fix/lvgl_port_deinit_fix branch

November 13, 2025 13:56

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet