-
Notifications
You must be signed in to change notification settings - Fork 18
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
K64F linker script refactor, also fix some misc bugs #402
Conversation
..._Freescale/TARGET_MCUXpresso_MCUS/TARGET_MCU_K64F/device/TOOLCHAIN_GCC_ARM/MK64FN1M0xxx12.ld
Show resolved
Hide resolved
..._Freescale/TARGET_MCUXpresso_MCUS/TARGET_MCU_K64F/device/TOOLCHAIN_GCC_ARM/MK64FN1M0xxx12.ld
Show resolved
Hide resolved
..._Freescale/TARGET_MCUXpresso_MCUS/TARGET_MCU_K64F/device/TOOLCHAIN_GCC_ARM/MK64FN1M0xxx12.ld
Show resolved
Hide resolved
..._Freescale/TARGET_MCUXpresso_MCUS/TARGET_MCU_K64F/device/TOOLCHAIN_GCC_ARM/MK64FN1M0xxx12.ld
Show resolved
Hide resolved
This is cool, I like that.
|
Unfortunately I don't have much experience with linker script, only a marginal awareness. So I am not able to say anything to these changes :D |
If you mean like an mcuboot header, no, mcuboot prepends that to the binary file after it's been linked. If you are rolling your own format, and you just want to include a specific C struct at the start of the image, then yeah, you can do that with a linker script. |
9b6c7e7
to
2cf4f59
Compare
Summary of changes
This MR refactors the K64F linker script to support memory bank configuration. This is needed as I'm using it to test the MCUBoot bootloader.
I also fixed a couple misc bugs I ran into while working on MCUBoot.
Impact of changes
K64F can now be configured to use only a portion of flash using
target_overrides.memory_bank_config
in mbed_app.json.Migration actions required
Documentation
None
Pull request type
Test results
I reran the Mbed test suite for K64F. Results here, don't see any suspicious failures.