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

feat(CMSIS): Add CMSIS 6.0 support to MAX32657 #1180

Merged
merged 1 commit into from
Sep 23, 2024

Conversation

ozersa
Copy link
Contributor

@ozersa ozersa commented Sep 19, 2024

Use cmsis_gcc_m for MAX32657 if CMSIS version equal to v6.0
With CMSIS 6.0 cmsis_gcc.h file has been replaced with cmsis_gcc_m.h.
TF-M uses CMSIS v6.0 to support TF-M without adding a wrapper file max32657.h file has been updated.

After cmsis v6.0 (v6.1) , cmsis_gcc.h file added inside CMSIS files so exception case should be executed just for v6.0

Checklist Before Requesting Review

  • PR Title follows correct guidelines.
  • Description of changes and all other relevant information.
  • (Optional) Link any related GitHub issues using a keyword
  • (Optional) Provide info on any relevant functional testing/validation. For API changes or significant features, this is not optional.

With CMSIS 6.0 cmsis_gcc.h file has been replaced with cmsis_gcc_m.h
TF-M uses CMSIS v6.0 to support it without adding a wrapper file
max32657.h file has been updated

https://github.com/ARM-software/CMSIS_6/blob/v6.0.0/CMSIS/Core/Include/cmsis_version.h
https://github.com/zephyrproject-rtos/trusted-firmware-m/blob/main/platform/ext/cmsis/CMSIS/Core/Include/cmsis_version.h

Signed-off-by: Sadik Ozer <sadik.ozer@analog.com>
@sihyung-maxim sihyung-maxim changed the title feat(CMSIS): To MAX32657 support CMSIS 6.0 feat(CMSIS): Add CMSIS 6.0 support to MAX32657 Sep 19, 2024
@sihyung-maxim sihyung-maxim merged commit cc12853 into main Sep 23, 2024
17 checks passed
@sihyung-maxim sihyung-maxim deleted the feat/to_support_cmsis6 branch September 23, 2024 17:43
sihyung-maxim pushed a commit to analogdevicesinc/hal_adi that referenced this pull request Sep 23, 2024
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.

2 participants