Skip to content

USB audio callback rx and tx #3339

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

Merged
merged 2 commits into from
Dec 21, 2016
Merged

Conversation

jamike
Copy link
Contributor

@jamike jamike commented Nov 29, 2016

Description

Creation :

  • Call back api in USBAudio device for transmission complete and reception complete.
  • Specific api for filling/reading FIFO is also provided.

These apis allow to early fill/empty ISO endpoint FIFO. These early treaments which enable hardware reception/ transmission avoid to miss any periodic packet in ISO mode.

Status

READY

Migrations

This PR provides additional interfaces that can be optionally used by application.
USB_8 test demonstrates these api. No migration is needed.

@0xc0170
Copy link
Contributor

0xc0170 commented Nov 29, 2016

cc @sg-

@jamike
Copy link
Contributor Author

jamike commented Nov 30, 2016

By the way, it can be noted that this patch works on K64F, the audio loopback USB_8 works while USB_7 audio loopback doesnot work.

@jamike
Copy link
Contributor Author

jamike commented Dec 7, 2016

Is something blocking the review process ?

@0xc0170
Copy link
Contributor

0xc0170 commented Dec 9, 2016

Is something blocking the review process ?

The ball is on our side,e we have to review this extensions (one of the reasons is that this is currently in the unsupported folder). We shall get back to you soon. sorry for the delay.

@sg-
Copy link
Contributor

sg- commented Dec 15, 2016

Seems ok. It should be noted there is no automated testing or statement of quality about the USB code. We'd like to refactor and integrate with mbed OS 5 with testing to support the functionality and make it easier to accept contributions. Anyone interested in helping out please reach out.

@jamike please rebase to kick off testing again. thx.

@adbridge
Copy link
Contributor

/morph test-nightly

@mbed-bot
Copy link

Result: SUCCESS

Your command has finished executing! Here's what you wrote!

/morph test-nightly

Output

mbed Build Number: 1294

All builds and test passed!

@0xc0170 0xc0170 removed the needs: CI label Dec 21, 2016
@0xc0170 0xc0170 merged commit 08509e7 into ARMmbed:master Dec 21, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants