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

Fix SMP debugging issue on rp2040 #1174

Merged
merged 5 commits into from
Nov 4, 2024

Conversation

vishwamartur
Copy link
Contributor

Related to #1172

Add multicore_reset_core1 before multicore_launch_core1 in xPortStartScheduler function to fix debugging issue on rp2040 with SMP enabled.

  • Modify portable/ThirdParty/GCC/RP2040/port.c to include a call to multicore_reset_core1 before multicore_launch_core1 within the #if portRUNNING_ON_BOTH_CORES block.

Related to FreeRTOS#1172

Add `multicore_reset_core1` before `multicore_launch_core1` in `xPortStartScheduler` function to fix debugging issue on rp2040 with SMP enabled.

* Modify `portable/ThirdParty/GCC/RP2040/port.c` to include a call to `multicore_reset_core1` before `multicore_launch_core1` within the `#if portRUNNING_ON_BOTH_CORES` block.
@vishwamartur vishwamartur requested a review from a team as a code owner November 1, 2024 06:33
@0x3333
Copy link

0x3333 commented Nov 1, 2024

This works. For future reference: My last comment in the issue, about the Timer issue this is related to.

@aggarg
Copy link
Member

aggarg commented Nov 4, 2024

Reached out to Raspbery Pi to confirm that they are okay with the change.

@vishwamartur
Copy link
Contributor Author

Hi @aggarg and team,

Thanks for the update and for reaching out to Raspberry Pi for confirmation. Please let me know if any additional input or verification is needed from my end. I'll be happy to assist further.

Thanks again for your support!

Copy link

sonarcloud bot commented Nov 4, 2024

@aggarg aggarg merged commit f0d7945 into FreeRTOS:main Nov 4, 2024
16 checks passed
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.

6 participants