-
Notifications
You must be signed in to change notification settings - Fork 7.7k
Description
Summary
CMSIS 5 is deprecated upstream for a long time now and so CMSIS_6 modules was added to Zephyr main as its successor.
It is also part of 4.2 release and as part of the migration guide a request was made to module owners who are still using the older CMSIS to migrate to CMSIS_6. This issue is created to list all the modules we identified so far using the older CMSIS and to track the progress of their migration.
As per this job, below are the number of failures with platforms:
- apollo3_evb/apollo3_blue, apollo4p_blue_kxr_evb/apollo4p_blue, apollo4p_evb/apollo4p, rak11720/apollo3_blue
- mec_assy6941/mec1743_qlj, mec_assy6941/mec1743_qsz, mec_assy6941/mec1753_qlj, mec_assy6941/mec1753_qsz
- mimxrt1064_evk/mimxrt1064
- siwx917_dk2605a/siwg917m111mgtba, siwx917_rb4342a/siwg917m111mgtba, siwx917_rb4338a/siwg917m111mgtba
Describe the solution you'd like
We would like all module owners to use CMSIS_6 for Cortex-M targets.
This can be started simply by just replacing cmsis macros reported in the above ci job with the suggested cmsis_6's macros for now but the module owners should also check if all their use cases are satisfied with cmsis_6 and report here if somethings is missing.
Alternatives
To use a new config something like USE_CMSIS_5_MODULE for modules that want to continue using the older cmsis but for this the specific modules would have to answer the question "why cmsis and not cmsis_6?" first.
Additional Context
The main issue with having both cmsis and cmsis_6 is that they both get included in the build with older cmsis taking priority and it has resulted in issues like #93689 and PRs like #93707 being used to revert commits that passed in ci but failed externally.
This PR #93715 tried to fix the inclusion of the 2 modules but failed because of the modules listed above still using it.
Some more info is also provided in #93699