Skip to content

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

@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

2 participants