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

Freertos wifi examples don't build with the latest version of FreeRTOS-Kernel #428

Closed
peterharperuk opened this issue Oct 4, 2023 · 4 comments

Comments

@peterharperuk
Copy link
Contributor

FreeRTOS/FreeRTOS-Kernel@ae3a498

peterharperuk added a commit to peterharperuk/pico-examples that referenced this issue Oct 4, 2023
peterharperuk added a commit to peterharperuk/pico-examples that referenced this issue Oct 4, 2023
@kilograham
Copy link
Contributor

OK, so this is a bit more involved; they have merged the smp branch into the main branch, so we will need to do some more investigation of if anything got broken (mixture of FreeRTOS cores, and SDK cores etc) for the next SDK release

peterharperuk added a commit to peterharperuk/pico-examples that referenced this issue Oct 24, 2023
@raarts
Copy link

raarts commented Jan 4, 2024

FreeRTOS 11.0 (Released in December) now defines FREE_RTOS_KERNEL_SMP instead of portSUPPORT_SMP.
Also, the examples don't run at all if you use both cores, generating the following error in picow_freertos_ping.c:

assertion "get_core_num() == async_context_core_num(cyw43_async_context)" failed: file "/Users/raarts/pico/pico-sdk/src/rp2_common/pico_cyw43_driver/cyw43_driver.c", line 53, function: cyw43_irq_init

I don't understand enough of the code to try to fix it (FreeRTOS noob here).

@jdbruner
Copy link

jdbruner commented Mar 24, 2024

It looks like FreeRTOS is looking for configNUMBER_OF_CORES but the SDK and example code is using configNUM_CORES. A quick workaround is to define them to be the same thing.

@lurch
Copy link
Contributor

lurch commented Mar 25, 2024

It looks like FreeRTOS is looking for configNUMBER_OF_CORES but the SDK and example code is using configNUM_CORES. A quick workaround is to define them to be the same thing.

See #429 😉

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

No branches or pull requests

5 participants