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

tests: drivers/i2s/i2s_api: Build failed on 96b_argonkey #25180

Closed
erwango opened this issue May 10, 2020 · 2 comments
Closed

tests: drivers/i2s/i2s_api: Build failed on 96b_argonkey #25180

erwango opened this issue May 10, 2020 · 2 comments
Assignees
@erwango
Labels
area: I2S area: Tests Issues related to a particular existing or missing test bug The issue is a bug, or the PR is fixing a bug priority: medium Medium impact/importance bug

Comments

@erwango
Copy link
Member

erwango commented May 10, 2020

Describe the bug
Not much more info for now

To Reproduce
sanitycheck --ninja --warnings-as-errors --runtime-artifact-cleanup -b -p 96b_argonkey -T tests/drivers/i2s

Expected behavior
A clear and concise description of what you expected to happen.

Impact
What impact does this issue have on your progress (e.g., annoyance, showstopper)

Screenshots or console output
If applicable, add a screenshot (drag-and-drop an image), or console logs
(cut-and-paste text and put a code fence (```) before and after, to help
explain the issue.

Environment (please complete the following information):

  • OS: (e.g. Linux, MacOS, Windows)
  • Toolchain (e.g Zephyr SDK, ...)
  • Commit SHA or Version used

Additional context
Add any other context about the problem here.
@erwango erwango added bug The issue is a bug, or the PR is fixing a bug platform: STM32 ST Micro STM32 area: I2S labels May 10, 2020
@erwango erwango self-assigned this May 10, 2020
@carlescufi carlescufi added the area: Tests Issues related to a particular existing or missing test label May 10, 2020
@henrikbrixandersen
Copy link
Member

Potentially fixed by #25163.

erwango added a commit to erwango/zephyr that referenced this issue May 11, 2020
@erwango
tests/drivers: i2s: Fix use of k_sleep
c2a77db 
k_sleep requires k_timeout_t as argument.
Use K_USEC helper when required in i2s driver tests.

Fixes zephyrproject-rtos#25180

Signed-off-by: Erwan Gouriou <erwan.gouriou@linaro.org>
@erwango erwango mentioned this issue May 11, 2020
Closed
@erwango erwango added priority: medium Medium impact/importance bug has-pr and removed platform: STM32 ST Micro STM32 labels May 11, 2020
@erwango
Copy link
Member Author

erwango commented May 12, 2020

Fixed by #25163

@erwango erwango closed this as completed May 12, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@erwango erwango

Labels
area: I2S area: Tests Issues related to a particular existing or missing test bug The issue is a bug, or the PR is fixing a bug priority: medium Medium impact/importance bug
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

tests/drivers: i2s: Fix use of k_sleep erwango/zephyr
3 participants
@henrikbrixandersen @carlescufi @erwango