-
Notifications
You must be signed in to change notification settings - Fork 3k
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: ublox cellular api refactoring #11292
Conversation
@mudassar-ublox, thank you for your changes. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Needs unlock/unlock
fixes. ATHandler calls should be within lock/unlock, including get_last_error
and restore/set_at_timeout
.
_at->cmd_start("AT+CFUN=4"); | ||
_at->cmd_stop_read_resp(); | ||
_at->at_cmd_discard("+CFUN", "=4"); | ||
#endif | ||
if (_at->get_last_error() == NSAPI_ERROR_OK) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Need to be within lock/unlock
.
#endif | ||
|
||
return _at->unlock_return_error(); | ||
return _at->get_last_error(); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Need to be within lock/unlock
.
break; | ||
} else { | ||
_at.clear_error(); | ||
wait_ms(1000); | ||
} | ||
} | ||
t1.stop(); | ||
_at.unlock(); | ||
return _at.get_last_error(); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Need to be within lock/unlock.
@mudassar-ublox These changes are part of my PR: #11306. How do you want to proceed with this? |
Shall this be closed? |
It is fine to close it as changes are present in another pull request |
Closing as same changes present in PR #11306. |
Description
ublox cellular api refactoring. Updated methods with latest methods.
Pull request type
Reviewers
Release Notes