Skip to content

EdgeTx patches for Lvgl 8.4. #3

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

Merged
merged 2 commits into from
Apr 28, 2025
Merged

EdgeTx patches for Lvgl 8.4. #3

merged 2 commits into from
Apr 28, 2025

Conversation

philmoz
Copy link
Collaborator

@philmoz philmoz commented May 20, 2024

Lvgl 8.4 with the EdgeTx patches applied.

@pfeerick pfeerick merged commit decc7c7 into release/v8.4 Apr 28, 2025
@pfeerick
Copy link
Member

pfeerick commented Apr 29, 2025

What is supposed to be happening with the branches here... are we actually supposed to be merging into release/v8.4 or v8.4-patched as a patched version of that branch, with release/v8.4 there as a copy from upstream? I have a feeling it should be the latter, so have removed the commit for now. Also EdgeTX/edgetx#5038 is pointing to origin/v8.4-patched

@philmoz
Copy link
Collaborator Author

philmoz commented Apr 29, 2025

Yes v8.4-patched should be used for our changes.

Is commit 8f905f6 from 8.2 also needed for 8.4?

@pfeerick
Copy link
Member

pfeerick commented Apr 29, 2025

I don't see any reference to STM32H7RS in v8.4-patched branches lv_gpu_stm32_dma2d.c, so I think some equivalent (as 8.4 code has extra instructions there now also) may be needed. @raphaelcoeffic Care to wack that mole? Else I can blindly copy 'n paste the #elif defined(STM32H7) block and change AHB3ENR to AHB5ENR 🙈 🙊

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.

2 participants