-
Notifications
You must be signed in to change notification settings - Fork 1k
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
Remove all references to tft_setup.h and Sketch_with_tft_setup from code #2746
Comments
Maybe there is more discussion on this elsewhere, but I was always annoyed by the very idea of a requirement to "tamper with the library and its directory tree" to use it. Unless my stupidity makes me unable to understand how it should work, that breaks the rational of things like github and the updating of libraries. Since I didn't want to go that way, I copied the setup to my own sketch directory and added "#define USER_SETUP_LOADED // Indicates a specific setup is already loaded" to make it work. Much cleaner. |
I completely agree. It took 2 weeks to try to launch an ESP32S3 with a GC9A01 screen. the screen refused to work. I tried to enter #define USE_HSPI_PORT, USE_VSPI_PORT, and manually entered #define MISO 11 MOSI 11 CS10, but nothing helped. Полностью согласен. 2 недели ушло на попытку запустить ESP32S3 с экраном GC9A01. экран отказывался работать. Пробовал прописывать #define USE_HSPI_PORT, USE_VSPI_PORT, вручную прописывал #define MISO 11 MOSI 11 CS10, но ничего не помогало. |
Changes had to be made to TFT_eSPI_ESP32_S3.h Изменения пришлось вносить в TFT_eSPI_ESP32_S3.h
|
It appears at some point the capability to include a
tft_setup.h
file (versus tampering with the library and its directory tree) was deprecated. If that's final, then please remove all the remaining references totft_setup.h
andSketch_with_tft_setup
from the codebase. Otherwise people will continue to spend significant time attempting to do something that's actually impossible with this library as it is today, based on all the references out there to this absent capability.The text was updated successfully, but these errors were encountered: