-
Notifications
You must be signed in to change notification settings - Fork 1.4k
doc: Add sysbuild migration guide #15845
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
Merged
Merged
Changes from all commits
Commits
Show all changes
18 commits
Select commit
Hold shift + click to select a range
9913339
doc: nrf: migration: Add sysbuild migration guide
nordicjm 2289dd6
doc: nrf: build: Add sysbuild image guide
nordicjm c290e2b
doc: nrf: build: Add zephyr samples sysbuild page
nordicjm 7042872
doc: nrf: build: Deprecate multi image
nordicjm 7f57f73
doc: nrf: build: bootloaders: Add sysbuild documentation
nordicjm 799a5e1
doc: nrf: guides: nrf70: Update for sysbuild
nordicjm 65f2623
snippets: nrf70-fw-patch-ext-flash: Fix alignment
nordicjm aa2a1d8
doc: nrf: libraries: dfu: Improve multi image documentation
nordicjm aa6571a
doc: nrf: guides: Update guides for sysbuild
nordicjm b224fb8
doc: nrf: protocols: bt: bt_mesh: Update guides for sysbuild
nordicjm 733c7ad
doc: nrf: zigbee: fota: Update Kconfig for sysbuild
nordicjm 238590f
doc: nrf: bluetooth_services: fast_pair: Update Kconfig for sysbuild
nordicjm 83fae2e
doc: nrf: config: bootloaders: downgrade: Update Kconfig for sysbuild
nordicjm 39b43ab
doc: nrf: build: app: output_build_files Update for sysbuild
nordicjm 99d6419
doc: nrf: config: bootloader: downgrade: Fix build number comment
nordicjm 47ef7ee
doc: nrf: config: Add sysbuild forced options page
nordicjm 6853c9d
sysbuild: Rearrange options
nordicjm 900b252
doc: nrf: build: bootloaders: Add firmware loader documentation
nordicjm File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or 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
449 changes: 449 additions & 0 deletions
449
doc/nrf/config_and_build/bootloaders/bootloader_adding_sysbuild.rst
Large diffs are not rendered by default.
Oops, something went wrong.
This file contains hidden or 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
This file contains hidden or 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
This file contains hidden or 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
Oops, something went wrong.
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.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It seems like for nrf53 this is only sensible mode for mcuboot, as default will only apply for app core and not net core
and result in version mismatch - application could roll back to previous working version, but netcore will be ignored (am I right?) maybe we should also add note here about this here
Uh oh!
There was an error while loading. Please reload this page.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
However sometimes you want to have this enabled for other reasons.
It is correct in NCS context this is the only use-case where this is required is when having multi-image updates to ensure that you don't get version mismatch.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
OK, so I think that this config may be a little hard to understand on its own for nrf53 devices (especially that in some cases this may work as expected, if net core is still compatible with previous version it may be unnoticed for a long time and result in hard to debug issues), and IMHO it would be nice to see some clarification either here, or maybe there is better place for such note. I just feel like this information is an inside knowledge, and it could be misunderstood or unnoticed by the users of nrf53.