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(PeriphDrivers): Update RTC SEC and SSEC register size #859

Merged
merged 2 commits into from
Jan 22, 2024
Merged

fix(PeriphDrivers): Update RTC SEC and SSEC register size #859

merged 2 commits into from
Jan 22, 2024

Conversation

ozersa
Copy link
Contributor

@ozersa ozersa commented Jan 9, 2024

Description

RTC SEC and SSEC register size is 32bit and 12bits This value is set to 8bits for MAX32670, MAX32660, MAX32672.
This commit set register size to the correct values.

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.

Links:
https://www.analog.com/media/en/technical-documentation/user-guides/max32670max32671-user-guide.pdf
https://www.analog.com/media/en/technical-documentation/user-guides/max32672-user-guide.pdf
https://www.analog.com/media/en/technical-documentation/user-guides/max32660-user-guide.pdf

Register size as per of UGs
image

@github-actions github-actions bot added MAX32660 Related to the MAX32660 (ME11) MAX32670 Related to the MAX32670 (ME15) MAX32672 Related to the MAX32672 (ME21) MAX32675 Related to the MAX32675 (ME16) Register Change This issue or pull request involves a change to the MSDK registers. labels Jan 9, 2024
@ozersa
Update RTC SEC and SSEC register size
8318226 
RTC SEC and SSEC register size is 32bit and 12bits
This value is set to 8bits for MAX32670, MAX32660,
MAX32672.
This commit set register size to the correct values.

Signed-off-by: Sadik Ozer <[email protected]>
@ozersa ozersa force-pushed the fix/rtc_regs_size branch from 5c0e3a2 to 8318226 Compare January 9, 2024 07:25
Jake-Carter
Jake-Carter approved these changes Jan 9, 2024
View reviewed changes
Copy link
Contributor

@Jake-Carter Jake-Carter left a comment

Choose a reason for hiding this comment

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

Thanks @ozersa

@Jake-Carter
Copy link
Contributor

@sihyung-maxim do we need to run the register SVD workflow for this?

@sihyung-maxim
Copy link
Contributor

/generate-register-files

@sihyung-maxim
Copy link
Contributor

Our BTM-CI is under maintenance, and the workspace is outdated. Will have to fix next week.

@sihyung-maxim sihyung-maxim merged commit e20c2cf into main Jan 22, 2024
8 checks passed
@sihyung-maxim sihyung-maxim deleted the fix/rtc_regs_size branch January 22, 2024 15:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

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

@sihyung-maxim sihyung-maxim sihyung-maxim approved these changes

Assignees
No one assigned
Labels
MAX32660 Related to the MAX32660 (ME11) MAX32670 Related to the MAX32670 (ME15) MAX32672 Related to the MAX32672 (ME21) MAX32675 Related to the MAX32675 (ME16) 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.
3 participants
@ozersa @Jake-Carter @sihyung-maxim