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

Deek-Robot TFT #191

Closed
samo-sk opened this issue Aug 7, 2023 · 3 comments · May be fixed by #193
Closed

Deek-Robot TFT #191

samo-sk opened this issue Aug 7, 2023 · 3 comments · May be fixed by #193

Comments

@samo-sk
Copy link

samo-sk commented Aug 7, 2023

Hello,
I have a "Deek-Robot" TFT, which looks exactly like one pictured here, and uses ST7735. None of the options for initR works well – the screen size is OK for INITR_GREENTAB and INITR_BLACKTAB, but when using INITR_GREENTAB, red is swapped with blue, and when using INITR_BLACKTAB, the image on the display is shifted. This issue is the same as the one described in #180, but I'm not sure what type of display is used by the user who reported it. This issue is also described here.
The library works perfectly when initR(INITR_BLACKTAB) is used, but it is modified to set _colstart to 2 and _rowstart to 1 (like when INITR_GREENTAB is used). Could you please modify this library (more specifically, Adafruit_ST7735.cpp and Adafruit_ST7735.h) to create another option for initR for initializing the display like mentioned above? Also setRotation will need to be modified to behave just like when tabcolor is INITR_BLACKTAB when the new option is used.
Thank you.

@ladyada
Copy link
Member

ladyada commented Aug 9, 2023

please ask DEEK to submit a PR to help and we will review it!

@ladyada ladyada closed this as completed Aug 9, 2023
@samo-sk
Copy link
Author

samo-sk commented Aug 10, 2023

I have noticed this issue is a duplicate of #154.

@samo-sk
Copy link
Author

samo-sk commented Aug 11, 2023

Information for anybody reading this issue report after its closure:
The proposed fix is (in theory) incorrect, as the values of offsets (_colstart and _rowstart) are also present in Rcmd2green, a part of initialization code for displays with the mentioned offsets. However, the proposed change incorrectly uses the init code for blacktab (Rcmd2red), which is offset-less. However, this issue has no effect in practice, as the effects of Rcmd2green/Rcmd2red are overriden by Adafruit_ST77xx::setAddrWindow, which is called before anything is drawn on the display.

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 a pull request may close this issue.

2 participants