Skip to content

LILYGO_T_DISPLAY_S3 Macro #3703

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

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

LILYGO_T_DISPLAY_S3 Macro #3703

wants to merge 1 commit into from

Conversation

yhur
Copy link

@yhur yhur commented Apr 10, 2025

Dear Bodmer,

I have been enjoying your library while teaching students at a University and really appreciate that.

One thing I would like to add to this if you will is as follows.

The lab kit consists of LilyGo_T_Display_S3 and the students have been asked to modify the header file User_Setup_Select.h every time they created a new project under VSCode/PlatformIO. It's doable and not a big deal, but there are cases the students and I are spending time to deal with this instead of the main ideas.

So I would like to use the macro to have the project configured and use the right set of header files.
Once this is merged, the students just simply pass the macro in the platformio.ini as below.

Could you consider to merge this to your library?

[env:lilygo-t-display-s3]
platform = espressif32
board = lilygo-t-display-s3
framework = arduino
build_flags = -DARDUINO_USB_CDC_ON_BOOT=1 -DLILYGO_T_DISPLAY_S3
lib_deps = 
	bodmer/TFT_eSPI@^2.5.43

In order to use User_Setups/Setup206_LilyGo_T_Display_S3.h, just define or pass the macro LILYGO_T_DISPLAY_S3.
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