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

Common download scripts for NRF5x #3352

Merged
merged 5 commits into from
Dec 2, 2024

Conversation

kasjer
Copy link
Contributor

@kasjer kasjer commented Nov 28, 2024

This PR:

  • switches Nordic DK boards to use common download script download.sh.
    That way it's possible to select downloader with syscfg value MYNEWT_DOWNLOADER (jlink/nrfjprog/nrfutil/pyocd).
  • adds support to download using new Nordic tool nrfutil.
  • enables verification when nrfjprog is selected
  • adds second flash map for pca10059 (USB dongle that ships without debugger and SWD connector) selectable with syscfg value. When BSP_NRF_SDK_FLASH_LAYOUT=1 Nordic specific flash map is used. Additionally nrfutil will download firmware using nordicDfu protocol activated after reset button in pressed

@kasjer kasjer requested a review from sjanc November 28, 2024 13:29
@github-actions github-actions bot added Nordic Nordic related BSP size/m labels Nov 28, 2024
@kasjer kasjer force-pushed the kasjer/load-with-nrfutil branch from 79a1bc6 to f5e7612 Compare November 28, 2024 13:45
Copy link
Contributor

@sjanc sjanc left a comment

Choose a reason for hiding this comment

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

typos in syscfg, otherwise looks OK

NRFUTIL_TRAITS:
description: >
Traits passed to nrfutil command. If not set jlink is default.
It can be 'nordicDfu' to to uploade binary using Nordic SDK
Copy link
Contributor

Choose a reason for hiding this comment

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

nitpick: typos

This add initial support to program Nordic targets
with nrfutil.

Signed-off-by: Jerzy Kasenberg <[email protected]>
With this change it's possible to flash USB dongle with
nrfutil without connected debugger (dongle does not have
SWD connector).

Signed-off-by: Jerzy Kasenberg <[email protected]>
This also changes default downloader to nrfutil

Signed-off-by: Jerzy Kasenberg <[email protected]>
This also changes default downloader to nrfutil

Signed-off-by: Jerzy Kasenberg <[email protected]>
This enables verification after flashing.

Signed-off-by: Jerzy Kasenberg <[email protected]>
@kasjer kasjer force-pushed the kasjer/load-with-nrfutil branch from f5e7612 to 222e7dc Compare December 2, 2024 12:10
@kasjer kasjer merged commit 7a15120 into apache:master Dec 2, 2024
20 checks passed
@kasjer kasjer deleted the kasjer/load-with-nrfutil branch December 2, 2024 16:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants