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

applications: nrf_desktop: Add CMake warning for MCUboot using KMU #19440

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

MarekPieta
Copy link
Contributor

Using KMU in MCUboot requires manual provisioning of the MCUboot keys. Change adds a CMake warning to ensure that user is aware that an extra step is needed before programming the board.

Jira: NCSDK-30742

@MarekPieta MarekPieta added the backport v2.9-branch auto-create a PR with same commits to v2.9-branch label Dec 11, 2024
@MarekPieta MarekPieta added this to the 2.9.0 milestone Dec 11, 2024
@MarekPieta MarekPieta requested review from zycz, alstrzebonski and a team December 11, 2024 14:09
@MarekPieta MarekPieta requested a review from a team as a code owner December 11, 2024 14:09
@github-actions github-actions bot added the changelog-entry-required Update changelog before merge. Remove label if entry is not needed or already added. label Dec 11, 2024
@MarekPieta MarekPieta removed the changelog-entry-required Update changelog before merge. Remove label if entry is not needed or already added. label Dec 11, 2024
@MarekPieta MarekPieta requested a review from nvlsianpu December 11, 2024 14:11
@NordicBuilder
Copy link
Contributor

NordicBuilder commented Dec 11, 2024

CI Information

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

Inputs:

Sources:

sdk-nrf: PR head: e834359e3c950d5519cb758f0feaffded3bded11

more details

sdk-nrf:

PR head: e834359e3c950d5519cb758f0feaffded3bded11
merge base: 3bb163563d7f1839fc1218eb21cd2dc6f148c9d0
target head (main): ffa08e9d4e592eaf079eb6f9dccfe14ec9d50bf5
Diff

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 (1)
applications
│  ├── nrf_desktop
│  │  ├── sysbuild
│  │  │  │ CMakeLists.txt

Outputs:

Toolchain

Version: b77d8c1312
Build docker image: docker-dtr.nordicsemi.no/sw-production/ncs-build:b77d8c1312_912848a074

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

  • ◻️ Toolchain - Skipped: existing toolchain is used
  • ✅ Build twister
    • sdk-nrf test count: 93
  • ✅ Integration tests
    • ✅ desktop52_verification
Disabled integration tests
    • test_ble_nrf_config
    • test-fw-nrfconnect-apps
    • test-fw-nrfconnect-ble_mesh
    • test-fw-nrfconnect-ble_samples
    • test-fw-nrfconnect-boot
    • test-fw-nrfconnect-chip
    • test-fw-nrfconnect-fem
    • test-fw-nrfconnect-nfc
    • test-fw-nrfconnect-nrf-iot_libmodem-nrf
    • test-fw-nrfconnect-nrf-iot_lwm2m
    • test-fw-nrfconnect-nrf-iot_mosh
    • test-fw-nrfconnect-nrf-iot_nrf_provisioning
    • test-fw-nrfconnect-nrf-iot_positioning
    • test-fw-nrfconnect-nrf-iot_samples
    • test-fw-nrfconnect-nrf-iot_serial_lte_modem
    • test-fw-nrfconnect-nrf-iot_thingy91
    • test-fw-nrfconnect-nrf-iot_zephyr_lwm2m
    • test-fw-nrfconnect-nrf_crypto
    • test-fw-nrfconnect-ps
    • test-fw-nrfconnect-rpc
    • test-fw-nrfconnect-rs
    • test-fw-nrfconnect-tfm
    • test-fw-nrfconnect-thread
    • test-fw-nrfconnect-zigbee
    • test-low-level
    • test-sdk-audio
    • test-sdk-dfu
    • test-sdk-find-my
    • test-sdk-mcuboot
    • test-sdk-pmic-samples
    • test-sdk-sidewalk
    • test-sdk-wifi
    • test-secdom-samples-public

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

@github-actions github-actions bot added the changelog-entry-required Update changelog before merge. Remove label if entry is not needed or already added. label Dec 11, 2024
Using KMU in MCUboot requires manual provisioning of the MCUboot keys.
Change adds a CMake warning to ensure that user is aware that an extra
step is needed before programming the board.

Jira: NCSDK-30742

Signed-off-by: Marek Pieta <[email protected]>
@MarekPieta MarekPieta requested a review from nordicjm December 11, 2024 14:21
@MarekPieta MarekPieta removed the changelog-entry-required Update changelog before merge. Remove label if entry is not needed or already added. label Dec 11, 2024
@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.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport v2.9-branch auto-create a PR with same commits to v2.9-branch
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants