Skip to content

[nrf fromlist] tests: boards: nrf: i2s: i2s_divider: Confirm CLK value #2918

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.

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

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

nordic-segl
Copy link
Contributor

Add test which confirms correct clock divider selection.

Frame rate selected by the user may not be obtained due to PCLK clock value and finite number of clock divider value.
In such case, driver shall select divider that results in frame clock rate closest to the requested value.
There was a bug that was discarding perfect match divider.

Upstream PR #: TBD

@nordic-segl
Copy link
Contributor Author

Works in CI:

09:38:43  INFO    -  4/10 nrf54l15dk/nrf54l15/cpuapp tests/drivers/i2s/i2s_speed/drivers.i2s.speed.gpio_loopback.clk_divider PASSED (device: 1057701123, 3.948s <zephyr>)
09:38:50  INFO    -  9/10 nrf52840dk/nrf52840       tests/drivers/i2s/i2s_speed/drivers.i2s.speed.gpio_loopback.clk_divider PASSED (device: 683082178, 5.114s <zephyr>)
09:39:08  INFO    - 10/10 nrf5340dk/nrf5340/cpuapp  tests/drivers/i2s/i2s_speed/drivers.i2s.speed.gpio_loopback.clk_divider PASSED (device: 1050070915, 3.559s <zephyr>)

@nordic-segl nordic-segl force-pushed the NRFX-7607_I2S-find-correct-clock branch from 2ff20f7 to 33706db Compare May 28, 2025 08:33
@nordic-segl nordic-segl marked this pull request as ready for review May 28, 2025 08:36
@nordic-segl nordic-segl requested review from kl-cruz, anangl and a team May 28, 2025 08:37
…der value

Frame rate selected by the user may not be obtained due to
PCLK frequency and finite number of clock divider values.
In such case, driver shall select divider that results in
frame clock rate closest to the requested value.
There was a bug that was discarding perfect match divider.

Add test which confirms correct clock divider selection.

Upstream PR #: 90720

Signed-off-by: Sebastian Głąb <sebastian.glab@nordicsemi.no>
@nordic-segl nordic-segl force-pushed the NRFX-7607_I2S-find-correct-clock branch from 33706db to f97b619 Compare May 30, 2025 09:24
@nordic-segl nordic-segl changed the title [nrf fromlist] tests: drivers: i2s: i2s_speed: Confirm CLK value [nrf fromlist] tests: boards: nrf: i2s: i2s_divider: Confirm CLK value May 30, 2025
@nordic-segl
Copy link
Contributor Author

Changes in upstream were requested. PR was significantly reworked.

Copy link

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants