-
Notifications
You must be signed in to change notification settings - Fork 274
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
Feat/ssd1306 partial update #23
Conversation
For monochrome displays each byte of the buffer size can store data for 8 pixels. We divide LV_VER_RES_MAX by 8 because the display vertical side is grouped in pages of 1 byte, so we need 1byte times LV_HOR_RES_MAX to represent the data of one page.
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.
Looks good, just a small question.
d504887 changes the display buffer size, from 8192 to 1024 bytes when using the 128*64 px display. As far as I know, the display buffer for monochrome displays should be "in pixels", am I right? |
Yes, it's in pixels. So 128 x 64 / 8 = 1 kB is correct. |
This is ready to be reviewed, I choosed assert over error handling on data transmission because it shouldn't fail. |
I've working on improving the SSD1306 display driver, hopefully this changes also help us to start working on the MCU independent drivers.
TO DO: