DRAFT: Add support for the ESP32-C5 and move forward to IDF 5.4 (not final)#3670
Open
pjsg wants to merge 21 commits intonodemcu:dev-esp32from
Open
DRAFT: Add support for the ESP32-C5 and move forward to IDF 5.4 (not final)#3670pjsg wants to merge 21 commits intonodemcu:dev-esp32from
pjsg wants to merge 21 commits intonodemcu:dev-esp32from
Conversation
- If the initial filesystem mounting fails, then a subsequent file.format() does not mount it afterwards, necessitating a node.restart(). In the normal case, after file.format() the filesystem is mounted and ready for use. We now explicitly remount after a format to gain consistent behaviour. - If using USB CDC, the auto-format on first boot interferes with USB enumeration, resulting in a failed startup. We now avoid doing an auto-format if the USB CDC console is enabled. The behaviour is configurable/overrideable via Kconfig.
This allegedly resolves the reboot crashes seen on ESP32-S3 Zero. It also removes the peculiar need to explicitly call esp_vfs_console_register(), the absence of which previously would trigger an assertion failure in xQueueGenericSend on input.
To help compile-test the different console options, for starters.
Having gone through the changelog from 5.3.1, this looks safe enough...
pjsg
changed the title
Member
Author
|
It is quite tricky to make it connect to a 5GHz network, but I did succeed: {"bssid":"26:5a:4c:5d:73:fd","ssid":"Chateau Gladstone 5GHz","channel":36,"auth":3} If I tried to connect to a network with both 2.4 and 5.8 channels, then it always picked the 2.4 channel -- it didn't seem to make any difference what channel / scan_method I picked. |
Member
Author
|
I don't think that this branch should be merged until the version of the IDF is released that supports the C5. Not clear when that will be. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
This is just a tracking PR. The ESP32C5 branch builds in the CICD, but some of the other devices do not build as some of the modules don't build due to changes in the IDF.
Fixes #3671.
Make sure all boxes are checked (add x inside the brackets) when you submit your contribution, remove this sentence before doing so.
devbranch rather than for thereleasebranch.docs/*.To be completed below: Description of and rationale behind this PR.