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

SSD1307 dma #626

Open
wants to merge 7 commits into
base: master
Choose a base branch
from
Open

Conversation

AndrewCapon
Copy link
Contributor

@AndrewCapon AndrewCapon commented Apr 29, 2024

This also includes: #624

Added DMA transfers for data transfers to reduce cpu overhead.

display.h has a new pure virtual bool UpdateFinished(), returns true if the update has finished.

oled_display.h implements UpdateFinished() and calls the driver.

SSD130x4WireSpiTransport has been altered to support basic chained DMA via the finished callback, Config now has a useDma flag to enable DMA.

Some times:

Using SPI Prescaler = 2

Blocking
  1492.6 us total update time and Cpu

Dma
  Total update time = 1549.3us
  Cpu = (12.9us*16) + 0.3us = 206.7us

Copy link

github-actions bot commented Apr 29, 2024

Test Results

151 tests  ±0   151 ✅ ±0   0s ⏱️ ±0s
 16 suites ±0     0 💤 ±0 
  1 files   ±0     0 ❌ ±0 

Results for commit 8a602c9. ± Comparison against base commit f7727ed.

♻️ This comment has been updated with latest results.

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