Skip to content

Conversation

@Hi-Im-David
Copy link
Contributor

The tests using the AUXPLL require support for a custom decimation ratio in the NRFX PDM driver which will come at a later date and therefore these will be quarantined. The pdm_loopback test also requires NRFX 4.0 release due to use of GPPI which needs changes in MDK and this PR: #25312

Added supported for nRF7120 dmic sample using the AUXPLL.

Signed-off-by: David Jewsbury <david.jewsbury@nordicsemi.no>
Added supported for pdm_loopback, pdm_instances and dmic_api tests
for nRF7120 which uses both PCLK and AUXPLL as driving clocks for
the PDM.

Signed-off-by: David Jewsbury <david.jewsbury@nordicsemi.no>
@Hi-Im-David Hi-Im-David requested a review from a team as a code owner October 31, 2025 10:21
@Hi-Im-David Hi-Im-David marked this pull request as draft October 31, 2025 10:21
@NordicBuilder NordicBuilder added the changelog-entry-required Update changelog before merge. Remove label if entry is not needed or already added. label Oct 31, 2025
@NordicBuilder
Copy link
Contributor

CI Information

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

Inputs:

Sources:

more details

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 (0)

Outputs:

Toolchain

Version:
Build docker image:

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

  • ❌ Toolchain
  • ❌ Build twister
  • ❌ Integration tests

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

*
* SPDX-License-Identifier: LicenseRef-Nordic-5-Clause
*/
/* Loopback between P0.0 and P1.14 required */
Copy link
Contributor

Choose a reason for hiding this comment

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

Is P0.0 correct? Shouldn't this be P1.0?

*
* SPDX-License-Identifier: LicenseRef-Nordic-5-Clause
*/
/* Loopback between P0.0 and P1.14 required */
Copy link
Contributor

Choose a reason for hiding this comment

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

P1.0?

*
* SPDX-License-Identifier: LicenseRef-Nordic-5-Clause
*/
/* Loopback between P0.0 and P1.14 required */
Copy link
Contributor

Choose a reason for hiding this comment

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

P1.0?

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.

3 participants