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.

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

K22: Add USBDEVICE support #11284

Merged
merged 1 commit into from Aug 22, 2019

Conversation

@mmahadevan108
Copy link
Contributor

commented Aug 21, 2019

Description

Add USBDevice support. Below is the test result:
| target | platform_name | test suite | result | elapsed_time (sec) | copy_method |
|----------|---------------|--------------------------------|--------|--------------------|-------------|
| K22F-IAR | K22F | mbed-os-tests-usb_device-basic | OK | 58.72 | default |
mbedgt: test suite results: 1 OK
mbedgt: test case report:
| target | platform_name | test suite | test case | passed | failed | result | elapsed_time (sec) |
|----------|---------------|--------------------------------|--------------------------------------------|--------|--------|--------|--------------------|
| K22F-IAR | K22F | mbed-os-tests-usb_device-basic | endpoint test abort | 1 | 0 | OK | 16.43 |
| K22F-IAR | K22F | mbed-os-tests-usb_device-basic | endpoint test data correctness | 1 | 0 | OK | 0.67 |
| K22F-IAR | K22F | mbed-os-tests-usb_device-basic | endpoint test data toggle reset | 1 | 0 | OK | 0.45 |
| K22F-IAR | K22F | mbed-os-tests-usb_device-basic | endpoint test halt | 1 | 0 | OK | 8.94 |
| K22F-IAR | K22F | mbed-os-tests-usb_device-basic | endpoint test parallel transfers | 1 | 0 | OK | 4.42 |
| K22F-IAR | K22F | mbed-os-tests-usb_device-basic | endpoint test parallel transfers ctrl | 1 | 0 | OK | 4.76 |
| K22F-IAR | K22F | mbed-os-tests-usb_device-basic | usb control basic test | 1 | 0 | OK | 0.51 |
| K22F-IAR | K22F | mbed-os-tests-usb_device-basic | usb control sizes test | 1 | 0 | OK | 0.42 |
| K22F-IAR | K22F | mbed-os-tests-usb_device-basic | usb control stall test | 1 | 0 | OK | 0.34 |
| K22F-IAR | K22F | mbed-os-tests-usb_device-basic | usb control stress test | 1 | 0 | OK | 0.49 |
| K22F-IAR | K22F | mbed-os-tests-usb_device-basic | usb device reset test | 1 | 0 | OK | 1.32 |
| K22F-IAR | K22F | mbed-os-tests-usb_device-basic | usb repeated construction destruction test | 1 | 0 | OK | 1.71 |
| K22F-IAR | K22F | mbed-os-tests-usb_device-basic | usb soft reconnection test | 1 | 0 | OK | 1.62 |
mbedgt: test case results: 13 OK

target platform_name test suite result elapsed_time (sec) copy_method
K22F-IAR K22F mbed-os-tests-usb_device-hid OK 23.3 default
mbedgt: test suite results: 1 OK
mbedgt: test case report:
target platform_name test suite test case passed failed
---------- --------------- ------------------------------ ------------------------------------ -------- --------
K22F-IAR K22F mbed-os-tests-usb_device-hid Configuration descriptor, generic 1 0
K22F-IAR K22F mbed-os-tests-usb_device-hid Configuration descriptor, keyboard 1 0
K22F-IAR K22F mbed-os-tests-usb_device-hid Configuration descriptor, mouse 1 0
K22F-IAR K22F mbed-os-tests-usb_device-hid HID class descriptors, generic 1 0
K22F-IAR K22F mbed-os-tests-usb_device-hid HID class descriptors, keyboard 1 0
K22F-IAR K22F mbed-os-tests-usb_device-hid HID class descriptors, mouse 1 0
K22F-IAR K22F mbed-os-tests-usb_device-hid Raw input/output, 1-byte reports 1 0
K22F-IAR K22F mbed-os-tests-usb_device-hid Raw input/output, 20-byte reports 1 0
K22F-IAR K22F mbed-os-tests-usb_device-hid Raw input/output, 64-byte reports 1 0
mbedgt: test case results: 9 OK
target platform_name test suite result elapsed_time (sec) copy_method
K22F-IAR K22F mbed-os-tests-usb_device-serial OK 31.57 default
mbedgt: test suite results: 1 OK
mbedgt: test case report:
target platform_name test suite test case passed failed
---------- --------------- --------------------------------- ---------------------------------- -------- --------
K22F-IAR K22F mbed-os-tests-usb_device-serial CDC RX multiple bytes 1 0
K22F-IAR K22F mbed-os-tests-usb_device-serial CDC RX multiple bytes concurrent 1 0
K22F-IAR K22F mbed-os-tests-usb_device-serial CDC RX single bytes 1 0
K22F-IAR K22F mbed-os-tests-usb_device-serial CDC RX single bytes concurrent 1 0
K22F-IAR K22F mbed-os-tests-usb_device-serial CDC USB reconnect 1 0
K22F-IAR K22F mbed-os-tests-usb_device-serial CDC loopback 1 0
K22F-IAR K22F mbed-os-tests-usb_device-serial Serial USB reconnect 1 0
K22F-IAR K22F mbed-os-tests-usb_device-serial Serial getc 1 0
K22F-IAR K22F mbed-os-tests-usb_device-serial Serial line coding change 1 0
K22F-IAR K22F mbed-os-tests-usb_device-serial Serial printf/scanf 1 0
K22F-IAR K22F mbed-os-tests-usb_device-serial Serial terminal reopen 1 0
mbedgt: test case results: 11 OK

The USB MSD tests uses RAM space to mount a filesystem. K22F does not have enough RAM space to run this test, below are test results when the test was modified to use FlashIAP
| target | platform_name | test suite | result | elapsed_time (sec) | copy_method |
|----------|---------------|------------------------------|--------|--------------------|-------------|
| K22F-IAR | K22F | mbed-os-tests-usb_device-msd | OK | 88.06 | default |
mbedgt: test suite results: 1 OK
mbedgt: test case report:
| target | platform_name | test suite | test case | passed | failed | result | elapsed_time (sec) |
|----------|---------------|------------------------------|-------------------------------------------------|--------|--------|--------|--------------------|
| K22F-IAR | K22F | mbed-os-tests-usb_device-msd | mount/unmount and data test - Heap block device | 1 | 0 | OK | 20.58 |
| K22F-IAR | K22F | mbed-os-tests-usb_device-msd | mount/unmount test - Heap block device | 1 | 0 | OK | 51.01 |
| K22F-IAR | K22F | mbed-os-tests-usb_device-msd | storage initialization | 1 | 0 | OK | 0.74 |
mbedgt: test case results: 3 OK

Pull request type

[ ] Fix
[ ] Refactor
[X] Target update
[ ] Functionality change
[ ] Docs update
[ ] Test update
[ ] Breaking change
K22: Add USBDEVICE support
Signed-off-by: Mahesh Mahadevan <mahesh.mahadevan@nxp.com>
@mmahadevan108

This comment has been minimized.

Copy link
Contributor Author

commented Aug 21, 2019

@ciarmcom

This comment has been minimized.

Copy link
Member

commented Aug 21, 2019

@mmahadevan108, thank you for your changes.
@ARMmbed/mbed-os-maintainers please review.

@ciarmcom ciarmcom requested a review from ARMmbed/mbed-os-maintainers Aug 21, 2019

@0xc0170
Copy link
Member

left a comment

One-liner commit but for future, removing spaces should be in separate commit

@0xc0170 0xc0170 added needs: CI and removed needs: review labels Aug 22, 2019

@0xc0170

This comment has been minimized.

Copy link
Member

commented Aug 22, 2019

CI started

@mbed-ci

This comment has been minimized.

Copy link

commented Aug 22, 2019

Test run: SUCCESS

Summary: 11 of 11 test jobs passed
Build number : 3
Build artifacts

@0xc0170 0xc0170 merged commit 1182d1e into ARMmbed:master Aug 22, 2019

25 checks passed

continuous-integration/jenkins/pr-head This commit looks good
Details
jenkins-ci/build-ARM Success
Details
jenkins-ci/build-GCC_ARM Success
Details
jenkins-ci/build-IAR Success
Details
jenkins-ci/cloud-client-test Success
Details
jenkins-ci/dynamic-memory-usage RTOS ROM(+0 bytes) RAM(+0 bytes)
Details
jenkins-ci/exporter Success
Details
jenkins-ci/greentea-test Success
Details
jenkins-ci/mbed2-build-ARM Success
Details
jenkins-ci/mbed2-build-GCC_ARM Success
Details
jenkins-ci/mbed2-build-IAR Success
Details
jenkins-ci/unittests Success
Details
travis-ci/astyle Success!
Details
travis-ci/docs Success!
Details
travis-ci/doxy-spellcheck Success!
Details
travis-ci/events Success! Runtime is 8562 cycles.
Details
travis-ci/gitattributestest Success!
Details
travis-ci/include_check Success!
Details
travis-ci/licence_check Success!
Details
travis-ci/littlefs Success! Code size is 8464B.
Details
travis-ci/psa-autogen Success!
Details
travis-ci/tools-py2.7 Success!
Details
travis-ci/tools-py3.5 Success!
Details
travis-ci/tools-py3.6 Success!
Details
travis-ci/tools-py3.7 Success!
Details

@mmahadevan108 mmahadevan108 deleted the NXPmicro:K22_Add_USB_Support branch Aug 22, 2019

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
4 participants
You can’t perform that action at this time.