Add nrf54lm20dk and nrf54h20dk builds - #13
Draft
noahp wants to merge 1 commit into
Draft
Conversation
|
![github-advanced-security[bot]](https://avatars.githubusercontent.com/in/57789?s=60&v=4){kind=small}
Comment on lines
+229
to
+237
| needs: [validate, publish] | ||
| if: ${{ inputs.dry_run }} | ||
| runs-on: ubuntu-24.04 | ||
| steps: | ||
| - name: Dry-run summary | ||
| if: ${{ inputs.dry_run }} | ||
| run: | | ||
| echo "DRY RUN: skipped git tag and GitHub release for ${{ needs.validate.outputs.tag }}." | ||
| echo "Built ${{ steps.release_files.outputs.hex }}, ${{ steps.release_files.outputs.elf }}, ${{ steps.release_files.outputs.spdx }} and uploaded symbols to Memfault." | ||
| echo "Built and renamed all boards' files and uploaded symbols to Memfault." | ||
| echo "Renamed release files are attached as workflow artifacts for inspection." |
There was a problem hiding this comment.
Pull request overview
Expands quickstart-bluetooth from a single-target (nRF54L15) app to a multi-board app by adding nRF54LM20 (single-image) and nRF54H20 (sysbuild multi-image with cpurad controller) support, and updates CI/release automation and docs accordingly.
Changes:
- Add CI and release pipelines that build artifacts for all supported boards, including sysbuild + merged
.hexfor nRF54H20. - Add nRF54H20 sysbuild configuration (ipc_radio + radio_loader overlays) plus board-specific configs/overlays.
- Update docs/tests/gateway messaging to refer to “supported DKs” rather than only nRF54L15.
Reviewed changes
Copilot reviewed 25 out of 25 changed files in this pull request and generated 2 comments.
Show a summary per file
| File | Description |
|---|---|
| test/gateway/README.md | Updates gateway prerequisites to allow multiple supported DKs. |
| test/gateway/package.json | Generalizes gateway description to all supported boards. |
| test/gateway/gateway.js | Updates gateway header comment to reflect multi-board support. |
| test/e2e/test_project_key.py | Generalizes e2e test header comment to supported DKs. |
| test/e2e/test_device_info.py | Generalizes e2e test header comment to supported DKs. |
| RELEASE.md | Updates release process documentation for per-board builds/artifacts. |
| README.md | Documents supported boards and sysbuild vs no-sysbuild build/flash steps. |
| CLAUDE.md | Updates repository guidance to reflect multi-board + sysbuild support. |
| app/sysbuild/radio_loader.overlay | Adds sysbuild overlay for radio_loader partition/RAM selection. |
| app/sysbuild/ipc_radio/prj.conf | Adds cpurad ipc_radio image configuration for nRF54H20 BLE controller. |
| app/sysbuild/ipc_radio/boards/nrf54h20dk_nrf54h20_cpurad.overlay | Adds cpurad DT overlay (entropy/rng, memory regions, partitions). |
| app/sysbuild/ipc_radio/boards/nrf54h20dk_nrf54h20_cpurad.conf | Adds cpurad Kconfig overrides for nRF54H20 ipc_radio image. |
| app/src/main.c | Updates top-level control mapping comment to list supported DKs. |
| app/prj.conf | Moves DIS HW revision string to per-board config files. |
| app/Kconfig.sysbuild | Enables required sysbuild defaults for nRF54H20 radio-loader/ipc-radio flow. |
| app/common/memory_map_tcm.dtsi | Adds MRAM partition changes to support radio loader + loaded cpurad firmware slots. |
| app/common/memory_map_ram_pm_cpurad.dtsi | Adds pm_ramfunc relocation for cpurad power-management needs. |
| app/common/memory_map_ram_cpurad.dtsi | Expands cpurad TCM RAM region mapping for loaded firmware. |
| app/boards/nrf54lm20dk_nrf54lm20a_cpuapp.conf | Adds board-specific DIS HW revision string for nRF54LM20. |
| app/boards/nrf54l15dk_nrf54l15_cpuapp.conf | Adds board-specific DIS HW revision string for nRF54L15. |
| app/boards/nrf54h20dk_nrf54h20_cpuapp.overlay | Adds nRF54H20 cpuapp overlay (disable unused peripherals, PM runtime for console UART). |
| app/boards/nrf54h20dk_nrf54h20_cpuapp.conf | Adds nRF54H20 cpuapp Kconfig overrides (enable cpurad, PM settings). |
| .github/workflows/release.yml | Reworks release workflow into per-board build/publish and combined release asset upload. |
| .github/workflows/build.yml | Extends CI build workflow to a board matrix (including sysbuild flag). |
| .github/workflows/build-app.yml | Parameterizes reusable build workflow by board/board-id/sysbuild; adds sysbuild hex merge + normalized artifacts. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Comment on lines
182
to
184
| --software-type "${{ needs.build.outputs.fw_type }}" \ | ||
| --software-version "${{ needs.build.outputs.fw_version }}" \ | ||
| --check-uploaded \ |
Comment on lines
+10
to
+14
| * Theoretically it would not be necessary to relocate it if the whole code is in TCM. | ||
| * The changes in Zephyr would be required to allow it. | ||
| * As a workaround, relocate the pm_ramfunc to the end of the RAM block but before | ||
| * the MCUboot trailer. | ||
| */ |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
No description provided.