Skip to content

enable iar exporter for nRF52840 #4303

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
May 30, 2017
Merged

Conversation

nvlsianpu
Copy link
Contributor

Description

Enable IAR exporter for nRF52840-xxAA devices

Status

READY

@0xc0170
Copy link
Contributor

0xc0170 commented May 11, 2017

/morph export-build

@pan-
Copy link
Member

pan- commented May 11, 2017

@nvlsianpu Is debugging supposed to work on IAR with nRF52840-xxAA devices ?

@mbed-bot
Copy link

Result: FAILURE

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

/morph export-build

Output

mbed Build Number: 30

Exporter Build failed!

@theotherjimmy theotherjimmy changed the title nRF52840: enable iar exporter enable iar exporter for nRF52840 May 11, 2017
@nvlsianpu
Copy link
Contributor Author

It is supposed to work, as it is similar to nRF52832.

@0xc0170
Copy link
Contributor

0xc0170 commented May 12, 2017

@nvlsianpu The CI failed: 17:26:50 C:\mj\workspace\bm_wrap\206\examples\mbed-os-example-blinky\mbed-os\cmsis\core_cm4.h(1600) : Error[Ta097]: Cannot call intrinsic function "__nounwind __DSB" from Thumb mode in this architecture This is for IAR, and this new platform.

@0xc0170
Copy link
Contributor

0xc0170 commented May 15, 2017

Can you have a look at the failure?

@nvlsianpu
Copy link
Contributor Author

I will do this.

@nvlsianpu
Copy link
Contributor Author

nvlsianpu commented May 16, 2017

@0xc0170 Please check whether you installed proper MDK for IAR (8.11.1 or newer). Link for install setups: http://www.nordicsemi.com/eng/nordic/Products/nRF52840/nRF5x-MDK-IAR/58863

@0xc0170
Copy link
Contributor

0xc0170 commented May 16, 2017

@0xc0170 Please check whether you installed proper MDK for IAR (8.11.1 or newer). Link for install setups: http://www.nordicsemi.com/eng/nordic/Products/nRF52840/nRF5x-MDK-IAR/58863

That would explain it. Is this captured anywhere what version is required ?

@bridadan Please can you check if that this MDK is installed in the CI?

@bridadan
Copy link
Contributor

@0xc0170 @nvlsianpu We are still currently using IAR v7.80. Does the NRF52840 support that version of IAR?

@nvlsianpu
Copy link
Contributor Author

@nvlsianpu
Copy link
Contributor Author

hint https://developer.mbed.org/teams/Nordic-Semiconductor/wiki/Nordic-platforms-readme#off-line-ide-nrf5-support was added.

@nvlsianpu
Copy link
Contributor Author

@bridadan

We are still currently using IAR v7.80. Does the NRF52840 support that version of IAR?

Yes. Even older one would work. Only nRF5x-MDK-IAR is required.

@0xc0170
Copy link
Contributor

0xc0170 commented May 17, 2017

@nvlsianpu CAn you please capture it in the commit message what is required to have (>thisversion at least must be installed) ?

@bridadan Please restart CI once it is installed in the CI, thank you

@bridadan
Copy link
Contributor

@nvlsianpu Thanks for the tips! All the machines should have this installed now.

/morph export-build

@mbed-bot
Copy link

Result: FAILURE

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

/morph export-build

Output

mbed Build Number: 34

Exporter Build failed!

@0xc0170
Copy link
Contributor

0xc0170 commented May 18, 2017

@bulislaw Can you have a look at the report above? It pinpoints to core_cm4 header file (the virtual nvic inclusion that was added recently) ?

@sg- sg- added needs: CI and removed needs: work labels May 18, 2017
@bulislaw
Copy link
Member

The virtual header file points to cmsis_nvic.h and it only influences NVIC functions. Question: Was it ever working? If it was maybe it's an easy bisect.

@0xc0170
Copy link
Contributor

0xc0170 commented May 26, 2017

/morph export-build

@mbed-bot
Copy link

Result: SUCCESS

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

/morph export-build

Output

mbed Build Number: 45

All exports and builds passed!

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.

9 participants