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

[boron] forward compatibility with 460800 baudrate on SARA R4 #2104

Merged
merged 1 commit into from
May 18, 2020

Conversation

avtolstoy
Copy link
Member

Problem

We want to increase the baudrate on SARA R4-based devices to improve OTA times, however compared to all the other modems, AT+IPR setting on SARA R4 is persistent (!).

We need to have an intermediate release in 1.5.x line supporting both baudrates and changing the persistent value to 115200 in order to support downgrades from 2.x.

Solution

Try to talk to the modem on both 460800 and 115200 and change persistent value to 115200.

Steps to Test

  1. Force LTE Boron/BSoM modem to use 460800 (AT+IPR=460800), power down device completely, flash system-part out of this branch
  2. The device should connect to the cloud
  3. Downgrade to 1.5.1, the device should connect to the cloud

Example App

N/A

References

  • [CH54763]

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)

@avtolstoy avtolstoy added this to the 1.5.2 milestone May 15, 2020
@avtolstoy avtolstoy requested a review from keeramis May 15, 2020 14:05
@avtolstoy avtolstoy mentioned this pull request May 15, 2020
6 tasks
Copy link
Contributor

@keeramis keeramis left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Tested on

  • OK Older BoronLTE cell version 02.00 --> picks up 115200
  • OK Newer BoronLTE cell version 02.04 --> picks up 115200 when 115200 configured on cell radio
  • OK Newer BoronLTE cell version 02.04 --> tests with 115200 and when it fails, moves to 460800 and gets to work

@avtolstoy avtolstoy added ready to merge PR has been reviewed and tested and removed needs review labels May 18, 2020
@avtolstoy avtolstoy merged commit 55c291b into release/v1.5.x May 18, 2020
@avtolstoy avtolstoy deleted the feature/1.5.x-forward-compat-r4-boron branch May 18, 2020 18:22
@avtolstoy avtolstoy mentioned this pull request May 18, 2020
6 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement ready to merge PR has been reviewed and tested
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants