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

CMSIS-Core(M) Doc: added info about deprecated symbols. #161

Merged
merged 2 commits into from
May 13, 2024

Conversation

vovamarch
Copy link
Collaborator

No description provided.

@vovamarch vovamarch requested review from JonatanAntoni and a user May 10, 2024 13:06
JonatanAntoni
JonatanAntoni previously approved these changes May 13, 2024
Copy link

@ghost ghost left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

All core_xxx.h core header files contain a defgroup CMSIS_register_aliases which must be renamed to CMSIS_deprecated_aliases. This PR misses some core_xxx.h files.
The backward compatibility misses the typo correction for ENDIANNESS (PR #150).

@vovamarch
Copy link
Collaborator Author

All core_xxx.h core header files contain a defgroup CMSIS_register_aliases which must be renamed to CMSIS_deprecated_aliases. This PR misses some core_xxx.h files. The backward compatibility misses the typo correction for ENDIANNESS (PR #150).

Aligned the core header files to all have defgroup CMSIS_register_aliases and deprecated items under #ifndef CMSIS_DISABLE_DEPRECATED

Copy link

@ghost ghost left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

looks good to me.

@JonatanAntoni JonatanAntoni merged commit fa6d6da into ARM-software:main May 13, 2024
7 checks passed
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