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

WIP: Add support for EPD 3in7 #69

Closed
wants to merge 1 commit into from

Conversation

Alexx-G
Copy link
Contributor

@Alexx-G Alexx-G commented Apr 3, 2021

Relates to #68

I've relied on driver implementation for STM32 and RPi.

@Alexx-G Alexx-G mentioned this pull request Apr 3, 2021
@caemor caemor changed the title Add support for EPD 3in7 - WIP WIP: Add support for EPD 3in7 Apr 14, 2021
@caemor
Copy link
Owner

caemor commented May 16, 2021

Hey Alexx,
how is your current status?
greetings,
Chris

@mangelajo
Copy link
Contributor

I am preparing an updated PR for this display, as I recently got one looking for a fast-refresh display.

@mangelajo
Copy link
Contributor

I am preparing an updated PR for this display, as I recently got one looking for a fast-refresh display.

Nice I got it working with the latest release. One of the issues seemed to be that the busy signal reading was inverted.

I have also verified that the Box::new(Displayxxx:default()) approach works, I'll document that in a separate PR too.

@caemor I guess that based on previous discussions the approach for 4-gray/1-gray is having 2 different versions of the display driver, right?

@peckpeck
Copy link
Contributor

Is it of any use having a B/W only version ? Is it any faster ?
If no a simple 4 gray implementation should be enough

@mangelajo
Copy link
Contributor

I have posted the PR here #129

We could close this PR at this point, original authorship is retained in the commit

@mangelajo
Copy link
Contributor

Is it of any use having a B/W only version ? Is it any faster ? If no a simple 4 gray implementation should be enough

Memory usage is half, for 4 grayscale it doesn't work in ESP32C3 as far as I checked.

I will do timing checks , and try to factor-out common parts across 4gray/BW.

@caemor caemor closed this Nov 30, 2022
@peckpeck
Copy link
Contributor

i think esp32c3 has enough memory, but i understand the argument

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.

4 participants