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

[nrfconnect] Extended DFU documentation about using mcumgr tool. #7251

Merged

Conversation

kkasperczyk-no
Copy link
Contributor

Problem

Examples documentation about performing Device Firmware Upgrade over BLE describes only case using Nordic mobile app. It would be beneficial to have also documentation that shows how to perform firmware upgrade from PC command line tool.

Change overview

  • Moved DFU description from examples to separate guide readme.
  • Extended documentation by case using mcumgr command line tool to perform DFU.
  • Removed unnecessary part of the code allowing to advertise over BLE with non conn type that was added in very early verison
    of BLE Manager implementation.

Testing

Change introduces mainly documentation, so it was verified manually that introduced steps work properly. Moreover it was verified that changed BLE configs doesn't have negative impact and after introducing them following things work properly:

  • Performing DFU over BLE with smartphone running Nordic's nRF Connect for Mobile application with Thread on accessory disabled and enabled.
  • Performing DFU over BLE with mcumgr command line tool on Linux with Thread on accessory disabled and enabled.
  • Device commissioning over BLE using Python CHIP controller.

@kkasperczyk-no kkasperczyk-no force-pushed the dfu_mcumgr_support_pr branch 4 times, most recently from fa49c8c to 0b4c660 Compare June 1, 2021 06:35
Examples documentation about performing Device Firmware Upgrade
over BLE describes only case using Nordic mobile app. It would
be beneficial to have also documentation that shows how to perform
firmware upgrade from PC command line tool.

* Moved DFU description from examples to separate guide readme.
* Extended documentation by case using mcumgr command line tool
to perform DFU.
* Removed unnecessary part of the code allowing to advertise
over BLE with non conn type that was added in very early verison
of BLE Manager implementation.
@github-actions
Copy link

github-actions bot commented Jun 1, 2021

Size increase report for "nrfconnect-example-build" from c0ac96c

File Section File VM
chip-lock.elf text -32 -32
chip-shell.elf text -32 -32
Full report output
BLOAT REPORT

Files found only in the build output:
    report.csv

Comparing ./master_artifact/chip-lock.elf and ./pull_artifact/chip-lock.elf:

sections,vmsize,filesize
.debug_loc,0,-24
text,-32,-32
.debug_info,0,-48
.debug_line,0,-48

Comparing ./master_artifact/chip-shell.elf and ./pull_artifact/chip-shell.elf:

sections,vmsize,filesize
.debug_loc,0,-24
text,-32,-32
.debug_info,0,-48
.debug_line,0,-48


@bzbarsky-apple bzbarsky-apple merged commit 17821b9 into project-chip:master Jun 1, 2021
nikita-s-wrk pushed a commit to nikita-s-wrk/connectedhomeip that referenced this pull request Sep 23, 2021
…ject-chip#7251)

Examples documentation about performing Device Firmware Upgrade
over BLE describes only case using Nordic mobile app. It would
be beneficial to have also documentation that shows how to perform
firmware upgrade from PC command line tool.

* Moved DFU description from examples to separate guide readme.
* Extended documentation by case using mcumgr command line tool
to perform DFU.
* Removed unnecessary part of the code allowing to advertise
over BLE with non conn type that was added in very early verison
of BLE Manager implementation.
@kkasperczyk-no kkasperczyk-no deleted the dfu_mcumgr_support_pr branch December 22, 2021 11:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants