Skip to content
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

[nrf noup] treewide: Add support for sysbuild assigned images #338

Draft
wants to merge 4 commits into
base: main
Choose a base branch
from

Conversation

nordicjm
Copy link
Contributor

@nordicjm nordicjm commented Oct 1, 2024

No description provided.

Adds support for image IDs that are assigned by sysbuild, which
allows for dynamically supporting different configurations without
needing dummy images to support different modes. Also fixes
multiple deficiencies with the previous code where things were not
properly accounted for e.g. using the swap algorithm including all
swap status parts when updating s0/s1 MCUboot image which could
overwrite and corrupt the image data in the other slot

Signed-off-by: Jamie McCrae <jamie.mccrae@nordicsemi.no>
Adds support for child/parent images into the new dynamic image
number system, without supporting any of the new features. This
commit is scheduled to be removed from the tree when child/parent
support is dropped so is purposely kept separate

Signed-off-by: Jamie McCrae <jamie.mccrae@nordicsemi.no>
fixup! [nrf noup] treewide: add NCS partition manager support

Fixes an issue whereby the end address of MCUboot had no
alignment requirements what-so-ever and would produce completely
invalid configurations for NSIB that prevented it from being
upgradeable

Signed-off-by: Jamie McCrae <jamie.mccrae@nordicsemi.no>
Fixes the implementation so that it works with sysbuild. This will
need to be reworked separately when support for parent/child
images is removed

Signed-off-by: Jamie McCrae <jamie.mccrae@nordicsemi.no>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant