Rename configNUMBER_OF_CORES to configNUM_CORES #1530

Closed
peterharperuk wants to merge 1 commit from fix_freertos_smp_cyw43_assert into develop
peterharperuk commented 2023-10-30 09:09:49 -05:00 (Migrated from github.com)

The Freertos SMP branch has been merged into main with configNUM_CORES renamed to configNUMBER_OF_CORES

Fixes #1526

The Freertos SMP branch has been merged into main with configNUM_CORES renamed to configNUMBER_OF_CORES Fixes #1526
kilograham (Migrated from github.com) reviewed 2023-10-30 09:09:49 -05:00
jaguilar commented 2024-03-26 00:13:07 -05:00 (Migrated from github.com)

It would be really great if this got merged. I just spent a bunch of time trying to get things working, only to figure out that it all wasn't working because the affinity set wasn't being updated since the pico SDK thinks that there is only one core (since it's looking at the wrong #define).

At a minimum, maybe we could put an #error somewhere that indicates that FreeRTOS 11 is not supported yet?

It would be really great if this got merged. I just spent a bunch of time trying to get things working, only to figure out that it all wasn't working because the affinity set wasn't being updated since the pico SDK thinks that there is only one core (since it's looking at the wrong #define). At a minimum, maybe we could put an #error somewhere that indicates that FreeRTOS 11 is not supported yet?
peterharperuk commented 2024-08-09 08:15:29 -05:00 (Migrated from github.com)

Fixed

Fixed

Pull request closed

Sign in to join this conversation.
No reviewers
No milestone
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set

Reference
gh-raspberrypi/pico-sdk!1530
No description provided.