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

Allow setting of CIRCUITPY_DISPLAY_LIMIT in settings.toml #10158

Open
wants to merge 13 commits into
base: 9.2.x
Choose a base branch
from

Conversation

RetiredWizard
Copy link

Set the maximum supported number of displayio displays. This value overrides the CIRCUITPY_DISPLAY_LIMIT compile-time flag.

I do have some questions about this solution.

I'm not sure if this fixes #1760 since Tannewt has said he hopes to implement a better solution in CP 11. I'm also not sure main.c was the right location to insert the call to malloc_display_memory but I don't know displayio well enough to know if there was a better place somewhere in the displayio modules.

So far I've only been testing with the RP2350 feather (DVI display) with a TFT featherwing (SPI display). I'll go ahead and get some more configurations set up for testing but figured I get this posted so others could start reviewing.

@RetiredWizard
Copy link
Author

RetiredWizard commented Mar 21, 2025

It looks like boards with built-in displays don't like pointer references to the display structures, I was unsuccessful getting a pointer/array combination of variables to work so I think I'll either have to look at the way the boards with builtin displays are accessing the structures or consider leaving the compile time portion of the array in place and just access the dynamically allocated array elements for display indexes > (compile time)CIRCUITPY_DISPLAY_LIMIT.

Fun for another night.....

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.

1 participant