Skip to content

STM32 LPTICKER update for targets supporting RTC #6739

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

Conversation

jeromecoutant
Copy link
Collaborator

Description

STM32 LPTICKER update for targets supporting RTC

This completes #6474
And should be the last PR from ST in the ticker feature branch

Targets:

  • STM32L1, STM32F0, STM32F3, most of STM32F4
  • STM32L0, STM32L4, STM32F7 and few STM32F4 if user decides to disable lpticker_lptim config

@bulislaw @c1728p9 @screamerbg

Pull request type

[ ] Fix
[X] Refactor
[ ] New target
[ ] Feature
[ ] Breaking change

@0xc0170
Copy link
Contributor

0xc0170 commented Apr 25, 2018

/morph build

@mbed-ci
Copy link

mbed-ci commented Apr 25, 2018

Build : FAILURE

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

@jeromecoutant
Copy link
Collaborator Author

Please, start again morph build
Sorry

@0xc0170
Copy link
Contributor

0xc0170 commented Apr 25, 2018

/morph build

@mbed-ci
Copy link

mbed-ci commented Apr 25, 2018

Build : SUCCESS

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

Triggering tests

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

@marcemmers
Copy link
Contributor

Looks like a big improvement over the previous versions.

@jeromecoutant Would it still be an idea to implement a way to set the rtc time without disturbing the rtc_read_lp() function like we discussed earlier?

@jeromecoutant
Copy link
Collaborator Author

Yes, updating the rtc write time will follow soon,
but I don't have any test yet to check this, and it is not "mandatory" to make LP ticker working

@marcemmers
Copy link
Contributor

Ah ok, never mind then ;)

@mbed-ci
Copy link

mbed-ci commented Apr 25, 2018

@mbed-ci
Copy link

mbed-ci commented Apr 25, 2018

@0xc0170 0xc0170 merged commit 26164e2 into ARMmbed:feature-hal-spec-ticker Apr 26, 2018
@jeromecoutant jeromecoutant deleted the PR_LP_NEW_API_RTC branch April 30, 2018 14:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants