Skip to content

Conversation

@greg-fer
Copy link
Contributor

@greg-fer greg-fer commented Jan 6, 2026

Added a link to the page about Erasing the device in nRF Util docs. NCSIDB-1744.

Added a link to the page about Erasing the device in nRF Util docs.
NCSIDB-1744.

Signed-off-by: Grzegorz Ferenc <[email protected]>
Copilot AI review requested due to automatic review settings January 6, 2026 14:59
@greg-fer greg-fer added CI-disable Disable CI for this PR doc only labels Jan 6, 2026
@NordicBuilder NordicBuilder added doc-required PR must not be merged without tech writer approval. changelog-entry-required Update changelog before merge. Remove label if entry is not needed or already added. labels Jan 6, 2026
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR adds a link to the nRF Util documentation page about erasing devices and improves the clarity of related instructions across documentation files. The changes enhance documentation by providing direct references to external resources and streamlining step numbering.

Key Changes:

  • Added a new link definition for "Erasing the device" in the nRF Util documentation
  • Updated instructions to reference the new link when describing device erase operations
  • Improved formatting consistency in the "device command" link text

Reviewed changes

Copilot reviewed 4 out of 4 changed files in this pull request and generated 2 comments.

File Description
doc/nrf/links.txt Added new link reference to nRF Util's device erasing guide
samples/crypto/kmu_cracen_usage/README.rst Restructured provisioning steps to reference the new erase documentation link and improved step numbering
doc/nrf/app_dev/device_guides/nrf54l/kmu_provision.rst Updated SoC erase instructions to reference the new documentation link
doc/nrf/app_dev/device_guides/nrf54h/ug_nrf54h20_ironside_update.rst Standardized "device command" link text formatting

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

#. Provision the example keys to the board using nRF Util.
Complete the following steps:
#. Perform a full erase of the device.
You can use nRF Util's `device command <Erasing the device_>`_ for this purpose:
Copy link

Copilot AI Jan 6, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The link text 'device command' is inconsistent with the actual link target. The text suggests the link points to general device command documentation, but it actually points to the specific erasing guide. Consider changing the link text to something like 'erase command' or adjusting to match the pattern used in other files (e.g., line 107 of kmu_provision.rst uses the same pattern).

Suggested change
You can use nRF Util's `device command <Erasing the device_>`_ for this purpose:
You can use nRF Util's `erase command <Erasing the device_>`_ for this purpose:

Copilot uses AI. Check for mistakes.
Before uploading keys, ensure that the SoC is unprovisioned.
If the SoC has been previously provisioned and you need to use a different set of keys, you must first erase the SoC with the following erase command:
If the SoC has been previously provisioned and you need to use a different set of keys, you must first erase the SoC.
You can use nRF Util's `device command <Erasing the device_>`_ for this purpose:
Copy link

Copilot AI Jan 6, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The link text 'device command' is inconsistent with the actual link target. The text suggests the link points to general device command documentation, but it actually points to the specific erasing guide. Consider changing the link text to something like 'erase command' to be more specific and accurate.

Suggested change
You can use nRF Util's `device command <Erasing the device_>`_ for this purpose:
You can use nRF Util's `erase command <Erasing the device_>`_ for this purpose:

Copilot uses AI. Check for mistakes.
@NordicBuilder
Copy link
Contributor

CI Information

To view the history of this post, click the 'edited' button above
Build number: 1

Inputs:

Sources:

more details

Github labels

Enabled Name Description
ci-disabled Disable the ci execution
ci-all-test Run all of ci, no test spec filtering will be done
ci-force-downstream Force execution of downstream even if twister fails
ci-run-twister Force run twister
ci-run-zephyr-twister Force run zephyr twister
List of changed files detected by CI (0)

Outputs:

Toolchain

Version:
Build docker image:

Test Spec & Results: ✅ Success; ❌ Failure; 🟠 Queued; 🟡 Progress; ◻️ Skipped; ⚠️ Quarantine

  • ◻️ Toolchain
  • ◻️ Build twister
  • ◻️ Integration tests

Note: This message is automatically posted and updated by the CI

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

changelog-entry-required Update changelog before merge. Remove label if entry is not needed or already added. CI-disable Disable CI for this PR doc only doc-required PR must not be merged without tech writer approval.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants