Skip to content

Conversation

erdemsimsek
Copy link

  • Added audio aux ppl support to i2s_nrf_tdm
  • Triggered I2S_TRIGGER_DROP in test_i2s_config_attempt_in_wrong_state function for proper clean-up

@github-actions
Copy link

Hello @erdemsimsek, and thank you very much for your first pull request to the Zephyr project!
Our Continuous Integration pipeline will execute a series of checks on your Pull Request commit messages and code, and you are expected to address any failures by updating the PR. Please take a look at our commit message guidelines to find out how to format your commit messages, and at our contribution workflow to understand how to update your Pull Request. If you haven't already, please make sure to review the project's Contributor Expectations and update (by amending and force-pushing the commits) your pull request if necessary.
If you are stuck or need help please join us on Discord and ask your question there. Additionally, you can escalate the review when applicable. 😊

Erdem Simsek added 2 commits October 22, 2025 10:27
Add support for audio_auxpll clock source as an alternative to audiopll
in the i2s_nrf_tdm driver. This enables TDM functionality on platforms
that use the auxiliary PLL for audio clocking.

- Add audio_auxpll node detection and configuration
- Update clock management to support both audiopll and audio_auxpll
- Add build assertions for supported frequency configurations

Signed-off-by: Erdem Simsek <[email protected]>
Add I2S_TRIGGER_DROP call in the error test to ensure proper cleanup
after stopping the I2S device. This validates that the drop trigger
is properly handled in error conditions and resources are released.

Signed-off-by: Erdem Simsek <[email protected]>
@erdemsimsek erdemsimsek force-pushed the add-audio-aux-pll-support-to-nrf-tdm branch from ac7cfab to 6f5a852 Compare October 22, 2025 09:32
@sonarqubecloud
Copy link

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area: I2S area: Tests Issues related to a particular existing or missing test platform: nRF Nordic nRFx

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants