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.

Sign up for GitHub

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

pm: misc cleanup and fixes #6344

Merged
merged 8 commits into from
Jan 24, 2017
Merged

pm: misc cleanup and fixes #6344

Changes from 1 commit
Commits
Show all changes
8 commits
Select commit Hold shift + click to select a range
efa0aa1
cpu/cc430|chronos: moved periph_common dep to CPU
haukepetersen Jan 13, 2017
cd53fe9
cpu/arm7: moved pm_reboot to periph/pm.c
haukepetersen Jan 13, 2017
409516c
cpu/cortexm: moved pm.c to periph/pm.c
haukepetersen Jan 13, 2017
801eece
cpu/cortexm: removed unused pm include in panic.c
haukepetersen Jan 13, 2017
61045d1
cpu/lpc1768: unuse module periph_common
haukepetersen Jan 13, 2017
a37ba98
cpu/msp430-common: moved pm_reboot to periph/pm.c
haukepetersen Jan 13, 2017
55c4477
periph/pm: minor doc clarification
haukepetersen Jan 13, 2017
0fc8c66
sys/pm_layered: fixed doxygen group
haukepetersen Jan 13, 2017
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
periph/pm: minor doc clarification
  • Loading branch information
@haukepetersen
haukepetersen committed Jan 19, 2017
commit 55c4477038ac6fc3fcc23645a96f10ee8d5aca7d
5 changes: 1 addition & 4 deletions drivers/include/periph/pm.h
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -12,10 +12,7 @@
* @brief The kernels power management interface
* @{
*
* The following functions *must* be available for every platform:
*
* pm_reboot()
* pm_off()
* This interface *must* be implemented for every platform in RIOT.
Copy link
Member

Choose a reason for hiding this comment

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

Is this already the case?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Yes, at least rudimentary.

Copy link
Member

Choose a reason for hiding this comment

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

I was wondering about pm_off() but before, I haven't seen there is fall-back code.

*
* @file
* @brief Power management interface
Expand Down