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

cpu: saml21: re-add periph/pm initialization #6335

Merged
merged 1 commit into from
Jan 13, 2017

Conversation

kaspar030
Copy link
Contributor

@kaspar030 kaspar030 commented Jan 12, 2017

This fell off when migrating from lpm to periph/pm in #6160.

@kaspar030 kaspar030 added Area: pm Area: (Low) power management Type: bug The issue reports a bug / The PR fixes a bug (including spelling errors) labels Jan 12, 2017
@haukepetersen haukepetersen added the CI: ready for build If set, CI server will compile all applications for all available boards for the labeled PR label Jan 13, 2017
Copy link
Contributor

@haukepetersen haukepetersen left a comment

Choose a reason for hiding this comment

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

ACK

@kaspar030
Copy link
Contributor Author

&go.

@kaspar030 kaspar030 merged commit 5da88f7 into RIOT-OS:master Jan 13, 2017
@kaspar030 kaspar030 deleted the fix_saml21_pm_init branch January 13, 2017 09:22
#ifdef FEATURE_PERIPH_PM
/* enable power managemet module */
MCLK->APBAMASK.reg |= MCLK_APBAMASK_PM;
PM->CTRLA.reg = PM_CTRLA_MASK & (~PM_CTRLA_IORET);
Copy link
Contributor

Choose a reason for hiding this comment

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

But why disable I/O retention?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Area: pm Area: (Low) power management CI: ready for build If set, CI server will compile all applications for all available boards for the labeled PR Type: bug The issue reports a bug / The PR fixes a bug (including spelling errors)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants