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

Set ESP32 CONFIG_FREERTOS_UNICORE=y by default #2696

Merged
merged 1 commit into from
Jan 6, 2024

Conversation

mikee47
Copy link
Contributor

@mikee47 mikee47 commented Jan 5, 2024

As discussed in #2653 there is an issue on dual-core CPUs which causes hang and WDT on IDF 5.0 (and 5.2).
Setting CONFIG_FREERTOS_UNICORE=y fixes the problem. This setting is described as follows:

This version of FreeRTOS normally takes control of all cores of the CPU. Select this if you only want to start it on the first core. This is needed when e.g. another process needs complete control over the second core.

I'd suggest this is a better default behaviour anyway.

Copy link

what-the-diff bot commented Jan 5, 2024

PR Summary

  • Simplifying Start-Up Process
    A new comment has been added to make sure that we don't initialize the Watchdog Timer (WTD) at startup. This reduces the complexity at system initiation.

  • Improving Compatibility with Dual-Core CPUs
    A new configuration has been put in place to address complications encountered while dealing with dual-core CPUs. This improvement will ensure smooth operation and improved performance when our software is run on a system with multiple processing units.

@slaff slaff added this to the 5.2.0 milestone Jan 6, 2024
@slaff slaff merged commit 2ab9794 into SmingHub:develop Jan 6, 2024
36 checks passed
@slaff slaff mentioned this pull request Feb 5, 2024
5 tasks
@mikee47 mikee47 deleted the fix/esp32-unicore branch March 19, 2024 15:18
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