Corrected forced version number for TFT_eSPI #4423

spiff72 · 2024-12-24T06:24:26Z

in platformio_override.ini

details, and force an older version of dependent library TFT_eSPI.

netmindz · 2024-12-24T10:33:35Z

usermods/TTGO-T-Display/platformio_override.ini

+platform = ${esp32.platform}
+platform_packages = ${esp32.platform_packages}
+build_unflags = ${common.build_unflags}
+build_flags = ${common.build_flags} ${esp32.build_flags} -D WLED_RELEASE_NAME=\"ESP32\" 


Should be something like WLED_T-Display for the release name to indicate this is not a vanilla ESP32 build

Just fixed this - thanks!

Revised WLED_RELEASE_NAME to "WLED_T-Display"

Update platformio_override.ini

softhack007 · 2025-01-04T15:57:44Z

usermods/TTGO-T-Display/README.md

@@ -87,5 +69,7 @@ xtensa-esp32-elf-g++: fatal error: no input files
 ```
 try building again. Sometimes this happens on the first build attempt and subsequent attempts build correctly.

+Once the compilation is done and loaded onto the TTGO T-Display module, you may need to complete the initial setup of WLED via the AP interface, configure the WiFi settings, let it restart and then go to the LED preferences option in the WLED web interface.  The Data GPIO pin needs to be set to pin 2 (or whatever pin you used), and then reboot WLED on the device.  This should get the display working.  I am unable to get the overrides to change the initial pin setting of 16 (which conflicts with the display).


"I am unable to get the overrides to change the initial pin setting of 16 (which conflicts with the display)."

@spiff72 using -D DATA_PINS=2 did not work?
Actually any build_flags for pin assignment will be overwritten by your cfg.json, so you'll need to wipe your device first if you want to test default pins assigned in build_flags.

WLED/platformio_override.sample.ini

Lines 61 to 70 in ae4de27

; PIN defines - uncomment and change, if needed:

; -D DATA_PINS=2

; or use this for multiple outputs

; -D DATA_PINS=1,3

; -D BTNPIN=0

; -D IRPIN=4

; -D RLYPIN=12

; -D RLYMDE=1

; -D RLYODRAIN=0

; -D LED_BUILTIN=2 # GPIO of built-in LED

softhack007 · 2025-01-04T16:01:23Z

usermods/TTGO-T-Display/platformio_override.ini

 ;  -D IRPIN=4
 ;  -D RLYPIN=12
 ;  -D RLYMDE=1
+;   -D WLED_DISABLE_BROWNOUT_DET
+;PIN defines - uncomment and change, if needed:


Line 27 might be part of the problem you had with default pins - platformIO is very picky about indentations, and a comment starting at column 0 might be enough to "end" the multi-line string prematurely.

Fixed (hopefully)

I confirmed that this issue has been fixed, and the data pin is being reassigned correctly.

softhack007

In summary, it's still a bit unclear to me where the display driver flags are added.
As the usermod proposes a platformio_override.ini to be used for esp32, the buildenv in this override file should be focussed on what's needed to configure the usermod.

Also I'd like to have all debug outputs use DEBUG_PRINTLN() instead of Serial.println()

softhack007 · 2025-01-04T16:07:33Z

usermods/TTGO-T-Display/usermod.cpp

  tft.print("mA (estimated)");
+  Serial.println("Print estimated current");


Please use DEBUG_PRINTLN for all debug outputs.

softhack007 · 2025-01-04T16:10:21Z

usermods/TTGO-T-Display/usermod.cpp

@@ -92,10 +96,10 @@ bool displayTurnedOff = false;
 void userLoop() {

  // Check if we time interval for redrawing passes.
-  if (millis() - lastUpdate < USER_LOOP_REFRESH_RATE_MS) {
+  if (millis() - lastUpdate_mod < USER_LOOP_REFRESH_RATE_MS) {


Where does USER_LOOP_REFRESH_RATE_MS come from?
I did not see it defined in the usermod (may I overlooked something)

It's defined 5 lines prior (line 94)

softhack007 · 2025-01-04T16:14:44Z

usermods/TTGO-T-Display/platformio_override.ini

+#----------
+# SAMPLE
+#----------
+[env:WLED_T-Display]


Where do we define the pins needed by the display driver?
defines in the usermod are commented out, so there should be a different place?

Pins needed by the display are defined in the TFT_eSPI library when the user_setup_select.h file is modified (as described in the readme.md file).

softhack007 · 2025-01-04T16:16:53Z

usermods/TTGO-T-Display/platformio_override.ini

+;   -D LED_BUILTIN=2 # GPIO of built-in LED
+lib_deps = ${esp32.lib_deps}
+  #For use of the TTGO T-Display ESP32 Module with integrated TFT display uncomment the following line
+    ;TFT_eSPI @ 2.5.33


Any special reason that TFT library is commented out?

Yes - the following line is forcing an older version (2.1.4) instead.

softhack007 · 2025-01-04T16:20:30Z

usermods/TTGO-T-Display/platformio_override.ini

+monitor_filters = esp32_exception_decoder
+board_build.partitions = ${esp32.default_partitions}
+
+; # additional build flags for audioreactive


The part for audioreactive is outdated - please replace with the proposed flags from platformio_override.sample.ini

WLED/platformio_override.sample.ini

Lines 143 to 150 in ae4de27

; Use Audioreactive usermod and configure I2S microphone

; ${esp32.AR_build_flags} ;; default flags required to properly configure ArduinoFFT

; ;; don't forget to add ArduinoFFT to your libs_deps: ${esp32.AR_lib_deps}

; -D AUDIOPIN=-1

; -D DMTYPE=1 # 0-analog/disabled, 1-I2S generic, 2-ES7243, 3-SPH0645, 4-I2S+mclk, 5-I2S PDM

; -D I2S_SDPIN=36

; -D I2S_WSPIN=23

; -D I2S_CKPIN=19

WLED/platformio_override.sample.ini

Line 32 in ae4de27

; ${esp32.AR_lib_deps} ;; needed for USERMOD_AUDIOREACTIVE

softhack007 · 2025-01-04T16:52:25Z

@spiff72 another user has proposed some enhancements in

Refactor the ST7789_display usermod into a more generic TFT_display usermod #4072

Maybe both PRs should be merged together?

spiff72 · 2025-01-05T04:45:42Z

Refactor the ST7789_display usermod into a more generic TFT_display usermod Refactor the ST7789_display usermod into a more generic TFT_display usermod #4072

I'm sorry, but I'm not sure what to make of this one (or what you are suggesting). I see that the platformi_override_sample.ini for that usermod appears to define a lot of the things needed for the (different) display that board uses, but I think it's better to have the user edit the file in the library as called out in the readme.md file.

…i file.

spiff72 added 2 commits December 24, 2024 00:19

Updated to fix compilation errors, adapt to some of the new WLED

1466dde

details, and force an older version of dependent library TFT_eSPI.

fixed forced version number of TFT_eSPI in platformio_override.ini

79f3ef9

netmindz reviewed Dec 24, 2024

View reviewed changes

spiff72 added 2 commits December 24, 2024 13:04

Update platformio_override.ini

2646165

Revised WLED_RELEASE_NAME to "WLED_T-Display"

Merge pull request #1 from spiff72/spiff72-patch-1

7d5b67d

Update platformio_override.ini

softhack007 reviewed Jan 4, 2025

View reviewed changes

softhack007 requested changes Jan 4, 2025

View reviewed changes

softhack007 added the usermod usermod related label Jan 4, 2025

adjustments based on feedback

3325b0b

spiff72 added 2 commits January 4, 2025 23:48

Merge branch 'Aircoookie:main' into master

e28f045

Updated readme file and tweaked indents on the platformio_override.in…

98215f5

…i file.

spiff72 requested a review from softhack007 January 5, 2025 05:36

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Corrected forced version number for TFT_eSPI #4423

Corrected forced version number for TFT_eSPI #4423

spiff72 commented Dec 24, 2024

netmindz Dec 24, 2024

spiff72 Dec 24, 2024

softhack007 Jan 4, 2025

softhack007 Jan 4, 2025 •

edited

Loading

spiff72 Jan 5, 2025

spiff72 Jan 5, 2025

softhack007 left a comment

softhack007 Jan 4, 2025

spiff72 Jan 5, 2025

softhack007 Jan 4, 2025

spiff72 Jan 5, 2025

softhack007 Jan 4, 2025

spiff72 Jan 5, 2025

softhack007 Jan 4, 2025

spiff72 Jan 5, 2025

softhack007 Jan 4, 2025

spiff72 Jan 5, 2025

softhack007 commented Jan 4, 2025

spiff72 commented Jan 5, 2025

	; PIN defines - uncomment and change, if needed:
	; -D DATA_PINS=2
	; or use this for multiple outputs
	; -D DATA_PINS=1,3
	; -D BTNPIN=0
	; -D IRPIN=4
	; -D RLYPIN=12
	; -D RLYMDE=1
	; -D RLYODRAIN=0
	; -D LED_BUILTIN=2 # GPIO of built-in LED

		tft.print("mA (estimated)");
		Serial.println("Print estimated current");

	; Use Audioreactive usermod and configure I2S microphone
	; ${esp32.AR_build_flags} ;; default flags required to properly configure ArduinoFFT
	; ;; don't forget to add ArduinoFFT to your libs_deps: ${esp32.AR_lib_deps}
	; -D AUDIOPIN=-1
	; -D DMTYPE=1 # 0-analog/disabled, 1-I2S generic, 2-ES7243, 3-SPH0645, 4-I2S+mclk, 5-I2S PDM
	; -D I2S_SDPIN=36
	; -D I2S_WSPIN=23
	; -D I2S_CKPIN=19

Corrected forced version number for TFT_eSPI #4423

Are you sure you want to change the base?

Corrected forced version number for TFT_eSPI #4423

Conversation

spiff72 commented Dec 24, 2024

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

softhack007 Jan 4, 2025 • edited Loading

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

softhack007 left a comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

softhack007 commented Jan 4, 2025

spiff72 commented Jan 5, 2025

softhack007 Jan 4, 2025 •

edited

Loading