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

Cellular: call AT+CGEREP after sim is ready #10081

Merged
merged 1 commit into from
Mar 18, 2019

Conversation

jarvte
Copy link
Contributor

@jarvte jarvte commented Mar 13, 2019

Description

Current implementation did call AT+CGEREP before sim was ready and it was failing in most modems.
Now call is made after sim is ready.

Pull request type

[x] Fix
[ ] Refactor
[ ] Target update
[ ] Functionality change
[ ] Docs update
[ ] Test update
[ ] Breaking change

Reviewers

@AriParkkila

Release Notes

AT+CGEREP call was made before sim was ready and it was failing in most modems.
Now call is made after sim is ready.

@ciarmcom ciarmcom requested review from AriParkkila and a team March 13, 2019 18:01
@ciarmcom
Copy link
Member

@jarvte, thank you for your changes.
@AriParkkila @ARMmbed/mbed-os-wan @ARMmbed/mbed-os-maintainers please review.

@0xc0170
Copy link
Contributor

0xc0170 commented Mar 14, 2019

Needs rebase now

@jarvte
Copy link
Contributor Author

jarvte commented Mar 15, 2019

@0xc0170 rebase done

@cmonr
Copy link
Contributor

cmonr commented Mar 15, 2019

@jarvte Sorry, looks like another rebase is needed.

Current implementation did call AT+CGEREP before sim was ready
and it was failing in most modems.
@jarvte
Copy link
Contributor Author

jarvte commented Mar 18, 2019

@0xc0170 rebase done, also fix for astyle after rebase, hence the force push

@0xc0170
Copy link
Contributor

0xc0170 commented Mar 18, 2019

Ci started

@mbed-ci
Copy link

mbed-ci commented Mar 18, 2019

Test run: SUCCESS

Summary: 13 of 13 test jobs passed
Build number : 1
Build artifacts

@0xc0170 0xc0170 merged commit 6b84b14 into ARMmbed:master Mar 18, 2019
@jarvte jarvte deleted the move_call_CGEREP branch March 18, 2019 10:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants