Skip to content

Conversation

philmoz
Copy link
Collaborator

@philmoz philmoz commented Jul 19, 2025

As there is plenty of RAM available on color radios, this PR increases the memory allocation to LVGL.
Mainly for Lua scripts that want to use images.

@philmoz philmoz added this to the 2.11.3 milestone Jul 19, 2025
@philmoz philmoz added color Related generally to color LCD radios lua labels Jul 19, 2025
@philmoz philmoz force-pushed the philmoz/more-lvgl-memory branch from 90b7081 to 786240b Compare July 21, 2025 10:46
@pfeerick
Copy link
Member

Isn't doubling the amount of ram "allocated" to LVGL in simulator going to cause issues down the track with scripts running on simulator but not on the radio? We want the simulator to behave as much like the radio as we can make it, after all.

@pfeerick pfeerick changed the title feat(color): increase RAM allocation to LVGL for Lua scripts with many images. feat(color): increase RAM allocation to LVGL for Lua scripts with many images Jul 22, 2025
@philmoz
Copy link
Collaborator Author

philmoz commented Jul 23, 2025

Isn't doubling the amount of ram "allocated" to LVGL in simulator going to cause issues down the track with scripts running on simulator but not on the radio? We want the simulator to behave as much like the radio as we can make it, after all.

Difficult to match since simulator runs on 64bit CPU's so memory allocation differs. @offer-shmuely ran into an issue where a script would run on the radio; but not in the simulator.

@pfeerick pfeerick merged commit 7264b28 into main Jul 23, 2025
42 checks passed
@pfeerick pfeerick deleted the philmoz/more-lvgl-memory branch July 23, 2025 00:21
@offer-shmuely
Copy link
Contributor

Great.
I like the sim to represent the tx as close as possible
Now it was the opposite,
Also, I like to ask a better log error, both for the sim, with the name of the faulty image.
And maybe even on the image are itself, some kind of mark or text with "no mem"

pfeerick pushed a commit that referenced this pull request Jul 25, 2025
e.g. for Lua scripts with many images
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
color Related generally to color LCD radios lua
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants