Skip to content

u-blox ODIN-W2 drivers update #3323

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

Merged

Conversation

andreaslarssonublox
Copy link

Description

This is an update of the u-blox ODIN-W2 drivers:

  • Fixed stability problems with Wi-Fi
  • Improved SDIO speed for Wi-Fi

Known limitations

  • Use of mbed TLS on this platform is not secure and not production ready. This will be fixed in an upcoming release.
  • Only possible to use either Wi-Fi or Ethernet as MAC-interface. This is a limitation in mbed-os currently.
  • WEP is not implemented yet.
  • The OdinWiFiInterface class can only be instantiated once.

Status

READY

Migrations

NO

Related PRs

Todos

Deploy notes

Test results for mbed-os tests

Some additional Wi-Fi tests have also been included in the mbed-os tests.
Two of five ethernet tests are failing because we cannot connect the board to the internal network.

test_report_mbed_os_arm_eth.txt
test_report_mbed_os_arm_wifi.txt
test_report_mbed_os_gcc_eth.txt
test_report_mbed_os_gcc_wifi.txt
test_report_mbed_os_iar_eth.txt
test_report_mbed_os_iar_wifi.txt

Other

The binaries were built and tested using the following:
mbed-os:
SHA-1: 9abb7c3
u-blox ODIN-W2 drivers:
SHA-1: dfb0da8ebe77fe76640cb03a89b1fdd7b6c1776a

@andreaslarssonublox
Copy link
Author

andreaslarssonublox commented Nov 24, 2016

@0xc0170 @bulislaw

@andreaslarssonublox andreaslarssonublox changed the title Ublox odin w2 drivers update u-blox odin w2 drivers update Nov 24, 2016
@andreaslarssonublox andreaslarssonublox changed the title u-blox odin w2 drivers update u-blox ODIN-W2 drivers update Nov 24, 2016
@adbridge
Copy link
Contributor

/morph test_nightly

@bulislaw
Copy link
Member

LGTM

@0xc0170
Copy link
Contributor

0xc0170 commented Nov 24, 2016

+1

@0xc0170
Copy link
Contributor

0xc0170 commented Nov 24, 2016

@bulislaw @geky Any tests to be run?

@jankii01 Can we run this update with client ?

@0xc0170
Copy link
Contributor

0xc0170 commented Nov 24, 2016

@andreaslarssonublox have you run client app with this patch?

@bulislaw
Copy link
Member

@0xc0170 could we merge #3147 before 5.3? They won't run on CI as we don't have any Odin boards but would be good to have them in so people can actually test their code.

@0xc0170
Copy link
Contributor

0xc0170 commented Nov 24, 2016

@0xc0170 could we merge #3147 before 5.3? They won't run on CI as we don't have any Odin boards but would be good to have them in so people can actually test their code.

Thanks for the reminder, done.

@andreaslarssonublox Can you run those tests please and share results?

@andreaslarssonublox
Copy link
Author

andreaslarssonublox commented Nov 24, 2016

Added test report for the new wifi tests for GCC, ARM and IAR. Looks OK.
test_report_mbed_os_gcc_wifi_2.txt
test_report_mbed_os_arm_wifi_2.txt
test_report_mbed_os_iar_wifi_2.txt

I have not tested with the client example.

@adbridge
Copy link
Contributor

@andreaslarssonublox Are you at all able to run client tests with this today?

@andreaslarssonublox
Copy link
Author

@adbridge @0xc0170
I did some tests with mbed-os-example-client with the main.cpp from this PR:
https://github.com/ARMmbed/oob-mbed-os-example-client/pull/18
https://github.com/bulislaw/oob-mbed-os-example-client-1/tree/wifi_enc_fix

I merged in latest mbed-os.

The following tests were verified by opening the https://connector.mbed.com page and see that the device was connected.

Results ethernet(EMAC removed from target.json):
GCC - OK
ARM - OK
IAR - OK

Results Wi-Fi:
GCC - OK
ARM - OK
IAR - Not OK - does not register to device server

I tried with an older version of the IAR binary for the ODIN-W2 drivers(5.2.2) but this has the same faulty behavior. So I guess we can not merge in the ODIN board support PR 18 yet https://github.com/ARMmbed/oob-mbed-os-example-client/pull/18.

@adbridge
Copy link
Contributor

Thanks @andreaslarssonublox
There don't appear to be any regressions and the test results look good.

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.

4 participants