Skip to content

mgmt: mcumgr: img_mgmt: rewrite to claim data untill write completes#741

Open
eivindj-nordic wants to merge 1 commit intonrfconnect:mainfrom
eivindj-nordic:dfu_improv
Open

mgmt: mcumgr: img_mgmt: rewrite to claim data untill write completes#741
eivindj-nordic wants to merge 1 commit intonrfconnect:mainfrom
eivindj-nordic:dfu_improv

Conversation

@eivindj-nordic
Copy link
Copy Markdown
Contributor

This allows only one write operation in flight at the same time, though the next will be started on the event of write complete if applicable.

Wait for all image write operations to complete before rebooting the firmware_loader application. This is currently using the new img_mgmt_write_in_progress() function directly in main, which works, but jumps over some layers of abstraction.

@eivindj-nordic eivindj-nordic added this to the v2.0.0 milestone Apr 11, 2026
@eivindj-nordic eivindj-nordic self-assigned this Apr 11, 2026
@eivindj-nordic eivindj-nordic requested a review from a team as a code owner April 11, 2026 09:52
@eivindj-nordic eivindj-nordic added the bug Something isn't working label Apr 11, 2026
@github-actions github-actions bot added the changelog-entry-required Update changelog before merge. Remove label if entry is not needed or already added. label Apr 11, 2026
@eivindj-nordic
Copy link
Copy Markdown
Contributor Author

eivindj-nordic commented Apr 11, 2026

Updates to changes done after previous release, no changelog required.

Added new Kconfig options.

@eivindj-nordic eivindj-nordic removed the changelog-entry-required Update changelog before merge. Remove label if entry is not needed or already added. label Apr 11, 2026
@eivindj-nordic eivindj-nordic requested a review from a team April 11, 2026 09:53
@github-actions
Copy link
Copy Markdown

You can find the documentation preview for this PR here.

@github-actions github-actions bot added the changelog-entry-required Update changelog before merge. Remove label if entry is not needed or already added. label Apr 11, 2026
@eivindj-nordic eivindj-nordic removed the changelog-entry-required Update changelog before merge. Remove label if entry is not needed or already added. label Apr 11, 2026
@eivindj-nordic eivindj-nordic requested review from a team as code owners April 11, 2026 11:01
@github-actions github-actions bot added the doc-required PR must not be merged without tech writer approval. label Apr 11, 2026
@eivindj-nordic eivindj-nordic force-pushed the dfu_improv branch 6 times, most recently from 7ef7ec2 to 3911496 Compare April 11, 2026 11:20
This allows only one write operation in flight at the same time, though
the next will be started on the event of write complete if applicable.

Wait for all image write operations to complete before rebooting the
firmware_loader application. This is currently using the new
img_mgmt_write_in_progress() function directly in main, which works, but
jumps over some layers of abstraction.

Signed-off-by: Eivind Jølsgard <eivind.jolsgard@nordicsemi.no>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working doc-required PR must not be merged without tech writer approval.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant