-
Notifications
You must be signed in to change notification settings - Fork 3k
Sleep manager implements all the API's in platform file #9280
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
Conversation
@deepikabhavnani, thank you for your changes. |
Do you need to update the includes after a move like this? In other words:
should become:
|
Thanks, updated headers 👍 |
9115482
to
3e9f7bb
Compare
@SenRamakri / @kegilbert - Please review |
3e9f7bb
to
91d27c1
Compare
Rebased on top of master to pull in travis fixes |
91d27c1
to
639bc6b
Compare
Rebased to resolve conflicts |
639bc6b
to
1572cfc
Compare
This should be ready for CI, I'll label it as such, but please can you rebase again to get latest travis fix (once it is green we start CI) |
….changes Not sure why mbed_sleep_manager.c was in hal, moving it to platform
File was moved from hal -> platform folder, hence includes should be changed accordingly.
1572cfc
to
a1a9051
Compare
Rebase done 👍 |
CI started |
Test run: SUCCESSSummary: 11 of 11 test jobs passed |
Description
Sleep manager implements all the API's in platform/mbed_sleep_manager.h. Not sure why mbed_sleep_manager.c was in hal, moving it to platform.
Pull request type
Reviewers