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

Fix NUCLEO_L476RG linker scripts #4554

Merged
merged 2 commits into from Jun 19, 2017

Conversation

Projects
None yet
5 participants
@LMESTM
Contributor

LMESTM commented Jun 14, 2017

Description

Fix to #4553

Following Merge pull request #4063 from LMESTM/17q2_L4_bootloader
the NUCLEO_L476RG binairies could not boot anymore.

The change done in #4063 was derived from work on NUCLEO_L429ZI target
which supports uvisor. The VECTORS defintiion is introduced as part of
uvisor support and requires further changes in ld file which were missing.
As uvisor is not considered yet, we remove VECTORS for now and will
introduce only when needed.

While fixing this issue, I also found out that FLASH size was wrongly defined
and fixed this in 2nd commit

Status

READY

Related PRs

  • #4543 also needed => MERGED

Todos

  • Tests
    Tested with basic OS2 and OS5 localy

LMESTM added some commits Jun 14, 2017

NUCLEO_L476RG: GCC_ARM ld file fix
Following
Merge pull request #4063 from LMESTM/17q2_L4_bootloader
the NUCLEO_L476RG binairies could not boot anymore.

The change done in #4063 was derived from work on NUCLEO_L429ZI target
which supports uvisor. The VECTORS defintiion is introduced as part of
uvisor support and requires further changes in ld file which were missing.
As uvisor is not considered yet, we remove VECTORS for now and will
introduce only when needed.
NUCLEO_L476RG: FLASH size of 1MB, not 2MB
MBED_APP_SIZE was erroneously defined to 2MB for this target,
while it's only 1MB.

@LMESTM LMESTM changed the title from Fix NUCLEO_L476RG linker scripts to Fix NUCLEO_L476RG GCC_ARM linker scripts Jun 15, 2017

@LMESTM LMESTM changed the title from Fix NUCLEO_L476RG GCC_ARM linker scripts to Fix NUCLEO_L476RG linker scripts Jun 15, 2017

@LMESTM

This comment has been minimized.

Contributor

LMESTM commented Jun 15, 2017

@0xc0170 - feedback or tests welcome

@LMESTM

This comment has been minimized.

Contributor

LMESTM commented Jun 15, 2017

@0xc0170 @adbridge I' like this fix to be considered as well in priority before next release if #4063 is also part of the release

@sg-

This comment has been minimized.

Member

sg- commented Jun 15, 2017

/morph test

@mbed-bot

This comment has been minimized.

mbed-bot commented Jun 16, 2017

Result: SUCCESS

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

/morph test

Output

mbed Build Number: 562

All builds and test passed!

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

@theotherjimmy theotherjimmy merged commit 3f46421 into ARMmbed:master Jun 19, 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
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment