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.

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

lp_ticker test is not compatible with unified sleep implementation #3215

Closed
stevew817 opened this issue Nov 7, 2016 · 3 comments
Closed

Comments

@stevew817
Copy link
Contributor

The lp_ticker test in TESTS/mbed-hal is fundamentally incompatible with the unified sleep API in use by Silicon Labs targets.

At

there is a call to sleep(), which will turn off us_ticker on Silicon Labs platforms, since there is no object initialized that is using the high-power, high-resolution ticker. Therefore, the us_ticker_read function is not suitable for tracking time across sleep calls.

Please change this test to rely on the RTC as an always-on clock, or use us_ticker properly and set it to timeout after the lp_ticker is expected to return.

@bridadan
Copy link
Contributor

bridadan commented Nov 7, 2016

cc @bulislaw

@bulislaw
Copy link
Member

@stevew817 could you provide a patch, that uses the us_ticker properly on Silicon Labs platforms?

@ghost
Copy link

ghost commented Oct 27, 2017

GitHib issue review: Closed due to inactivity. Please re-file if critical issues found.

@ghost ghost closed this as completed Oct 27, 2017
This issue was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

5 participants