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

[R510] AT command timeouts after warm boot #2373

Merged
merged 1 commit into from
Nov 5, 2021

Conversation

keeramis
Copy link
Contributor

@keeramis keeramis commented Nov 3, 2021

Problem

Some of R510 warm boots take longer than expected and failing SLO criteria. In one of the runs with debugging enabled, it is noticed a run that took 115sec during warm boot (expected <30s) was blocked at ATI9 command for 90sec (which is default AT command timeout) and was causing the additional delays. It's next command though, works.

Solution

Add additional AT-OK checks at init() which helps if there was an internal race condition that could block the AT interface.

Steps to Test

Example App

Apply warm resets on any test app where cellular is initialized

References

SC90828


Completeness

  • User is totes amazing for contributing!
  • Contributor has signed CLA (Info here)
  • Problem and Solution clearly stated
  • Run unit/integration/application tests on device
  • Added documentation
  • Added to CHANGELOG.md after merging (add links to docs and issues)

@keeramis keeramis added this to the 3.2.0 milestone Nov 3, 2021
@avtolstoy avtolstoy force-pushed the fix/sc90828_r510_additional_at_chks branch from 6553a62 to 6d5bd8e Compare November 5, 2021 23:32
@avtolstoy avtolstoy merged commit 5aaa8de into develop Nov 5, 2021
@avtolstoy avtolstoy deleted the fix/sc90828_r510_additional_at_chks branch November 5, 2021 23: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
2 participants