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.

Sign up for GitHub

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

Jump to bottom

fix(CMSIS): Regenerate trng_regs.h using correct SVD file for MAX32670 #692

Merged
merged 1 commit into from
Aug 10, 2023
Merged

fix(CMSIS): Regenerate trng_regs.h using correct SVD file for MAX32670 #692

merged 1 commit into from
Aug 10, 2023

Conversation

sihyung-maxim
Copy link
Contributor

@sihyung-maxim sihyung-maxim commented Jul 28, 2023 •
edited
Loading

Description

The incorrect SVD file was used to generate the ME15's trng_regs.h file. Also, the ME15 only has one AES key.

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.

@github-actions github-actions bot added MAX32670 Related to the MAX32670 (ME15) Register Change This issue or pull request involves a change to the MSDK registers. labels Jul 28, 2023
@Jake-Carter Jake-Carter merged commit bb5e6af into main Aug 10, 2023
16 of 18 checks passed
@Jake-Carter Jake-Carter deleted the fix/me15_trng branch August 10, 2023 20:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Jacob-Scheiffler Jacob-Scheiffler Jacob-Scheiffler approved these changes

@Jake-Carter Jake-Carter Jake-Carter approved these changes

@cwarer1 cwarer1 cwarer1 approved these changes

Assignees
No one assigned
Labels
MAX32670 Related to the MAX32670 (ME15) Needs Review Register Change This issue or pull request involves a change to the MSDK registers.
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@sihyung-maxim @cwarer1 @Jake-Carter @Jacob-Scheiffler