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

[LTE] AT+CNUM command causing registration failure on LTE devices #1627

Merged
merged 1 commit into from Dec 7, 2018

Conversation

@technobly
Copy link
Member

commented Dec 7, 2018

Problem

AT+CNUM command is used in the registration process just to log the phone number of the inserted SIM. It has been observed that some SIMs in LTE devices have invalid characters in the phone number string that causes this command to error.

40.796 AT send       9 "AT+CNUM\r\n"
40.826 AT read ERR  49 "\r\n+CME ERROR: invalid characters in dial string\r\n"

This prevents the registration process from completing and the device will blink green continuously.

Solution

Remove the AT+CNUM command.

Steps to Test

Test on a unit that was previously causing this error. It should now connect without a problem.


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
  • (N/A) documentation
  • Added to CHANGELOG.md after merging (add links to docs and issues)
removes AT+CNUM command
- When the AT+CNUM command errors, it will prevent the registration process from completing.
- It is also not used for any reason other than logging the phone number of the inserted SIM.

@technobly technobly added the bug label Dec 7, 2018

@technobly technobly requested a review from sergeuz Dec 7, 2018

@sergeuz
sergeuz approved these changes Dec 7, 2018

@sergeuz sergeuz merged commit 8ce5367 into develop Dec 7, 2018

2 checks passed

continuous-integration/travis-ci/pr The Travis CI build passed
Details
continuous-integration/travis-ci/push The Travis CI build passed
Details

@sergeuz sergeuz deleted the fix/atcnum branch Dec 7, 2018

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
2 participants
You can’t perform that action at this time.