Skip to content
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.

Sign up for GitHub

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

Jump to bottom

drivers: Extend coverage for i2s driver #71083

Merged
merged 1 commit into from
Apr 17, 2024
Merged

drivers: Extend coverage for i2s driver #71083

merged 1 commit into from
Apr 17, 2024

Conversation

nordic-bami
Copy link
Contributor

Add more tests cases in the i2s driver error management area
Signed-off-by: Bartosz Miller bartosz.miller@nordicsemi.no

@zephyrbot zephyrbot requested a review from anangl April 4, 2024 11:47
@github-actions GitHub Actions
Copy link

github-actions bot commented Apr 4, 2024

Hello @nordic-bami, 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. 😊

@nordic-bami nordic-bami force-pushed the extend_i2s_driver_coverage branch from 786452b to aba743a Compare April 4, 2024 11:49
anangl
anangl requested changes Apr 4, 2024
View reviewed changes
@anangl anangl requested a review from tmon-nordic April 4, 2024 14:14
@nordic-bami nordic-bami force-pushed the extend_i2s_driver_coverage branch from aba743a to cbe9536 Compare April 5, 2024 06:53
@nordic-bami nordic-bami requested a review from anangl April 8, 2024 05:59
@nordic-bami nordic-bami force-pushed the extend_i2s_driver_coverage branch from cbe9536 to d83baa3 Compare April 8, 2024 12:35
@tmon-nordic tmon-nordic requested review from teburd and andyross April 9, 2024 06:42
@nordic-bami nordic-bami force-pushed the extend_i2s_driver_coverage branch from d83baa3 to 9005418 Compare April 9, 2024 07:18
@nordic-bami nordic-bami requested a review from tmon-nordic April 9, 2024 07:19
@nordic-bami nordic-bami force-pushed the extend_i2s_driver_coverage branch 2 times, most recently from 1dc8fc6 to 7d64ab7 Compare April 10, 2024 06:00
@nordic-bami nordic-bami force-pushed the extend_i2s_driver_coverage branch 3 times, most recently from eafd9d5 to e27d5fc Compare April 10, 2024 12:09
@nordic-bami nordic-bami requested a review from anangl April 10, 2024 12:13
anangl
anangl previously approved these changes Apr 10, 2024
View reviewed changes
@nordic-bami nordic-bami force-pushed the extend_i2s_driver_coverage branch from e27d5fc to ff947b4 Compare April 10, 2024 12:42
nordic-bami added a commit to nordic-bami/sdk-zephyr that referenced this pull request Apr 11, 2024
@nordic-bami
[nrf fromlist] tests: Extend tests coverage for I2S driver
5e53629 
Upstream PR: zephyrproject-rtos/zephyr#71083

Add more tests cases in the i2s driver error management area
Signed-off-by: Bartosz Miller <bartosz.miller@nordicsemi.no>
@nordic-bami nordic-bami mentioned this pull request Apr 11, 2024
Closed
@nordic-bami
Copy link
Contributor Author

Please review.

@nordic-bami nordic-bami force-pushed the extend_i2s_driver_coverage branch from ff947b4 to 87ce5fb Compare April 16, 2024 07:12
@nordic-bami
drivers: Extend coverage for i2s driver
6365f0e 
Add more tests cases in the i2s driver error management area
Signed-off-by: Bartosz Miller <bartosz.miller@nordicsemi.no>
@nordic-bami nordic-bami force-pushed the extend_i2s_driver_coverage branch from 87ce5fb to 6365f0e Compare April 17, 2024 06:07
@henrikbrixandersen henrikbrixandersen merged commit 32dd4ec into zephyrproject-rtos:main Apr 17, 2024
17 checks passed
@nordic-bami nordic-bami deleted the extend_i2s_driver_coverage branch April 30, 2024 08:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@henrikbrixandersen henrikbrixandersen henrikbrixandersen approved these changes

@anangl anangl anangl approved these changes

@nordic-segl nordic-segl nordic-segl approved these changes

@nordic-piks nordic-piks nordic-piks approved these changes

@tmon-nordic tmon-nordic tmon-nordic approved these changes

@teburd teburd Awaiting requested review from teburd

@andyross andyross Awaiting requested review from andyross

Assignees

@anangl anangl

Labels
area: I2S
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
7 participants
@nordic-bami @henrikbrixandersen @anangl @nordic-segl @nordic-piks @tmon-nordic @zephyrbot