Add CMake support #5
Open
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
As per the latest guidelines on Mbed OS standard pin names, all buttons are numbered, starting at `BUTTON1`. `USER_BUTTON` is not available on DISCO_L475VG_IOT01A anymore.
The majority of flash devices support reading single bytes without alignment concerns. The config `mcuboot.read-granularity` enables additional code to handle read alignment, and it comes with a slight code size and performance overhead.
Mbed OS builds .bin images by default for most targets, but many targets (e.g. ST targets running the latest ST-LINK firmware) also support .hex files, making this demo much easier to prepare. In case a target doesn't support .hex files, README.md already contains steps for merging .hex files into a single one and converting it to a .bin file.
A missing include was added into main.cpp to fix the build. Co-authored-by: Lingkai Dong <[email protected]>
Co-authored-by: Lingkai Dong <[email protected]>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
As mentioned in mcu-tools/mcuboot#1153,
"Currently, MbedOS applications using this port require that they are built with the old mbed-cli. However,
mbed-toolsis the new, future command-line tool for MbedOS;mbed-toolsis also known as Mbed CLI 2. This new tool makes use of "Ninja as a build system, and CMake to generate the build environment and manage the build process in a compiler-independent manner." More information can be found here."A CMakeLists.txt was added as a result; the
mcuboot.libnow points to this fork. This pull request has a companion pull request in thembed-mcuboot-demoapplication repository; both are required for the full mbed-tools build workflow.Note: This pull requests builds on the changes from #4.
@LDong-Arm