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.

Sign up for GitHub

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

Jump to bottom

manifest: sdk-zephyr: [nrf fromlist] modules: hal_nordic: bump regtool to 9.0.0 #21130

Open
wants to merge 2 commits into
base: main
Choose a base branch
from
Open

manifest: sdk-zephyr: [nrf fromlist] modules: hal_nordic: bump regtool to 9.0.0 #21130

wants to merge 2 commits into from

Conversation

NordicBuilder
Copy link
Contributor

@NordicBuilder NordicBuilder commented Mar 25, 2025 •
edited by jonathannilsen
Loading

DNM before #21152

Automatically created by action-manifest-pr GH action from PR: nrfconnect/sdk-zephyr#2663

@NordicBuilder NordicBuilder requested review from a team as code owners March 25, 2025 09:07
@github-actions github-actions bot added manifest changelog-entry-required Update changelog before merge. Remove label if entry is not needed or already added. labels Mar 25, 2025
@NordicBuilder
Copy link
Contributor Author

NordicBuilder commented Mar 25, 2025 •
edited
Loading

The following west manifest projects have changed revision in this Pull Request:

Name Old Revision New Revision Diff
zephyr nrfconnect/sdk-zephyr@60cd202 (main) nrfconnect/sdk-zephyr#2663 nrfconnect/sdk-zephyr#2663/files

⛔ DNM label due to: 1 project with PR revision

Note: This message is automatically posted and updated by the Manifest GitHub Action.

@NordicBuilder
Copy link
Contributor Author

NordicBuilder commented Mar 25, 2025 •
edited
Loading

CI Information

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

Inputs:

Sources:

sdk-nrf: PR head: e54e7c4ef498d30fd356cb13671235d6e58eb2d4
zephyr: PR head: 51f6d6d472721cb3b5483bbe753404e2d22e0fb4

more details

sdk-nrf:

PR head: e54e7c4ef498d30fd356cb13671235d6e58eb2d4
merge base: 7f5401f70e20647dfa15592d8c3c0a43e3b634a4
target head (main): 7f5401f70e20647dfa15592d8c3c0a43e3b634a4
Diff

zephyr:

PR head: 51f6d6d472721cb3b5483bbe753404e2d22e0fb4
merge base: 60cd202d25ddbf615f9e542e91eb2b4effb06bcf
target head (main): 60cd202d25ddbf615f9e542e91eb2b4effb06bcf
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 (4) 
scripts
│  ├── requirements-build.txt
│  │ requirements-fixed.txt
west.yml
zephyr
│  ├── modules
│  │  ├── hal_nordic
│  │  │  │ CMakeLists.txt

Outputs:

Toolchain

Version: 7dbe0e2658
Build docker image: docker-dtr.nordicsemi.no/sw-production/ncs-build:7dbe0e2658_8bf7ca4353

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

  • ◻️ Toolchain - Skipped: existing toolchain is used
  • ✅ Build twister
    • sdk-nrf test count: 59
    • sdk-zephyr test count: 478
  • ✅ Integration tests
    • ✅ test-fw-nrfconnect-nfc
    • ✅ test-fw-nrfconnect-rs
    • ✅ test-fw-nrfconnect-fem
    • ✅ test-low-level
Disabled integration tests
    • desktop52_verification
    • doc-internal
    • 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-nrf-iot_cloud
    • test-fw-nrfconnect-nrf-iot_libmodem-nrf
    • test-fw-nrfconnect-nrf-iot_mosh
    • 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-proprietary_esb
    • test-fw-nrfconnect-ps
    • test-fw-nrfconnect-rpc
    • test-fw-nrfconnect-tfm
    • test-fw-nrfconnect-thread
    • test-sdk-audio
    • test-sdk-dfu
    • test-sdk-find-my
    • test-sdk-mcuboot
    • test-sdk-pmic-samples
    • test-sdk-wifi
    • test-secdom-samples-public

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

@github-actions GitHub Actions
Copy link

You can find the documentation preview for this PR here.

@jonathannilsen jonathannilsen force-pushed the auto-manifest-sdk-zephyr-2663 branch from 5bec9b1 to 225dc20 Compare March 25, 2025 09:35
@jonathannilsen jonathannilsen requested review from a team as code owners March 25, 2025 09:35
@NordicBuilder NordicBuilder force-pushed the auto-manifest-sdk-zephyr-2663 branch 3 times, most recently from 294793f to 9a55547 Compare March 25, 2025 09:45
@jonathannilsen jonathannilsen force-pushed the auto-manifest-sdk-zephyr-2663 branch from 9a55547 to 3ab1e4b Compare March 25, 2025 11:52
jonathannilsen and others added 2 commits March 26, 2025 07:48
@jonathannilsen @NordicBuilder
scripts: requirements: bump nrf-regtool to 9.0.0
a025108 
Added support for VPR GPIOs and a more concise output print.
This version also contains a large cleanup of the package
and removes several features that are not used by NCS.

Ref: NCSDK-32438

Signed-off-by: Jonathan Nilsen <jonathan.nilsen@nordicsemi.no>
@NordicBuilder
manifest: Update sdk-zephyr revision (auto-manifest PR)
e54e7c4 
Automatically created by Github Action

Signed-off-by: Nordic Builder <pylon@nordicsemi.no>
@NordicBuilder NordicBuilder force-pushed the auto-manifest-sdk-zephyr-2663 branch from 3ab1e4b to e54e7c4 Compare March 26, 2025 07:48
@jonathannilsen jonathannilsen requested a review from a team March 26, 2025 07:49
karstenkoenig
karstenkoenig approved these changes Mar 26, 2025
View reviewed changes
west.yml
@@ -65,7 +65,7 @@ manifest:
# https://developer.nordicsemi.com/nRF_Connect_SDK/doc/latest/zephyr/guides/modules.html
- name: zephyr
repo-path: sdk-zephyr
revision: 60cd202d25ddbf615f9e542e91eb2b4effb06bcf
Copy link
Contributor

Choose a reason for hiding this comment

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

This will require sdk-zephyr to be merged first, then this sdk-nrf PR will need to be updated with the new sha and then merged once that is green. In this time window you can't bump sdk-zephyr reference through any other PR as it will lack the nrf-regtool 9.0.0 version in the toolchain, only this one will have it, making it a blocker for all sdk-zephyr manifest changes. Is this is needed because of the major bump of nrf-regtool?

Copy link
Contributor

Choose a reason for hiding this comment

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

Good point, and not really, I can create a separate PR for the requirements bump to circumvent this

Copy link
Contributor

Choose a reason for hiding this comment

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

#21152

@jonathannilsen jonathannilsen mentioned this pull request Mar 26, 2025
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@jonathannilsen jonathannilsen jonathannilsen left review comments

@karstenkoenig karstenkoenig karstenkoenig approved these changes

At least 2 approving reviews are required to merge this pull request.

Assignees
No one assigned
Labels
changelog-entry-required Update changelog before merge. Remove label if entry is not needed or already added. DNM manifest manifest-zephyr
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@NordicBuilder @karstenkoenig @jonathannilsen