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

fix(CMSIS): MSDK-1249: Add OVR Correction to SystemCoreClock Calculation for MAX32662 MAX32672 MAX32675 #867

Merged
merged 1 commit into from
Jan 13, 2024

Conversation

ahmetalincak
Copy link
Contributor

Description

MAX32662, MAX32672 and MAX32675 has the OVR bit to operate CPU voltage with different modes and this affects also the IPO clock. We need to consider this when calculating the system core clock.

PS. This has already done for MAX32660 in here

@github-actions github-actions bot added MAX32672 Related to the MAX32672 (ME21) MAX32675 Related to the MAX32675 (ME16) MAX32662 Related to the MAX32662 (ME12) labels Jan 12, 2024
@ahmetalincak ahmetalincak changed the title fix(CMSIS): Update SystemCoreClock calculation with considering the OVR fix(CMSIS): MSDK-1249: Update SystemCoreClock calculation with considering the OVR Jan 12, 2024
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 @ahmetalincak looks good!

@Jake-Carter Jake-Carter changed the title fix(CMSIS): MSDK-1249: Update SystemCoreClock calculation with considering the OVR fix(CMSIS): MSDK-1249: Add OVR Correction to SystemCoreClock Calculation for MAX32662 MAX32672 MAX32675 Jan 13, 2024
@Jake-Carter Jake-Carter merged commit eb5b8e5 into main Jan 13, 2024
13 checks passed
@Jake-Carter Jake-Carter deleted the fix/correction-SystemCoreClock-calculation branch January 13, 2024 08:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
MAX32662 Related to the MAX32662 (ME12) MAX32672 Related to the MAX32672 (ME21) MAX32675 Related to the MAX32675 (ME16)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants