Skip to content
This repository was archived by the owner on Dec 18, 2025. It is now read-only.

Update core_cm33.h#1254

Open
AlbertHuang-CPU wants to merge 4 commits into
ARM-software:developfrom
AlbertHuang-CPU:patch-2
Open

Update core_cm33.h#1254
AlbertHuang-CPU wants to merge 4 commits into
ARM-software:developfrom
AlbertHuang-CPU:patch-2

Conversation

@AlbertHuang-CPU

Copy link
Copy Markdown
Contributor

Correct a typo.

Correct a typo.
@JonatanAntoni

Copy link
Copy Markdown
Member

@AlbertHuang-CPU, thanks for pointing this out. We need to be carefull in renaming struct members. This can easily break exiting user applications which should be avoided as much as possible.

Comment thread CMSIS/Core/Include/core_cm33.h

@JonatanAntoni JonatanAntoni left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

All touched files needs to receive an update to their header information (i.e. file date and version).

@ARM-software ARM-software deleted a comment from grasci-arm Jun 23, 2022
@JonatanAntoni

Copy link
Copy Markdown
Member

@AlbertHuang-CPU, are you going to finish this patch?

@AlbertHuang-CPU

Copy link
Copy Markdown
Contributor Author

Hi, what need to do? to update the file date or to resolve conversation in the codes, or?

@jkrech

jkrech commented Jun 24, 2022

Copy link
Copy Markdown
Collaborator

Yes, please.

@JonatanAntoni

Copy link
Copy Markdown
Member

Outdated

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants