Skip to content

STM32 : Fix issue to exit deepsleep when RTC has not been initialized #5747

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 1 commit into from
Jan 8, 2018

Conversation

jeromecoutant
Copy link
Collaborator

Description

After deep sleep period, RTC clock has to be synchronized.
This patch makes it OK for all applications (even if RTC has been configured before target reset).

Status

READY

@0xc0170
Copy link
Contributor

0xc0170 commented Dec 21, 2017

@jeromecoutant How does this work? RTC handle is in the global scope here, assigning RTC to it does sync . It is assigned in Init(), that is not sufficient?

@jeromecoutant
Copy link
Collaborator Author

@0xc0170

RTC handle is in the global scope here, assigning RTC to it does sync . It is assigned in Init(), that is not sufficient?

Yes, but it is assigned at the end of this Init procedure...
Sometimes, as RTC could be already init in a previous life, this RtcHandle is not set to the right value, and rtc_synchronize becomes infinite when exiting deep sleep.

@0xc0170
Copy link
Contributor

0xc0170 commented Jan 5, 2018

/morph build

@mbed-ci
Copy link

mbed-ci commented Jan 5, 2018

Build : SUCCESS

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

Triggering tests

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

@mbed-ci
Copy link

mbed-ci commented Jan 5, 2018

@mbed-ci
Copy link

mbed-ci commented Jan 5, 2018

@0xc0170
Copy link
Contributor

0xc0170 commented Jan 8, 2018

/morph uvisor-test

@theotherjimmy theotherjimmy changed the title STM32 : issue to exit deepsleep when RTC has not been initialized STM32 : Fix issue to exit deepsleep when RTC has not been initialized Jan 8, 2018
@theotherjimmy theotherjimmy merged commit 24b0387 into ARMmbed:master Jan 8, 2018
@jeromecoutant jeromecoutant deleted the PR_RTC_SLEEP branch January 8, 2018 16:37
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