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

Batch display command arguments #2059

Merged
merged 1 commit into from
Jun 19, 2024
Merged

Conversation

mark9064
Copy link
Member

@mark9064 mark9064 commented May 6, 2024

  • Address window and scrolling arguments are now sent in a single multi-byte transaction rather than multiple single-byte transactions
    • Faster: much less overhead due to the workaround required to send single bytes
    • Display update speed is mostly limited by the SPI bus so using the bus more efficiently helps significantly
  • Improved a few other documentation bits inside the display driver

This wasn't really possible without the fixing the SPI race conditions, but with #2046 merged we can enjoy this speed up now

Copy link

github-actions bot commented May 6, 2024

Build size and comparison to main:

Section Size Difference
text 376952B -48B
data 940B 0B
bss 63548B 8B

@JF002 JF002 added this to the 1.15.0 milestone May 12, 2024
@mark9064 mark9064 mentioned this pull request May 18, 2024
@JF002 JF002 merged commit f8f8993 into InfiniTimeOrg:main Jun 19, 2024
7 checks passed
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