Skip to content

Add system_reset call #6230

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

Merged
merged 3 commits into from
Mar 1, 2018
Merged

Add system_reset call #6230

merged 3 commits into from
Mar 1, 2018

Conversation

bulislaw
Copy link
Member

Description

  • Add official Mbed API for reseting the system using CMSIS call.
  • Add tests

Pull request type

  • Feature

* @return true if a target can go to deepsleep, false otherwise
*/
bool sleep_manager_can_deep_sleep(void);
#warning mbed_sleep.h has been replaced by mbed_power_mgmt.h, please update to mbed_power_mgmt.h [since mbed-os-5.9]
Copy link
Contributor

Choose a reason for hiding this comment

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

Just wanted to mention that I like how early this PR is 👍

platform/sleep.h Outdated
@@ -18,7 +18,7 @@
#ifndef MBED_OLD_SLEEP_H
#define MBED_OLD_SLEEP_H

#warning sleep.h has been replaced by mbed_sleep.h, please update to mbed_sleep.h [since mbed-os-5.3]
#include "platform/mbed_sleep.h"
#warning sleep.h has been replaced by mbed_power_mgmt.h, please update to mbed_power_mgmt.h [since mbed-os-5.3]
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 suppose to say since mbed-os-5.9? Or is this because sleep came before mbed_sleep?

Copy link
Member Author

Choose a reason for hiding this comment

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

Good catch!

@bulislaw bulislaw force-pushed the system_reset branch 3 times, most recently from 59cdbd9 to 5782a0f Compare February 28, 2018 13:24
@bulislaw
Copy link
Member Author

@cmonr I've updated the warnings to 5.8 as the PR is marked for 5.8-rc1

@adbridge
Copy link
Contributor

@bulislaw looks like this is also going to need a rebase. You may need to chase @SenRamakri for the review as @0xc0170 is out the rest of this week. Is there anyone else that could review? It is starting to get very close to not making it into 5.8....

SenRamakri
SenRamakri previously approved these changes Feb 28, 2018
@cmonr
Copy link
Contributor

cmonr commented Feb 28, 2018

@bulislaw Looks like there's still one lingering travis issue. Once it's all green, we can start the CI build.

@bulislaw
Copy link
Member Author

@cmonr @SenRamakri I've removed conflicting (and identical) system_reset implementation from ATMEL.

@cmonr
Copy link
Contributor

cmonr commented Feb 28, 2018

@bulislaw I like that the removed code was completely identical.

/morph build

@mbed-ci
Copy link

mbed-ci commented Feb 28, 2018

Build : SUCCESS

Build number : 1307
Build artifacts/logs : http://mbed-os.s3-website-eu-west-1.amazonaws.com/?prefix=builds/6230/

Triggering tests

/morph test
/morph uvisor-test
/morph export-build
/morph mbed2-build

@mbed-ci
Copy link

mbed-ci commented Mar 1, 2018

@mbed-ci
Copy link

mbed-ci commented Mar 1, 2018

@adbridge
Copy link
Contributor

adbridge commented Mar 1, 2018

@SenRamakri Please re-review and confirm you are happy with the changes

@cmonr cmonr merged commit 9ddb092 into ARMmbed:master Mar 1, 2018
@ghost ghost mentioned this pull request Mar 1, 2018
2 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants