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

doc: nrf util: usb driver prerequisite for win #19809

Merged
merged 1 commit into from
Jan 13, 2025

Conversation

greg-fer
Copy link
Contributor

@greg-fer greg-fer commented Jan 8, 2025

Added information about the required installation of USB Driver for J-Link on Windows. Moved installation instructions to nRF Util docs and removed them from the nRF54H20 Getting Started Guide. NCD-909.


Related nRF Util PR https://github.com/NordicSemiconductor/nrfutil-rs/pull/2563


  • Changelog entry

@greg-fer greg-fer added CI-disable Disable CI for this PR doc only labels Jan 8, 2025
@greg-fer greg-fer added this to the 3.0.0 milestone Jan 8, 2025
@greg-fer greg-fer requested review from KievDevel and qung January 8, 2025 16:55
@github-actions github-actions bot 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 8, 2025
@NordicBuilder
Copy link
Contributor

NordicBuilder commented Jan 8, 2025

CI Information

To view the history of this post, clich the 'edited' button above
Build number: 5

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

@NordicBuilder
Copy link
Contributor

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 Publish GitHub Action.

@greg-fer greg-fer force-pushed the doc_nrfutil_jlink_usb_driver_req branch from 66cba88 to 2c1f71e Compare January 10, 2025 10:29
@github-actions github-actions bot removed the changelog-entry-required Update changelog before merge. Remove label if entry is not needed or already added. label Jan 10, 2025
@greg-fer greg-fer force-pushed the doc_nrfutil_jlink_usb_driver_req branch 2 times, most recently from 376a63f to 4cf4351 Compare January 10, 2025 11:38
@greg-fer greg-fer marked this pull request as ready for review January 10, 2025 11:38
@greg-fer greg-fer requested review from a team and FrancescoSer as code owners January 10, 2025 11:38
* For Windows, add it to the environment variables.

* SEGGER J-Link |jlink_ver| and, on Windows, also the SEGGER USB Driver for J-Link from `SEGGER J-Link`_ |jlink_ver|.
For information how to install the USB Driver, see the `nRF Util prerequisites`_ documentation.
Copy link
Contributor

@peknis peknis Jan 10, 2025

Choose a reason for hiding this comment

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

Suggested change
For information how to install the USB Driver, see the `nRF Util prerequisites`_ documentation.
For information on how to install the USB Driver, see the `nRF Util prerequisites <nRF Util prerequisites_>`_ documentation.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I won't be updating prerequisites to prequisites.

Copy link
Contributor

Choose a reason for hiding this comment

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

Sorry for the typo. It is gone now.

* The latest version of |VSC| for your operating system from the `Visual Studio Code download page`_.
* In |VSC|, the latest version of the `nRF Connect for VS Code Extension Pack`_.

* Additionally for Windows users: SEGGER USB Driver for J-Link, required for support of older Nordic Semiconductor devices in nRF Util.
For information how to install the USB Driver, see the `nRF Util prerequisites`_ documentation.
Copy link
Contributor

@peknis peknis Jan 10, 2025

Choose a reason for hiding this comment

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

Suggested change
For information how to install the USB Driver, see the `nRF Util prerequisites`_ documentation.
For information on how to install the USB Driver, see the `nRF Util prerequisites <nRF Util prerequisites_>`_ documentation.

Copy link
Contributor

Choose a reason for hiding this comment

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

In the target page of the link, change "What J-link version..." to Which J-link version...


* Additionally for Windows users: SEGGER USB Driver for J-Link, required for support of older Nordic Semiconductor devices in nRF Util.
For information how to install the USB Driver, see the `nRF Util prerequisites`_ documentation.
Copy link
Contributor

@peknis peknis Jan 10, 2025

Choose a reason for hiding this comment

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

Suggested change
For information how to install the USB Driver, see the `nRF Util prerequisites`_ documentation.
For information on how to install the USB Driver, see the `nRF Util prerequisites <nRF Util prerequisites_>`_ documentation.

@@ -287,6 +287,9 @@ Among others, this package includes the J-Link RTT Viewer, which can be used for

It is recommended to use the |jlink_ver| of the package when you :ref:`installing_vsc`.

On Windows, you also need to install SEGGER USB Driver for J-Link, which is required for support of older Nordic Semiconductor devices in :ref:`requirements_nrf_util`.
For information how to install the USB Driver, see the `nRF Util prerequisites`_ documentation.
Copy link
Contributor

@peknis peknis Jan 10, 2025

Choose a reason for hiding this comment

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

Suggested change
For information how to install the USB Driver, see the `nRF Util prerequisites`_ documentation.
For information on how to install the USB Driver, see the `nRF Util prerequisites <nRF Util prerequisites_>`_ documentation.

Added information about the required installation of USB Driver
for J-Link on Windows. Moved installation instructions to nRF Util docs
and removed them from the nRF54H20 Getting Started Guide.
NCD-909.

Signed-off-by: Grzegorz Ferenc <[email protected]>
@greg-fer greg-fer force-pushed the doc_nrfutil_jlink_usb_driver_req branch from 4cf4351 to bcc0d22 Compare January 10, 2025 13:18
@greg-fer
Copy link
Contributor Author

Requested merge to @rlubos because @annwoj 's approval fulfills the requirement for nRF54H20-related doc edits, as per https://nordicsemi.atlassian.net/wiki/spaces/TECHDOC/pages/120302385/NCS+team+assignment

@rlubos rlubos merged commit 0e43ec2 into nrfconnect:main Jan 13, 2025
13 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
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.

8 participants