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

ST7735 Fixups #8

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

ST7735 Fixups #8

wants to merge 2 commits into from

Conversation

jkearins
Copy link

Hi loboris,

  1. My ST7735 display works with DEFAULT_DISP_TYPE as DISP_TYPE_ILI9341 but does not work with DISP_TYPE_ST7735, DISP_TYPE_ST7735B, DISP_TYPE_ST7735R. I corrected the code and now my display works as DISP_TYPE_ST7735B and DISP_TYPE_ST7735R.
  2. I think that the init sequence for 7735R in tftspi.c must be:
STP7735R_init
Rcmd2red  (not Rcmd2green)
Rcmd3
  1. Also you can add new type 7735G with init sequence:
STP7735R_init
Rcmd2green
Rcmd3

but for Green Label some offsets should be released:

colstart = 2;
rowstart = 1;

Best regards

gameofyou pushed a commit to gameofyou/ESP32_TFT_library that referenced this pull request Jul 15, 2020
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