-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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
samples: matter: Enable DFU over BLE SMP for nRF54H20 #17525
base: main
Are you sure you want to change the base?
Conversation
ArekBalysNordic
commented
Sep 27, 2024
- Works with external flash utilizing companion image.
- Uses SUIT CACHE RW to store app & rad images into the cache partition.
The following west manifest projects have been modified in this Pull Request:
Note: This message is automatically posted and updated by the Manifest GitHub Action. |
CI InformationTo view the history of this post, clich the 'edited' button above Inputs:Sources:sdk-nrf: PR head: 39768e66fb6e512c20c30476f5164938e6ca4308 more detailssdk-nrf:
matter:
Github labels
List of changed files detected by CI (16)
Outputs:ToolchainVersion: 9583beca34 Test Spec & Results: ✅ Success; ❌ Failure; 🟠 Queued; 🟡 Progress; ◻️ Skipped;
|
You can find the documentation preview for this PR at this link. It will be updated about 10 minutes after the documentation build succeeds. Note: This comment is automatically posted by the Documentation Publishing GitHub Action. |
5af2a13
to
b0a8b09
Compare
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.
Would it make sense to enable DFU over SMP for Matter bridge as well?
I guess it is not applicable there until the dfu_partition is so big. We need to wait for the new approach to enable it for Matter Bridge. |
@@ -26,6 +26,8 @@ SUIT_Envelope_Tagged: | |||
- {{ radio['dt'].chosen_nodes['zephyr,code-partition'].regs[0].size }} | |||
- - CAND_IMG | |||
- 0 | |||
- - CACHE_POOL |
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.
Actually this could be wrapped in
{%- if 'CONFIG_SUIT_DFU_CACHE_EXTRACT_IMAGE' not in radio['config'] or radio['config']['CONFIG_SUIT_DFU_CACHE_EXTRACT_IMAGE'] == '' %}
but this has to be aligned anyway in default SUIT templates, so I will probably do this myself
samples/matter/lock/sysbuild/ipc_radio/boards/nrf54h20dk_nrf54h20_cpurad.conf
Outdated
Show resolved
Hide resolved
- Works with external flash utilizing companion image. - Uses SUIT CACHE RW to store app & rad images into the cache partition. Signed-off-by: Arkadiusz Balys <[email protected]>
b0a8b09
to
39768e6
Compare