Skip to content
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

Update for ARMCC6 #13

Draft
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

microbit-sam
Copy link

@microbit-sam microbit-sam commented Jul 3, 2019

These changes are necessary to build the DAL in ARMCC6, which the mbed online compiler uses

Linked to:
nrf51-sdk: lancaster-university/nrf51-sdk#1
mbed-classic: #13
microbit-dal: lancaster-university/microbit-dal#449

@@ -2,7 +2,7 @@

MEMORY
{
FLASH (rx) : ORIGIN = 0x00016000, LENGTH = 0x2A000
FLASH (rx) : ORIGIN = 0x00018000, LENGTH = 0x28000

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think this was done because an overlapping soft device.
Is it possible that during testing a different soft device than yotta was used?
We need to double check that the same version of soft device is used in yotta and mbed-cli.

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@geowor01 please could you check

@microbit-carlos I've not got ARMCC6 to test this

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This was fixed upstream in ARMmbed/mbed-os@dfcb2c6.
The armcc5 linker file already contains what we think might be the right value:


We still need to figure out which linker file yotta is using to make sure the micro:bit builds with Yotta and GCC are use the same address.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants