Skip to content

Conversation

@abhishek2desh
Copy link

ESP32-C3-Devkit stable.

ESP32-C3-Devkit stable.
@blazoncek
Copy link
Contributor

Why? I have C3 and it is running stable with current set-up.
I would prefer not to use external platforms if not necessary.

Other than that, making unnecessary changes in platformio.ini is not welcome.

Copy link
Contributor

@blazoncek blazoncek left a comment

Choose a reason for hiding this comment

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

If you want custom build environment use platformio_override.ini and do not change upstream platformio.ini.


# Release / CI binaries
default_envs = nodemcuv2, esp8266_2m, esp01_1m_full, esp32dev, esp32_eth, esp32s2_saola, esp32c3, esp32s3dev_8MB
# default_envs = nodemcuv2, esp8266_2m, esp01_1m_full, esp32dev, esp32_eth, esp32s2_saola, esp32c3, esp32s3dev_8MB
Copy link
Contributor

Choose a reason for hiding this comment

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

Don't change that.


[env:esp32c3]
platform = espressif32@5.1.1
platform = https://github.com/tasmota/platform-espressif32/releases/download/v2.0.3rc1/platform-espressif32-2.0.3new.zip
Copy link
Contributor

Choose a reason for hiding this comment

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

Why not stock? Where is the issue with stock platform?

board_build.partitions = tools/WLED_ESP32_4MB_1MB_FS.csv
build_flags = ${common.build_flags} ${esp32c3.build_flags} #-D WLED_RELEASE_NAME=ESP32-C3
-D WLED_WATCHDOG_TIMEOUT=0
-D LEDPIN=2
Copy link
Contributor

Choose a reason for hiding this comment

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

Not needed.

@softhack007
Copy link
Member

Sorry but I will not approve this PR as it is today.
Reasons as explained by @blazoncek.

@abhishek2desh
Copy link
Author

abhishek2desh commented Dec 16, 2022 via email

@Aircoookie
Copy link
Member

I would support adding the tasmota-compiled platform as a commented out option. The other changes to platformio will not get merged.

@softhack007
Copy link
Member

Thanks @Aircoookie this solution seems to be a good compromise.

@softhack007
Copy link
Member

I have been playing around with a -C3 recently, and also saw issues with corrupted littlefs, that happen with espressif@5.1.1 in combination with ARDUINO_USB_CDC_ON_BOOT=0. These problems disappear when using espressif@5.2.0. Espressif@5.1.1 seems to work when ARDUINO_USB_CDC_ON_BOOT=1. So my proposal is to try with espressif@5.2.0.

@abhishek2desh can you explain a bit more why you picked platform=https://github.com/tasmota/platform-espressif32/releases/download/v2.0.3rc1/platform-espressif32-2.0.3new.zip ? It seems to be a pre-release from tasmota, based on arduino-esp32 v2.0.3 in "tasmota flavour". What makes this one special?

@blazoncek
Copy link
Contributor

@softhack007 indeed ARDUINO_USB_CDC_ON_BOOT caused many issues as well with S2 when I was testing it (corrupting LittleFS and/or rebooting/panicking).

@softhack007
Copy link
Member

A few users have confirmed already that littlefs corruption on -C3 is resolved by using platform=espressif@5.2.0. So I prefer to have this as an optional line in platformio.ini, instead of an older tasmota pre-release version.

Closing this PR, in favour of commit b8cc783

@Paras-PC
Copy link

Why? I have C3 and it is running stable with current set-up. I would prefer not to use external platforms if not necessary.

Other than that, making unnecessary changes in platformio.ini is not welcome.

Which file did you flash on your ESP-C3?
I want to flash a Seeed Studio XIAO ESP32C3

@blazoncek
Copy link
Contributor

Which file did you flash on your ESP-C3? I want to flash a Seeed Studio XIAO ESP32C3

My own compile from PIO.
Env posted on Discord.

@softhack007 softhack007 added cannot reproduce Developers are not able reproduce. Might be fixed already, or report is missing important details board request PR adding support for a specific board. labels Jan 3, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

board request PR adding support for a specific board. cannot reproduce Developers are not able reproduce. Might be fixed already, or report is missing important details

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants