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): Implement low power modes for MAX32657 #1317

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

Conversation

ttmut
Copy link
Contributor

@ttmut ttmut commented Jan 3, 2025

Description

This PR updates MAX32657 low power API as listed below.

  1. Implement standby mode.
  2. Implement SRAM retention.
  3. Remove unsupported API functions.
  4. Update wrap_max32_lp.h to map wrapper functions to MAX32657 low power modes.

ttmut added 2 commits January 3, 2025 13:33
Added standby mode and SRAM retention. Unavailable features are removed.

Signed-off-by: Tahsin Mutlugun <[email protected]>
Update wrap_max32_lp.h to accomodate the low power modes of MAX32657.

Signed-off-by: Tahsin Mutlugun <[email protected]>
@github-actions github-actions bot added MAX32657 Related to the MAX32655 (ME30) Zephyr MSDK Zephyr related change. labels Jan 3, 2025
Copy link
Contributor

@sihyung-maxim sihyung-maxim left a comment

Choose a reason for hiding this comment

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

Only semantics.

@ttmut
Copy link
Contributor Author

ttmut commented Jan 8, 2025

Only semantics.

Thanks for the feedback. Could you be more specific?

Copy link
Contributor

@sihyung-maxim sihyung-maxim left a comment

Choose a reason for hiding this comment

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

Sorry! Guess I didn't submit my comments with the review. Here is what I meant.

Comment on lines +91 to +106
/**
* @brief Enable retention regulator
*/
void MXC_LP_RetentionRegEnable(void);

/**
* @brief Disable retention regulator
*/
void MXC_LP_RetentionRegDisable(void);

/**
* @brief Is the retention regulator enabled
*
* @return 1 = enabled 0 = disabled
*/
int MXC_LP_RetentionRegIsEnabled(void);
Copy link
Contributor

Choose a reason for hiding this comment

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

Put action verb at the beginning of the function name for tone to be consistent in the rest of the file.

/**
 * @brief      Enable retention regulator
 */
void MXC_LP_EnableRetentionReg(void);

/**
 * @brief      Disable retention regulator
 */
void MXC_LP_DisableRetentionReg(void);

Copy link
Contributor

Choose a reason for hiding this comment

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

MXC_LP_RetentionRegIsEnabled is fine. Didn't mean to include that function in the review window.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
MAX32657 Related to the MAX32655 (ME30) Zephyr MSDK Zephyr related change.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants