Skip to content

Conversation

@softhack007
Copy link
Contributor

@softhack007 softhack007 commented Feb 6, 2024

pio

  • new environment for -C3
  • moved ARDUINO_USB_CDC_ON_BOOT=0 out of [env], as this flag is meant for "classic esp32" and may cause conflicts on other variants.
  • new buildenv for esp32 "pico32" board (not a -C3, but a bit special)
  • update WLED-Sync (due to broken library.json)

AppModLeds:

  • allowed pins on -C3
  • some pin cleanups and corrections for esp32, esp32-S2

pio
* new environment for -C3
* moved DARDUINO_USB_CDC_ON_BOOT=0 out of [env], as this flag is for "classic esp32" and may cause problems on other variants.

AppModLeds:
* allowed pins on -C3
* some pin cleanups and corrections for esp32, esp32-S2
use WLED-Sync with fixed manifest file
;; platform = espressif32@5.3.0 ;; WLED default framework version
platform = espressif32@6.3.0 ;; this one behaves better for debugging
upload_speed = 256000
build_unflags =
Copy link
Owner

Choose a reason for hiding this comment

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

Can build_unflags be removed now it is not set in [env] anymore?

Copy link
Contributor Author

@softhack007 softhack007 Feb 8, 2024

Choose a reason for hiding this comment

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

Well not strictly needed if you want to use native USB for debugging, however still needed for boards like this that also have a "real" serial UART-to-USB port.

https://docs.espressif.com/projects/esp-idf/en/latest/esp32s2/hw-reference/esp32s2/user-guide-s2-devkitc-1.html

For most -S2/-C3, the board.json includes -DARDUINO_USB_CDC_ON_BOOT=1, but for serial UART-to-USB you need -DARDUINO_USB_CDC_ON_BOOT=0.

@ewoudwijma ewoudwijma merged commit 0744f70 into ewoudwijma:main Feb 8, 2024
@softhack007 softhack007 deleted the C3_support branch February 17, 2024 19:11
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.

2 participants