Skip to content

change(esp_tinyusb): Update esp_tinyusb task affinity config type#269

Closed
Dazza0 wants to merge 1 commit intoespressif:masterfrom
Dazza0:change/update_tinyusb_affinity_config
Closed

change(esp_tinyusb): Update esp_tinyusb task affinity config type#269
Dazza0 wants to merge 1 commit intoespressif:masterfrom
Dazza0:change/update_tinyusb_affinity_config

Conversation

@Dazza0
Copy link
Contributor

@Dazza0 Dazza0 commented Nov 6, 2023

Change description

Please describe your change here

ESP-IDF updates the task affinity config option from hex to int, and provides constant values for each affinity value. This commit syncs TINYUSB_TASK_AFFINITY with those changes.

See internal MR !26701 for more details.

ESP-IDF updates the task affinity config option from hex to int, and provides
constant values for each affinity value. This commit syncs TINYUSB_TASK_AFFINITY
with those changes.
@CLAassistant
Copy link

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.
You have signed the CLA already but the status is still pending? Let us recheck it.

@Dazza0
Copy link
Contributor Author

Dazza0 commented Nov 6, 2023

@tore-espressif Do we need to bump the bugfix version for every change?

@tore-espressif
Copy link
Collaborator

@Dazza0 There isn't any actively developed feature ATM, so if you want to see this change deployed soon, you can bump the version.
Please check the failing jobs, too

cc @Jacques-Zhao do you want to merge #235 ?

@Dazza0
Copy link
Contributor Author

Dazza0 commented Nov 8, 2023

@tore-espressif Sorry, looks like there's not a good way to introduce this change to the managed components in a non-breaking manner. I'll closing this PR (and associated MR) and will delay this change until ESP-IDF v6.0.

@Dazza0 Dazza0 closed this Nov 8, 2023
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.

3 participants

Comments