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

STM32F413ZH: Add bootloader support #8553

Merged
merged 3 commits into from Oct 27, 2018

Conversation

Projects
None yet
7 participants
@bcostm
Contributor

bcostm commented Oct 26, 2018

Description

Add bootloader support for DISCO_F413ZH and NUCLEO_F413ZH platforms.

Fixes #8543

Pull request type

[X] Fix
[ ] Refactor
[ ] Target update
[ ] Functionality change
[ ] Docs update
[ ] Test update
[ ] Breaking change

@0xc0170 0xc0170 added the needs: CI label Oct 26, 2018

@LMESTM

LMESTM approved these changes Oct 26, 2018

@0xc0170

This comment has been minimized.

Member

0xc0170 commented Oct 26, 2018

/morph build


ER_IROM1 0x08000000 0x150000 { ; load address = execution address
; 1536KB FLASH (0x150000)

This comment has been minimized.

@jeromecoutant

jeromecoutant Oct 26, 2018

Contributor

1536K = 0x180000 ...

This comment has been minimized.

@bcostm

bcostm Oct 26, 2018

Contributor

👍 apparently it never bothered anybody ....

This comment has been minimized.

@0xc0170

0xc0170 Oct 26, 2018

Member

Good catch 🎉

@mbed-ci

This comment has been minimized.

mbed-ci commented Oct 26, 2018

Build : SUCCESS

Build number : 3474
Build artifacts/logs : http://mbed-os.s3-website-eu-west-1.amazonaws.com/?prefix=builds/8553/

Triggering tests

/morph test
/morph export-build
/morph mbed2-build

@jeromecoutant

OK with quick ST CI tests:

target platform_name test suite result elapsed_time (sec) copy_method
DISCO_F413ZH-ARM DISCO_F413ZH tests-mbed_drivers-echo OK 17.94 default
DISCO_F413ZH-GCC_ARM DISCO_F413ZH tests-mbed_drivers-echo OK 18.32 default
DISCO_F413ZH-IAR DISCO_F413ZH tests-mbed_drivers-echo OK 17.33 default
NUCLEO_F413ZH-ARM NUCLEO_F413ZH tests-mbed_drivers-echo OK 17.33 default
NUCLEO_F413ZH-GCC_ARM NUCLEO_F413ZH tests-mbed_drivers-echo OK 18.17 default
NUCLEO_F413ZH-IAR NUCLEO_F413ZH tests-mbed_drivers-echo OK 17.61 default
@0xc0170

This comment has been minimized.

Member

0xc0170 commented Oct 26, 2018

Aborted current CI jobs, restarting

/morph build

@mbed-ci

This comment has been minimized.

@mbed-ci

This comment has been minimized.

mbed-ci commented Oct 26, 2018

Build : SUCCESS

Build number : 3480
Build artifacts/logs : http://mbed-os.s3-website-eu-west-1.amazonaws.com/?prefix=builds/8553/

Triggering tests

/morph test
/morph export-build
/morph mbed2-build

@mbed-ci

This comment has been minimized.

@mbed-ci

This comment has been minimized.

@cmonr cmonr merged commit 01f864e into ARMmbed:master Oct 27, 2018

15 checks passed

ci-morph-build build completed
Details
ci-morph-exporter build completed
Details
ci-morph-mbed2-build build completed
Details
ci-morph-test test completed , RTOS ROM(+0 bytes) RAM(+0 bytes)
Details
continuous-integration/jenkins/pr-head This commit looks good
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
jenkins-ci/cloud-client-test Success
Details
jenkins-ci/unittests Success
Details
travis-ci/astyle Passed, 536 files (+0 files)
Details
travis-ci/docs Local docs testing has passed
Details
travis-ci/events Passed, runtime is 9751 cycles (+568 cycles)
Details
travis-ci/gitattributestest Local gitattributestest testing has passed
Details
travis-ci/licence_check Local licence_check testing has passed
Details
travis-ci/littlefs Passed, code size is 8372B (+0.00%)
Details
travis-ci/tools-py2.7 Local tools-py2.7 testing has passed
Details

@cmonr cmonr removed the needs: CI label Oct 27, 2018

@bcostm bcostm deleted the bcostm:F413ZH_bootloader branch Oct 29, 2018

@adbridge

This comment has been minimized.

Contributor

adbridge commented Nov 2, 2018

This one won't automatically merge.

@cmonr

This comment has been minimized.

Contributor

cmonr commented Nov 5, 2018

Aosu for future reference, auto-merges with --keep-cr flag

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