Skip to content

Add USB examples #62

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
Merged
merged 21 commits into from
Feb 11, 2020
Merged

Add USB examples #62

merged 21 commits into from
Feb 11, 2020

Conversation

mprse
Copy link
Contributor

@mprse mprse commented Jan 13, 2020 •
edited
Loading

This PR adds tested USB examples + description to this repository. Test summary can be found in the table below:

Board USBSerial USBAudio square wave USBAudio loopback USBAudio wav audio player USBAudio play sound USBHID USBCDC USBCDC ECM USBMSD SD USBMSD heap USBMIDI USBMIDI take me out to the ballgame example USBMouse absolute USBMouse relative USBMouseJoystick USBKeyboard USBMouseKeyboard
NUCLEO_F429ZI OK OK OK Not Supported OK OK OK OK Not Supported OK OK OK OK OK Don't have joystick OK OK
K64F OK OK OK OK OK OK OK OK OK OK OK OK OK OK Don't have joystick OK OK
DISCO_L475VG_IOT01A OK OK OK Not Supported OK OK OK OK Not Supported OK OK OK OK OK Don't have joystick OK ok
NUCLEO_F411RE Not Supported Not Supported Not Supported Not Supported Not Supported Not Supported Not Supported Not Supported Not Supported Not Supported Not Supported Not Supported Not Supported Not Supported Don't have joystick Not Supported Not Supported
NRF52840_DK OK Doesn't work Doesn't work Not Supported Doesn't work OK OK OK Not Supported Not Supported Not Supported Not Supported Not Supported Not Supported Don't have joystick Not Supported Not Supported
NUCLEO_F303RE Not Supported Not Supported Not Supported Not Supported Not Supported Not Supported Not Supported Not Supported Not Supported OK OK OK OK OK Don't have joystick OK OK
LPC55S69_NS Not Supported Not Supported Not Supported Not Supported Not Supported Not Supported Not Supported Not Supported Not Supported Not Supported Not Supported Not Supported Not Supported Not Supported Don't have joystick Not Supported Not Supported
NUCLEO_L073RZ Not Supported Not Supported Not Supported Not Supported Not Supported Not Supported Not Supported Not Supported Not Supported Not Supported Not Supported Not Supported Not Supported Not Supported Don't have joystick Not Supported Not Supported

jamesbeyond
jamesbeyond approved these changes Jan 14, 2020
View reviewed changes
Copy link
Contributor

@jamesbeyond jamesbeyond left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, thanks @mprse

@jamesbeyond
Copy link
Contributor

when possible, could you attached PR of updating the link in https://github.com/ARMmbed/mbed-os-5-docs as well

@mprse mprse force-pushed the usb_examples branch 2 times, most recently from 601341e to b6fb478 Compare January 15, 2020 14:09
@mprse mprse force-pushed the usb_examples branch 3 times, most recently from e77817d to 99d16e6 Compare January 23, 2020 12:47
@mprse
Add fix for USB CDC ECM example
9573498 
Additional delay is required to setup usb device before starting transmission.
@mprse mprse changed the title Add USB examples with - WIP Add USB examples Jan 30, 2020
@mprse
Copy link
Contributor Author

mprse commented Jan 30, 2020

This one should be ready to go.
Some examples have been fixed (updated deprecated functions, fixed minor issues). Detailed status can be found in the table provided in the description.

USBAudio examples look good but don't work on NRF52840_DK. The example works on other boards, so probably further investigation of USBAudio driver for NRF is required.

jamesbeyond
jamesbeyond approved these changes Feb 3, 2020
View reviewed changes
Copy link
Contributor

@jamesbeyond jamesbeyond left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good! well done!

@jamesbeyond
Copy link
Contributor

jamesbeyond commented Feb 3, 2020 •
edited
Loading

BTW, could you update the code links in https://github.com/ARMmbed/mbed-os-5-docs, and pointed them to here, and links the PR in here @mprse

@mprse
Copy link
Contributor Author

mprse commented Feb 3, 2020

BTW, could you update the code links in https://github.com/ARMmbed/mbed-os-5-docs, and pointed them to here, and links the PR in here @mprse

Yes, but I thought that to do that first this one needs to be merged. So I'm able to access code/description from this repo.

@AnotherButler AnotherButler mentioned this pull request Feb 6, 2020
Merged
@AnotherButler
Copy link
Contributor

Docs PRs up: ARMmbed/mbed-os-5-docs#1224, ARMmbed/mbed-os-5-docs#1225 and https://github.com/ARMmbed/mbed-os-5-docs/pull/1226/files

@jamesbeyond jamesbeyond merged commit bae9a06 into ARMmbed:master Feb 11, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@jamesbeyond jamesbeyond jamesbeyond approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@mprse @jamesbeyond @AnotherButler