Skip to content

Conversation

@dhsu-nordic
Copy link
Contributor

Previously failing due to incorrect pclk frequency on TDM which is now using pclk32m. The clock hfxo32m is also renamed to pclk32m.

Also rename hfxo32m to pclk32m.

Signed-off-by: Dhanoo Surasarang <[email protected]>
Previously failing due to incorrect pclk frequency on TDM.

Signed-off-by: Dhanoo Surasarang <[email protected]>
Copilot AI review requested due to automatic review settings December 5, 2025 15:09
@dhsu-nordic dhsu-nordic requested review from a team as code owners December 5, 2025 15:09
@NordicBuilder NordicBuilder added the changelog-entry-required Update changelog before merge. Remove label if entry is not needed or already added. label Dec 5, 2025
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR adds a new test case for the nRF7120 platform and corrects clock naming inconsistencies in the device tree. The changes enable a previously failing test case by ensuring TDM uses the correct 32MHz clock source.

  • Adds aclk_mck test case for nRF7120 platform that was previously failing
  • Renames hfxo32m clock to pclk32m for consistency and accuracy
  • Updates all clock references to use the renamed pclk32m

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.

File Description
tests/drivers/i2s/i2s_mclk/testcase.yaml Adds new test case configuration for nRF7120 platform with gpio_loopback fixture
dts/common/nordic/nrf7120_enga.dtsi Renames hfxo32m to pclk32m and updates all references to the renamed clock

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@NordicBuilder
Copy link
Contributor

NordicBuilder commented Dec 5, 2025

CI Information

To view the history of this post, click the 'edited' button above
Build number: 2

Inputs:

Sources:

sdk-nrf: PR head: b0086fa2debee33b5bbe0f1db08ef39051ef43c9

more details

sdk-nrf:

PR head: b0086fa2debee33b5bbe0f1db08ef39051ef43c9
merge base: 52ea3d71c4e7837acf2c83f44c7338eae835169b
target head (main): 21d701498b6b22848f66f72caa57aef41c04b1a5
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 (2)
dts
│  ├── common
│  │  ├── nordic
│  │  │  │ nrf7120_enga.dtsi
tests
│  ├── drivers
│  │  ├── i2s
│  │  │  ├── i2s_mclk
│  │  │  │  │ testcase.yaml

Outputs:

Toolchain

Version: 43683a87ea
Build docker image: docker-dtr.nordicsemi.no/sw-production/ncs-build:43683a87ea_5ea73affbf

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

  • ◻️ Toolchain - Skipped: existing toolchain is used
  • ✅ Build twister - Skipped: Skipping Build & Test as it succeeded in a previous run: 1
  • ✅ Integration tests
    • ✅ test-low-level
Disabled integration tests
    • test-fw-nrfconnect-nrf_lrcs_mosh
    • test-fw-nrfconnect-nrf_lrcs_positioning
    • desktop52_verification
    • test-fw-nrfconnect-apps
    • test-fw-nrfconnect-ble_mesh
    • test-fw-nrfconnect-ble_samples
    • test-fw-nrfconnect-chip
    • test-fw-nrfconnect-fem
    • test-fw-nrfconnect-nfc
    • test-fw-nrfconnect-nrf-iot_cloud
    • test-fw-nrfconnect-nrf-iot_libmodem-nrf
    • test-fw-nrfconnect-nrf-iot_lwm2m
    • test-fw-nrfconnect-nrf-iot_samples
    • test-fw-nrfconnect-nrf-iot_thingy91
    • test-fw-nrfconnect-nrf-iot_zephyr_lwm2m
    • test-fw-nrfconnect-nrf_crypto
    • test-fw-nrfconnect-ps-main
    • test-fw-nrfconnect-rpc
    • test-fw-nrfconnect-rs
    • test-fw-nrfconnect-tfm
    • test-fw-nrfconnect-thread-main
    • test-sdk-audio
    • test-sdk-dfu
    • test-sdk-find-my
    • test-sdk-mcuboot
    • test-sdk-wifi
    • test-secdom-samples-public

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

@rlubos rlubos merged commit 2b0ef01 into nrfconnect:main Dec 8, 2025
21 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

changelog-entry-required Update changelog before merge. Remove label if entry is not needed or already added.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants