Skip to content

Conversation

@Hi-Im-David
Copy link
Contributor

@Hi-Im-David Hi-Im-David commented May 23, 2025

Adding support for AUDIOPLL in nrf7120 which uses the AUXPLL IP. FICR definitions not finalised yet so will need further changes when they are.

(NOW merged) Waiting for zephyrproject-rtos/zephyr#91065 to be cherry picked to sdk-zephyr

@Hi-Im-David Hi-Im-David requested review from a team and rob-robinson-14 as code owners May 23, 2025 10:58
@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 May 23, 2025
@NordicBuilder
Copy link
Contributor

NordicBuilder commented May 23, 2025

CI Information

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

Inputs:

Sources:

sdk-nrf: PR head: 8eec5929c3e2550f23d72cf470e980a9cffb5d49

more details

sdk-nrf:

PR head: 8eec5929c3e2550f23d72cf470e980a9cffb5d49
merge base: 1df65e37e080d59618a8aea23c61aa0c212017d8
target head (main): cf2ac324d778b7ba1bd2fd79168bd4db06d4ff25
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)
boards
│  ├── nordic
│  │  ├── nrf7120pdk
│  │  │  │ nrf7120_cpuapp_common.dtsi
dts
│  ├── common
│  │  ├── nordic
│  │  │  │ nrf7120_enga.dtsi

Outputs:

Toolchain

Version: 8ea1732c3a
Build docker image: docker-dtr.nordicsemi.no/sw-production/ncs-build:8ea1732c3a_bba2ea5f2e

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

  • ◻️ Toolchain - Skipped: existing toolchain is used
  • ❌ Build twister
    • sdk-nrf test count: 282
  • ❌ Integration tests
Disabled integration tests
    • desktop52_verification
    • doc-internal
    • test_ble_nrf_config
    • 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_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-main
    • test-fw-nrfconnect-rpc
    • test-fw-nrfconnect-rs
    • test-fw-nrfconnect-tfm
    • test-fw-nrfconnect-thread-main
    • test-low-level
    • 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
Copy link

You can find the documentation preview for this PR here.

@Hi-Im-David Hi-Im-David force-pushed the nrf7120-enga-auxpll-support branch from e42b758 to 2b6132b Compare May 23, 2025 14:16
@Hi-Im-David Hi-Im-David force-pushed the nrf7120-enga-auxpll-support branch 4 times, most recently from 4c159c4 to e45a0c5 Compare June 4, 2025 15:43
@Hi-Im-David Hi-Im-David marked this pull request as draft June 4, 2025 15:44
@Hi-Im-David Hi-Im-David marked this pull request as ready for review June 4, 2025 15:46
@Hi-Im-David Hi-Im-David force-pushed the nrf7120-enga-auxpll-support branch from e45a0c5 to 96407ba Compare June 4, 2025 15:52
@Hi-Im-David Hi-Im-David force-pushed the nrf7120-enga-auxpll-support branch from 96407ba to 5e8f0bf Compare June 11, 2025 15:54
@Hi-Im-David
Copy link
Contributor Author

Now waiting for merge into sdk-zephyr: nrfconnect/sdk-zephyr#2994

@Hi-Im-David Hi-Im-David force-pushed the nrf7120-enga-auxpll-support branch 6 times, most recently from 9c214ca to c2aa4da Compare July 7, 2025 16:17
@Hi-Im-David Hi-Im-David requested review from anangl and nordicjm July 7, 2025 16:19
@Hi-Im-David
Copy link
Contributor Author

ping @anangl. Please could you review

Define AUDIOPLL, an AUXPLL IP instance used by the TDM and PDM IP. PLL
can be configured to set it's frequency based on marcros in
nrf-audiopll.h. FICR not fully defined yet. Also added TDM node.

Signed-off-by: David Jewsbury <[email protected]>
Enable audiopll (required by PDM and TDM) and setup frequency.

Signed-off-by: David Jewsbury <[email protected]>
@Hi-Im-David Hi-Im-David force-pushed the nrf7120-enga-auxpll-support branch from c2aa4da to 8eec592 Compare July 29, 2025 14:24
@Hi-Im-David Hi-Im-David requested a review from anangl July 29, 2025 14:25
Copy link
Contributor

Choose a reason for hiding this comment

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

Does nordic,dither-disable need to be defiend somewhere?

@github-actions
Copy link

This pull request has been marked as stale because it has been open (more than) 30 days with no activity. Remove the stale label or add a comment saying that you would like to have the label removed otherwise this pull request will automatically be closed in 7 days. Note, that you can always re-open a closed pull request at any time.

@github-actions github-actions bot added the Stale label Aug 29, 2025
@github-actions github-actions bot closed this Sep 6, 2025
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. Stale

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants