Skip to content
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.

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

bsp(esp32_p4_function_ev_board): Add deinit/remove/delete functions for display (BSP-598) #507

Merged
merged 1 commit into from
Feb 20, 2025

Conversation

espzav
Copy link
Collaborator

@espzav espzav commented Feb 11, 2025

ESP-BSP Pull Request checklist

  • Version of modified component bumped
  • CI passing

Change description

  • Add deinit/remove/delete functions for display
  • Tested memory after deinit - some lazy init (static) still keep there

@espzav espzav added this to the esp_lvgl_port 2.5.0 milestone Feb 11, 2025
Copy link

github-actions bot commented Feb 11, 2025

Test Results

 17 files  17 suites   5m 53s ⏱️
 68 tests 24 ✅  44 💤 0 ❌
408 runs  24 ✅ 384 💤 0 ❌

Results for commit edb30f2.

♻️ This comment has been updated with latest results.

@ThunderDai
Copy link

hi, have you test for avoid tear in direct mode?

@espzav espzav force-pushed the feat/bsp_deinit_display branch from a6bc5a2 to 8713649 Compare February 12, 2025 08:52
@espzav
Copy link
Collaborator Author

espzav commented Feb 12, 2025

hi, have you test for avoid tear in direct mode?

Now, I tested it. It is working same.

@espzav espzav removed this from the esp_lvgl_port 2.5.0 milestone Feb 13, 2025
@espzav
Copy link
Collaborator Author

espzav commented Feb 18, 2025

@tore-espressif PTAL

@espzav espzav force-pushed the feat/bsp_deinit_display branch 3 times, most recently from cc374a0 to 9d35694 Compare February 20, 2025 07:02
Copy link
Collaborator

@tore-espressif tore-espressif left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM! (I did not check the mem leaks though)

@espzav espzav force-pushed the feat/bsp_deinit_display branch from 9d35694 to edb30f2 Compare February 20, 2025 12:48
@espzav
Copy link
Collaborator Author

espzav commented Feb 20, 2025

LGTM! (I did not check the mem leaks though)

Thank you for review. I checked leaks and there are only lazy inits now.

@espzav espzav merged commit 531ad57 into master Feb 20, 2025
37 checks passed
@espzav espzav deleted the feat/bsp_deinit_display branch February 20, 2025 14:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants