Skip to content

u-blox rearrange ODIN target #4510

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
merged 1 commit into from
Jun 15, 2017

Conversation

kegilbert
Copy link
Contributor

Changed cherry-picking of #4089 from original PR (#4467).

No one assigned yet as reviewer, wanted to get a CI run through to see if this helped anything.


Rebase against master for: #4089

Smaller chunk of the Workshop branch additions: #4412

Feel free to add in however else should be a reviewer @c1728p9

@0xc0170
Copy link
Contributor

0xc0170 commented Jun 9, 2017

The first jenkins CI failed with

[UBLOX_EVK_ODIN_W2_armcc_wlan0] Error: L6218E: Undefined symbol SDIO_DataConfig (referred from cb_mmc.o).
01:04:03 [UBLOX_EVK_ODIN_W2_armcc_wlan0] Finished: 0 information, 0 warning and 1 error messages.
01:04:03 [UBLOX_EVK_ODIN_W2_armcc_wlan0] [ERROR] Error: L6218E: Undefined symbol SDIO_DataConfig (referred from cb_mmc.o).
01:04:03 [UBLOX_EVK_ODIN_W2_armcc_wlan0] Finished: 0 information, 0 warning and 1 error messages.
01:04:03 [UBLOX_EVK_ODIN_W2_armcc_wlan0] 

Please have a look ( I retriggered the build to confirm this).

@0xc0170
Copy link
Contributor

0xc0170 commented Jun 9, 2017

cc @andreaslarssonublox

@kegilbert
Copy link
Contributor Author

Seems to be the same error as before (#4467). Thanks for checking up on it and CCing @0xc0170!

@andreaslarssonublox
Copy link

Most likely the binaries for the ODIN driver are the old ones.
@kegilbert Could you check to see if this string "odin_driver_os_5_v2.0_rc1" is present in the binary otherwise it's probably an old binary.
Binaries:
TARGET_UBLOX_EVK_ODIN_W2\sdk\TOOLCHAIN_ARM\libublox-odin-w2-driver.ar
TARGET_UBLOX_EVK_ODIN_W2\sdk\TOOLCHAIN_GCC_ARM\libublox-odin-w2-driver.a
TARGET_UBLOX_EVK_ODIN_W2\sdk\TOOLCHAIN_IAR\libublox-odin-w2-driver.a

@kegilbert
Copy link
Contributor Author

@andreaslarssonublox TARGET_UBLOX_EVK_ODIN_W2\sdk\TOOLCHAIN_GCC_ARM\libublox-odin-w2-driver.a contained that version string, but the other two binaries did not.

@kegilbert kegilbert force-pushed the rearrange-odin-target-rebase branch from 8526666 to 6a270fb Compare June 14, 2017 18:21
@kegilbert kegilbert changed the title u-blox rearrange ODIN target - Rebase [2] u-blox rearrange ODIN target Jun 14, 2017
@kegilbert kegilbert requested review from 0xc0170, c1728p9 and theotherjimmy and removed request for 0xc0170 June 14, 2017 20:01
@kegilbert
Copy link
Contributor Author

/morph test

@mbed-bot
Copy link

Result: SUCCESS

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

/morph test

Output

mbed Build Number: 554

All builds and test passed!

@theotherjimmy
Copy link
Contributor

@kegilbert @andreaslarssonublox I just checked all three binaries, and they have the string "odin_driver_os_5_v2.0_rc1 (as part of a file string E:\build_odin_driver_os_5_v2.0_rc1\ublox-odin-w2-drivers-confidential\source\cb_wlan\common\cb_version.c). That should be fine.

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.

6 participants