-
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
Fix added to unlock AT handler mutex #11526
Conversation
@MarceloSalazar Will this fix be part of mbed-os release 5.14 ? Thanks |
@@ -116,6 +116,7 @@ nsapi_error_t UBLOX_AT::init() | |||
err = _at->at_cmd_discard("+CFUN", "=1"); // set full functionality | |||
} | |||
#endif | |||
_at->unlock(); |
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.
Could be return _at.unlock_return_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.
updated, thanks
6fa88a7
to
a269f71
Compare
CI started |
Test run: SUCCESSSummary: 11 of 11 test jobs passed |
Description
Fix added to unlock AT handler mutex introduced in mbed-os 5.14 release candidates
Targets verified with this fix : UBLOX_C030_U201 & UBLOX_C030_R412M
Pull request type
Reviewers
@RobMeades @MarceloSalazar @AriParkkila
Release Notes
In mbed-os 5.14 release candidate1 and 2, none of u-blox modem is working due to this problem