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

Updated u-blox ODIN-W2 driver binaries to 2.1 rc1 #4567

Merged
merged 2 commits into from Jun 20, 2017

Conversation

Projects
None yet
5 participants
@andreaslarssonublox
Contributor

andreaslarssonublox commented Jun 15, 2017

Description

This release contains the following fixes.

The release was built and tested with the following commit in mbed-os:
35999be
(this fix was added #4444 in order to pass tests)

Status

READY

Migrations

If this PR changes any APIs or behaviors, give a short description of what API users should do when this PR is merged.

NO

Related PRs

This release depends on the following fixes:

Todos

Deploy notes

N/A

Steps to test or reproduce

N/A

Test results

mbed_os_test_results_arm.txt
mbed_os_test_results_gcc_arm.txt
mbed_os_test_results_iar.txt

@0xc0170

This comment has been minimized.

Member

0xc0170 commented Jun 16, 2017

@andreaslarssonublox CAn youo please rebasE? I believe the latest odin reorganisation in the code that came from the workshop resulted in conflicts here

@0xc0170 0xc0170 added needs: work and removed needs: CI labels Jun 16, 2017

@andreaslarssonublox andreaslarssonublox force-pushed the u-blox:ublox_odin_driver_os_5_v2.1_rc1 branch to e12efc0 Jun 16, 2017

@andreaslarssonublox

This comment has been minimized.

Contributor

andreaslarssonublox commented Jun 16, 2017

Rebased it

@0xc0170

This comment has been minimized.

Member

0xc0170 commented Jun 20, 2017

/morph test

@mbed-bot

This comment has been minimized.

mbed-bot commented Jun 20, 2017

Result: SUCCESS

Your command has finished executing! Here's what you wrote!

/morph test

Output

mbed Build Number: 592

All builds and test passed!

@0xc0170 0xc0170 added ready for merge and removed needs: CI labels Jun 20, 2017

@adbridge

This comment has been minimized.

Contributor

adbridge commented Jun 20, 2017

@andreaslarssonublox does this PR really still depend on #4571 ?

@0xc0170

This comment has been minimized.

Member

0xc0170 commented Jun 20, 2017

@andreaslarssonublox does this PR really still depend on #4571 ?

How does this depend ? I tried this patch, on top of master (does not include referenced PR), and the wifi example builds . But I assume it fails to connect to wifi, correct (havent yet run it on the board).

@andreaslarssonublox

This comment has been minimized.

Contributor

andreaslarssonublox commented Jun 20, 2017

@0xc0170 For the debug profile it will assert after connection has been setup and after sending some data if #4571 is not fixed. When using the other profiles we have not seen it crashing. See #4500 (comment) for more info.

@adbridge adbridge merged commit 525debc into ARMmbed:master Jun 20, 2017

4 checks passed

Cam-CI uvisor Build & Test Success
Details
ci/morph-test Job has completed
Details
continuous-integration/jenkins/pr-head This commit looks good
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details

@sg- sg- removed the ready for merge label Jun 20, 2017

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment