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

Support for ST7567 LCD I2C display driver #310

Draft
wants to merge 51 commits into
base: master
Choose a base branch
from

Conversation

geeksville
Copy link
Contributor

This is a WIP driver for the ST7567 LCD chip. Do not merge. I'm just opening this as a draft to remind me and come back and make a clean PR later.

Thanks for the lib. I used it on previous versions of our meshtastic project, and just now added a device that has an LCD screen.

geeksville and others added 9 commits February 21, 2020 10:08
    On my app I noticed that each time I called setTargetFPS the transitions
    would become faster and faster, until finally they don't even take one
    frame.  The problem was that the FP math for calculating changeRatio
    would (because of rounding err) keep shrinking the number of ticksPerFrame.

    This change instead computs ticksPerFrame/perTransition based on the
    requested display time, so error can't accumulate from a series of calls.

    (In my app I only bump up target FPS before each transition but then
    I lower it down again afterwards to save CPU)
(Also makes it easier to share code when users of library are targeting
multiple platforms)
- allow -1 as value for _sda and _scl parameters
  (see:
  6a56151
  )
- use same autoinit logic
Treat SH1106 controller the same as SSD1306
@geeksville geeksville marked this pull request as draft October 23, 2020 09:47
@sachaw
Copy link

sachaw commented Mar 9, 2022

@marcelstoer Hey, any chance we could get these fixes merged, we're currently trying to minimize the amount of out-of-sync forks we have, thanks.

@marcelstoer
Copy link
Member

We ignore PRs labelled as 'Draft' by the authors. We'll take a look once the author considers it done.

@mc-hamster
Copy link

@sachaw > Hey, any chance we could get these fixes merged, we're currently trying to minimize the amount of out-of-sync forks we have, thanks.

If memory serves, this change has an issue with one of Meshtastic build targets. That's why it's still in draft.

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.

None yet

10 participants