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

Add lilyGo T-Display S3 board #653

Merged
merged 1 commit into from
Jan 5, 2025

Conversation

epiHATR
Copy link
Contributor

@epiHATR epiHATR commented Jan 5, 2025

Proposed Changes

In this PR, I have made some changes:

  • Add lilyGo T-Display S3 board configuration and logic handler
  • Change the source of ESP-PN532BLE to my folk repo since it's unable to compile using current code

Types of Changes

Add new board

Verification

Tested on LilyGo T-Display S3 with 2 default buttons and one additional button as bellow:

  • BOOT button: act as Previous
  • KEY button: act as Next button
  • Custom button at GPIO1 : Select button, startup button
  • Hold Boot & Key buttons for power off the board

IMG_6260

Testing

Tested in default board T-Display S3 by lilyGo

@epiHATR epiHATR force-pushed the add-liligo-t-display-s3-board branch 2 times, most recently from 72d9ba8 to dca665f Compare January 5, 2025 09:14
@epiHATR epiHATR force-pushed the add-liligo-t-display-s3-board branch from dca665f to 9e82de9 Compare January 5, 2025 09:27
@pr3y pr3y merged commit 1b1a7ab into pr3y:main Jan 5, 2025
5 checks passed
@epiHATR epiHATR deleted the add-liligo-t-display-s3-board branch January 5, 2025 14:20
@@ -72,8 +73,8 @@ lib_deps =
https://github.com/rennancockles/MFRC522-I2C
https://github.com/rennancockles/ESP-ChameleonUltra
https://github.com/rennancockles/ESP-Amiibolink
https://github.com/whywilson/ESP-PN532BLE
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

You probably have an older version in cache..

delete C:\Users\YourUser\.platformio\.cache folder

I will revert this library change

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I have updated in the new PR

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.

3 participants