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

feat(PeriphDrivers): Add OTP Powerdown feature for MAX32572 #952

Open
wants to merge 5 commits into
base: main
Choose a base branch
from

Conversation

sihyung-maxim
Copy link
Contributor

@sihyung-maxim sihyung-maxim commented Mar 14, 2024

Description

To disable the OTP for low power applications, there needs to be a 20HCLK cycle delay after the OTP controller powers down (OTP_CLKDIV.pd) and before the OTP peripheral clock is disabled.

Needs to be tested for ME55A2 LP measurements.

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.

@sihyung-maxim sihyung-maxim added WIP work in progress Needs Testing labels Mar 14, 2024
@github-actions github-actions bot added the MAX32572 Related to the MAX32572 (ME55) label Mar 14, 2024
@sihyung-maxim
Copy link
Contributor Author

/clang-format-run

@sihyung-maxim sihyung-maxim changed the title feat(PeriphDrivers): Add OTP Powerdown feature for LP feat(PeriphDrivers): Add OTP Powerdown feature for MAX32572 Mar 18, 2024
// Must wait 20HCLK cycles after powering down OTP and before
// disabling the OTP clock, so the OTP_STATUS.pwr_rdy
// is properly updated.
MXC_Delay(10);
Copy link
Contributor

Choose a reason for hiding this comment

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

Is this delay necessary if we poll the RDY bit before disabling the clock?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
MAX32572 Related to the MAX32572 (ME55) Needs Testing WIP work in progress
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants