-
Notifications
You must be signed in to change notification settings - Fork 1.1k
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
Add address_little_endian and fix display memory reuse #7891
Conversation
Add address_little_endian for epaper displays with little endian (low byte first) addresses. Also clears allocated display and display bus memory so it has a known state. The acep member wasn't always set so it varied accidentally. Fixes micropython#7560. May fix micropython#7778. Fixes micropython#5119.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I did not see any code issues, but I smoke-tested this on a PYPortal, CLUE, MONSTER M4SK, and a Macropad (I2C display). All but the Macropad were fine. On the Macropad, there is some kind of rolling/flickering effect. The first time I loaded it, the screen was unreadable. The second time, I could see the REPL, but there was a rolling effect down from the top.
Thanks for the testing! I didn't fully move Display over to the core attributes. Fixed now. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Retested on macropad; all fine now. Also thanks for redoing the validations.
Add address_little_endian for epaper displays with little endian
(low byte first) addresses.
Also clears allocated display and display bus memory so it has a
known state. The acep member wasn't always set so it varied
accidentally.
Fixes #7560. May fix #7778. Fixes #5119.