Skip to content

Conversation

@grafail
Copy link
Contributor

@grafail grafail commented Jan 2, 2025

This is a fix for an issue caused by allocating 2 buffers in a row that seems to cause this error: MemoryError: allocation flags must be the same for both buffers

Reproduction code:

fb1 = display_bus.allocate_framebuffer(sz, lcd_bus.MEMORY_SPIRAM)
fb2 = display_bus.allocate_framebuffer(sz, lcd_bus.MEMORY_SPIRAM)

@kdschlosser kdschlosser merged commit 6318e4c into lvgl-micropython:main Jan 3, 2025
3 checks passed
@kdschlosser
Copy link
Collaborator

Thanks for catching this. I did a rework on the initilization of the bus drivers and I removed the special handling for the RGB bus because it is no longer needed so I changed from using a void * buffer to using memoryviews that are stored in the structure for a bus driver. forgot to set the caps for the buffers.

If you are up for it I am ging to reduce the memory footprint of the bus drivers and I will need someone to test to make sure I didn't muck them up. If you are game for it that is going to be coming up as soon as I fix the issue with the USB-CDC stuff.

@grafail
Copy link
Contributor Author

grafail commented Jan 3, 2025

To be honest I have still not been able to get my display set up using micropython, but would be up for testing this.

@kdschlosser
Copy link
Collaborator

well give me the low down on what is going on with it...

@grafail
Copy link
Contributor Author

grafail commented Jan 3, 2025

well give me the low down on what is going on with it...
Added some more context here
#230

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