sam3u2c: Use CMSIS-DAP v2 interface for sam3u2c_if project

Disabling CMSIS-DAP v1 interface for improved performance

Author: mbrossard

docs/hic/sam3u2c.md

@@ -4,7 +4,7 @@ Based on ATSAM3U2C chip ([Datasheet](https://ww1.microchip.com/downloads/en/Devi
 - Cortex-M3 96 Mhz
 - 128 KB Flash
 - 36 KB RAM
-- High-speed USB 2.0 device controller: up to 7 bi-directional endpoints including EP0 (*)
+- High-speed USB 2.0 device controller: up to 7 endpoints including EP0 (*)
 - LQFP100 packaging
 
 (*) "up to 7 bidirectional Endpoints" (source: Datasheet](https://ww1.microchip.com/downloads/en/DeviceDoc/Atmel-6430-32-bit-Cortex-M3-Microcontroller-SAM3U4-SAM3U2-SAM3U1_Datasheet.pdf))

projects.yaml

@@ -215,6 +215,8 @@ projects:
     sam3u2c_if:
         - *module_if
         - *module_hic_sam3u2c
+        - records/usb/usb-bulk.yaml
+        - records/board/sam3u2c_if.yaml
         - records/family/all_family.yaml
     stm32f072x8_if:
         - *module_if

records/board/sam3u2c_if.yaml

@@ -0,0 +1,5 @@
+common:
+    macros:
+        - HID_ENDPOINT_DISABLE
+        - MSC_ENDPOINT_DISABLE
+        - DRAG_N_DROP_DISABLE

source/hic_hal/atmel/sam3u2c/DAP_config.h

@@ -83,10 +83,10 @@ This information includes:
 /// This configuration settings is used to optimize the communication performance with the
 /// debugger and depends on the USB peripheral. Typical vales are 64 for Full-speed USB HID or WinUSB,
 /// 1024 for High-speed USB HID and 512 for High-speed USB WinUSB.
-#ifndef HID_ENDPOINT            //HID end points currently set limits to 64
-#define DAP_PACKET_SIZE         512              ///< Specifies Packet Size in bytes.
-#else
+#if defined(HID_ENDPOINT) && !defined(HID_ENDPOINT_DISABLE) // HID end points currently set limits to 64
 #define DAP_PACKET_SIZE         64              ///< Specifies Packet Size in bytes.
+#else
+#define DAP_PACKET_SIZE         512             ///< Specifies Packet Size in bytes.
 #endif
 
 /// Maximum Package Buffers for Command and Response data.

source/hic_hal/atmel/sam3u2c/usb_config.c

@@ -87,6 +87,46 @@
 #endif
 #endif
 
+#if (BULK_ENDPOINT)
+/* SAM3UC only has 7 endpoints, and as a consequence BULK_ENDPOINT cannot
+ * be enabled at the same time as CDC_ENDPOINT and MSC_ENDPOINT.
+ */
+#if (CDC_ENDPOINT && MSC_ENDPOINT)
+#error "SAM3U2C does not have enough endpoints to support this configuration"
+#endif
+#if (CDC_ENDPOINT)
+#define USBD_HID_EP_INTOUT          0
+#define USBD_BULK_EP_BULKIN         1
+#define USBD_BULK_EP_BULKOUT        2
+#define USBD_HID_EP_INTIN           3
+#define USBD_CDC_ACM_EP_INTIN       4
+#define USBD_CDC_ACM_EP_BULKOUT     5
+#define USBD_CDC_ACM_EP_BULKIN      6
+#define USBD_MSC_EP_BULKIN          7 /* Unused */
+#define USBD_MSC_EP_BULKOUT         7 /* Unused */
+#else // !(CDC_ENDPOINT)
+#define USBD_MSC_EP_BULKIN          1
+#define USBD_MSC_EP_BULKOUT         2
+#define USBD_HID_EP_INTIN           3
+#define USBD_HID_EP_INTOUT          4
+#define USBD_BULK_EP_BULKOUT        5
+#define USBD_BULK_EP_BULKIN         6
+#define USBD_CDC_ACM_EP_INTIN       7 /* Unused */
+#define USBD_CDC_ACM_EP_BULKIN      7 /* Unused */
+#define USBD_CDC_ACM_EP_BULKOUT     7 /* Unused */
+#endif
+#else // !(BULK_ENDPOINT)
+#define USBD_HID_EP_INTOUT          0
+#define USBD_MSC_EP_BULKIN          1
+#define USBD_MSC_EP_BULKOUT         2
+#define USBD_HID_EP_INTIN           3
+#define USBD_CDC_ACM_EP_INTIN       4
+#define USBD_CDC_ACM_EP_BULKOUT     5
+#define USBD_CDC_ACM_EP_BULKIN      6
+#define USBD_BULK_EP_BULKIN         7 /* Unused */
+#define USBD_BULK_EP_BULKOUT        7 /* Unused */
+#endif
+
 // <e> USB Device
 //   <i> Enable the USB Device functionality
 #define USBD_ENABLE                 1
@@ -209,9 +249,7 @@
 //     </e>
 
 #define USBD_HID_ENABLE             HID_ENDPOINT
-#define USBD_HID_EP_INTIN           3
 #define USBD_HID_EP_INTIN_STACK     0
-#define USBD_HID_EP_INTOUT          0
 #define USBD_HID_EP_INTOUT_STACK    0
 #define USBD_HID_WMAXPACKETSIZE     64
 #define USBD_HID_BINTERVAL          1
@@ -258,9 +296,7 @@
 //     </e>
 
 #define USBD_MSC_ENABLE             MSC_ENDPOINT
-#define USBD_MSC_EP_BULKIN          1
 #define USBD_MSC_EP_BULKIN_STACK    0
-#define USBD_MSC_EP_BULKOUT         2
 #define USBD_MSC_EP_BULKOUT_STACK   0
 #define USBD_MSC_WMAXPACKETSIZE     64
 #define USBD_MSC_HS_ENABLE          1
@@ -374,16 +410,13 @@
 //     </e>
 
 #define USBD_CDC_ACM_ENABLE             CDC_ENDPOINT
-#define USBD_CDC_ACM_EP_INTIN           4
 #define USBD_CDC_ACM_EP_INTIN_STACK     0
 #define USBD_CDC_ACM_WMAXPACKETSIZE     16
 #define USBD_CDC_ACM_BINTERVAL          32
 #define USBD_CDC_ACM_HS_ENABLE          1
 #define USBD_CDC_ACM_HS_WMAXPACKETSIZE  64
 #define USBD_CDC_ACM_HS_BINTERVAL       2
-#define USBD_CDC_ACM_EP_BULKIN          6
 #define USBD_CDC_ACM_EP_BULKIN_STACK    0
-#define USBD_CDC_ACM_EP_BULKOUT         5
 #define USBD_CDC_ACM_EP_BULKOUT_STACK   0
 #define USBD_CDC_ACM_WMAXPACKETSIZE1    64
 #define USBD_CDC_ACM_HS_ENABLE1         1
@@ -431,20 +464,6 @@
 // </e>
 
 #define USBD_BULK_ENABLE             BULK_ENDPOINT
-// We don't have enough endpoints for BULK unless MSC or CDC are disabled.
-// MSC is the best choice, since this is only useful in the fixed target IF config and we can still upload firmware via DAP.
-// We'll still want MSC for the BL though, but in that case BULK isn't useful anyway.
-#if USBD_MSC_ENABLE == 0
-#define USBD_BULK_EP_BULKIN          1 // Use the MSC endpoints for BULK
-#define USBD_BULK_EP_BULKOUT         2
-#elif USBD_CDC_ACM_ENABLE == 0
-#define USBD_BULK_EP_BULKIN          5 // Use the CDC endpoints for BULK
-#define USBD_BULK_EP_BULKOUT         6
-#else
-#define USBD_BULK_EP_BULKIN          7 // Use non-existent endpoints to force an error
-#define USBD_BULK_EP_BULKOUT         8 // if BULK is enabled but MSC + CDC are still enabled
-#endif
-
 #define USBD_BULK_WMAXPACKETSIZE     64
 #define USBD_BULK_HS_ENABLE          1
 #define USBD_BULK_HS_WMAXPACKETSIZE  512