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(CMSIS,PeriphDrivers): Enable external clock selection on MAX32657 #1297

Merged
merged 2 commits into from
Dec 16, 2024

Conversation

ttmut
Copy link
Contributor

@ttmut ttmut commented Dec 13, 2024

Description

Uncomment clock selection code to allow external clock input to be used as system clock since external clock pin and system clock selection bits are now known.

Set INRO clock frequency to 131kHz.

@ttmut ttmut requested a review from sihyung-maxim December 13, 2024 18:22
@github-actions github-actions bot added the MAX32657 Related to the MAX32655 (ME30) label Dec 13, 2024
Now that external clock selection bits are defined, uncomment code to
allow external clock to be used.

Signed-off-by: Tahsin Mutlugun <[email protected]>
Update the code to use the correct clock rate of 131kHz for internal
nano-ring oscillator.

Signed-off-by: Tahsin Mutlugun <[email protected]>
@@ -38,7 +38,7 @@ extern "C" {
Update if use of this oscillator requires precise timing.*/
/* NOTE: INRO was previously named NANORING */
#ifndef INRO_FREQ
#define INRO_FREQ 100000
#define INRO_FREQ 131000
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 the nominal value that's indicated in the datasheet? The INRO frequency is highly variable which could be 131KHz on one part or 80KHz on another.

Copy link
Contributor Author

@ttmut ttmut Dec 16, 2024

Choose a reason for hiding this comment

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

Yes, it is given as 131kHz in both the user guide and the datasheet for MAX32657, if that is what you mean.

@sihyung-maxim sihyung-maxim merged commit d9efd38 into main Dec 16, 2024
11 checks passed
@sihyung-maxim sihyung-maxim deleted the fix/me30_clock branch December 16, 2024 20:27
sihyung-maxim pushed a commit to analogdevicesinc/hal_adi that referenced this pull request Dec 16, 2024
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)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants