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

fixed waveshare_esp32_s3_touch_lcd_169 waveshare_esp32_s3_lcd_169 bugs, Added variant for Waveshare ESP32-S3-Zero #1448

Open
wants to merge 5 commits into
base: develop
Choose a base branch
from

Conversation

Y1hsiaochunnn
Copy link

Description of Change

fixed waveshare_esp32_s3_touch_lcd_169 waveshare_esp32_s3_lcd_169 bugs,
They're mentioned here:#1436

Added variant for Waveshare ESP32-S3-Zero

Tests scenarios

Untested, pin definition is based on information in Waveshare wiki.

Related links

https://www.waveshare.com/wiki/ESP32-S3-Zero

arduino-esp32 PR espressif/arduino-esp32#10241

boards/waveshare-esp32-s3-zero.json Outdated Show resolved Hide resolved
boards/waveshare-esp32-s3-zero.json Show resolved Hide resolved
@valeros
Copy link
Member

valeros commented Aug 26, 2024

Hi @Y1hsiaochunnn, please review my comments above.

@Y1hsiaochunnn Y1hsiaochunnn requested a review from valeros August 27, 2024 01:03
@Y1hsiaochunnn
Copy link
Author

Hi @valeros,
revised it according to your suggestion

@valeros
Copy link
Member

valeros commented Aug 27, 2024

LGTM, the PR will be merged once support for Arduino core v3 is added.

@Y1hsiaochunnn
Copy link
Author

thanks alot

@Y1hsiaochunnn
Copy link
Author

Hi @valeros ,
arduino-esp32 has pre-released V.3.1.0-RC1 and has merged ESP32-S3-Touch-LCD-1.69, ESP32-S3-LCD-1.69 and ESP32-S3-Zero. Is platformio ready to merge this pr and release it?

@valeros
Copy link
Member

valeros commented Sep 2, 2024

@Y1hsiaochunnn As I already mentioned above, the PR will be merged once support for Arduino core v3 is added, see #1225

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants