Convert mbed-usb target, enable mbed-usb tests (#49)

* Convert mbed-usb target, enable mbed-usb tests

* Fix ByteBuffer compile error

* Add missing requirements, fix some pyserial issues

* Move CDC_ECM to its own target since it needs RTOS

Author: Jamie Smith

CMakeLists.txt

@@ -257,7 +257,6 @@ add_subdirectory(connectivity)
 
 # The directories below contain optional target libraries
 add_subdirectory(drivers/device_key EXCLUDE_FROM_ALL)
-add_subdirectory(drivers/usb EXCLUDE_FROM_ALL)
 add_subdirectory(features EXCLUDE_FROM_ALL)
 add_subdirectory(cmsis/CMSIS_5/CMSIS/RTOS2 EXCLUDE_FROM_ALL)
 add_subdirectory(cmsis/device/rtos EXCLUDE_FROM_ALL)

drivers/CMakeLists.txt

@@ -53,3 +53,4 @@ target_sources(mbed-core-sources
         source/Watchdog.cpp
 )
 
+add_subdirectory(usb)

drivers/usb/CMakeLists.txt

@@ -1,17 +1,14 @@
 # Copyright (c) 2020 ARM Limited. All rights reserved.
 # SPDX-License-Identifier: Apache-2.0
 
-add_library(mbed-usb INTERFACE)
+if(MBED_ENABLE_OS_INTERNAL_TESTS)
+    if(MBED_BUILD_GREENTEA_TESTS)
+        add_subdirectory(tests/TESTS)
+    endif()
+endif()
 
-target_include_directories(mbed-usb
-    INTERFACE
-        include
-        include/usb
-        include/usb/internal
-)
-
-target_sources(mbed-usb
-    INTERFACE
+if("DEVICE_USBDEVICE=1" IN_LIST MBED_TARGET_DEFINITIONS)
+    add_library(mbed-usb STATIC EXCLUDE_FROM_ALL
         source/AsyncOp.cpp
         source/ByteBuffer.cpp
         source/EndpointResolver.cpp
@@ -21,15 +18,43 @@ target_sources(mbed-usb
         source/TaskBase.cpp
         source/USBAudio.cpp
         source/USBCDC.cpp
-        source/USBCDC_ECM.cpp
         source/USBDevice.cpp
         source/USBHID.cpp
         source/USBKeyboard.cpp
         source/USBMIDI.cpp
-        source/USBMSD.cpp
         source/USBMouse.cpp
         source/USBMouseKeyboard.cpp
-        source/USBSerial.cpp
-)
+        source/USBSerial.cpp)
+
+    target_include_directories(mbed-usb
+        PUBLIC
+            include
+            include/usb
+            include/usb/internal
+    )
+
+    target_link_libraries(mbed-usb PUBLIC mbed-core-flags)
+
+    # USB Mass Storage Device library is separate because it pulls in a dependency on mbed-storage-blockdevice
+    add_library(mbed-usb-msd STATIC EXCLUDE_FROM_ALL
+        source/msd/USBMSD.cpp)
+
+    target_include_directories(mbed-usb-msd
+        PUBLIC
+            include/usb/msd
+    )
+
+    target_link_libraries(mbed-usb-msd PUBLIC mbed-usb mbed-storage-blockdevice)
+
+    # USB CDC ECM library is separate because it pulls in a dependency on mbed-rtos-flags
+    add_library(mbed-usb-cdc-ecm STATIC EXCLUDE_FROM_ALL
+        source/cdc_ecm/USBCDC_ECM.cpp)
+
+
+    target_include_directories(mbed-usb-cdc-ecm
+        PUBLIC
+            include/usb/cdc_ecm
+    )
 
-target_link_libraries(mbed-usb INTERFACE mbed-storage)
+    target_link_libraries(mbed-usb-cdc-ecm PUBLIC mbed-usb mbed-rtos-flags)
+endif()

drivers/usb/include/usb/USBCDC_ECM.h renamed to drivers/usb/include/usb/cdc_ecm/USBCDC_ECM.h

@@ -0,0 +1 @@
+add_subdirectory(usb_device)

drivers/usb/include/usb/USBMSD.h renamed to drivers/usb/include/usb/msd/USBMSD.h

@@ -27,6 +27,12 @@
 import six
 import mbed_host_tests
 
+# Pyserial 3.5 has a compatibility breaking capitalization change :((
+try:
+    from serial import portNotOpenError as PortNotOpenError
+except ImportError:
+    from serial import PortNotOpenError as PortNotOpenError
+
 
 MSG_KEY_DEVICE_READY = 'ready'
 MSG_KEY_SERIAL_NUMBER = 'usb_dev_sn'
@@ -130,7 +136,11 @@ def __init__(self):
 
     def port_open_wait(self):
         """Open the serial and wait until it's closed by the device."""
-        mbed_serial = serial.Serial(dsrdtr=False)
+
+        # Note: Need to set dsrdtr on open to true to avoid exception on Linux
+        # https://github.com/pyserial/pyserial/issues/67
+        mbed_serial = serial.Serial(dsrdtr=True)
+
         mbed_serial.dtr = False
         try:
             mbed_serial.port = retry_fun_call(
@@ -148,12 +158,12 @@ def port_open_wait(self):
         mbed_serial.dtr = True
         try:
             mbed_serial.read()  # wait until closed
-        except (serial.portNotOpenError, serial.SerialException):
+        except (PortNotOpenError, serial.SerialException):
             pass
 
     def port_open_close(self):
         """Open the serial and close it with a delay."""
-        mbed_serial = serial.Serial(timeout=0.5, write_timeout=0.1, dsrdtr=False)
+        mbed_serial = serial.Serial(timeout=0.5, write_timeout=0.1, dsrdtr=True)
         mbed_serial.dtr = False
         try:
             mbed_serial.port = retry_fun_call(
@@ -179,7 +189,7 @@ def send_data_sequence(self, chunk_size=1):
         chunk_size defines the size of data sent in each write operation.
         The input buffer content is discarded.
         """
-        mbed_serial = serial.Serial(write_timeout=0.1, dsrdtr=False)
+        mbed_serial = serial.Serial(write_timeout=0.1, dsrdtr=True)
         try:
             mbed_serial.port = retry_fun_call(
                 fun=functools.partial(self.get_usb_serial_name, self.dut_usb_dev_sn),  # pylint: disable=not-callable
@@ -214,7 +224,7 @@ def send_data_sequence(self, chunk_size=1):
 
     def loopback(self):
         """Open the serial and send back every byte received."""
-        mbed_serial = serial.Serial(timeout=0.5, write_timeout=0.1, dsrdtr=False)
+        mbed_serial = serial.Serial(timeout=0.5, write_timeout=0.1, dsrdtr=True)
         mbed_serial.dtr = False
         try:
             mbed_serial.port = retry_fun_call(

drivers/usb/source/USBCDC_ECM.cpp renamed to drivers/usb/source/cdc_ecm/USBCDC_ECM.cpp

@@ -0,0 +1,4 @@
+add_subdirectory(basic)
+add_subdirectory(hid)
+add_subdirectory(msd)
+add_subdirectory(serial)