-
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
RSSI getter fixed for ESP8266 #11304
Conversation
4c91dd1
to
a074721
Compare
@AnttiKauppila, thank you for your changes. |
CI started |
Test run: FAILEDSummary: 3 of 11 test jobs failed Failed test jobs:
|
This is now part of roll up PR: #11358 Please do not change this PR, or if you do, let us know asap |
I suspect this has a failure with client test, restarting rollup and will report back |
Client passed in the rollup without this PR, I'll start CI here to get results here as well |
CI started |
Test run: FAILEDSummary: 1 of 4 test jobs failed Failed test jobs:
|
Internal error, code -11, restarting the build cc @ARMmbed/mbed-os-test |
Restarted CI |
One example got breaking change, its fixed now but need to abort + restart current job here |
Test run: FAILEDSummary: 3 of 4 test jobs failed Failed test jobs:
|
Test run: SUCCESSSummary: 11 of 11 test jobs passed |
Looks like all good ! |
I still get RSSI result as 0 Using this example: Terminal output: WiFi example Scan: Connecting to Molino Metz... MAC: 18:fe:34:a0:80:f7 Done Not sure why the Mbed-os version is showing 5.13.1 my Revision History indicates 5.14.0-1 |
@star297 Example was not updated for 5.14. Should work now |
Thank's, just found that out, does work on Revision 6689 |
Description
ESP8266 RSSI getter issue fixed: #11252
Issue was caused because for scan() a callback was added for results and rssi() adopted same mechanism but code was not updated to reflect changes.
Pull request type
Reviewers
@star297
@ARMmbed/mbed-os-wan
Release Notes