Bump tinyusb to include MAX3421E host startup-hang fix (#3748)

ports/raspberrypi/boards/pimoroni_pico_dv_base_w/mpconfigboard.mk

@@ -20,6 +20,10 @@ CIRCUITPY_WIFI = 1
 
 CIRCUITPY_PICODVI = 1
 
+# No room: this board fills FLASH_FIRMWARE. Frees ~9kB. Not the board's I2S
+# line-out/HDMI audio, which audiobusio still provides.
+CIRCUITPY_USB_AUDIO = 0
+
 CFLAGS += \
     -DCYW43_PIN_WL_DYNAMIC=0 \
 	-DCYW43_DEFAULT_PIN_WL_HOST_WAKE=24 \

ports/stm/Makefile

@@ -205,6 +205,7 @@ endif
 ifneq ($(CIRCUITPY_USB),0)
 	ifeq ($(MCU_VARIANT),$(filter $(MCU_VARIANT),STM32L433xx))
 		SRC_C += lib/tinyusb/src/portable/st/stm32_fsdev/dcd_stm32_fsdev.c
+		SRC_C += lib/tinyusb/src/portable/st/stm32_fsdev/fsdev_common.c
 	else
 		SRC_C += lib/tinyusb/src/portable/synopsys/dwc2/dcd_dwc2.c
 		SRC_C += lib/tinyusb/src/portable/synopsys/dwc2/dwc2_common.c

shared-module/usb/core/Device.c

@@ -135,6 +135,7 @@ static bool _wait_for_callback(void) {
     switch (result) {
         case XFER_RESULT_SUCCESS:
             return true;
+        case XFER_RESULT_ABORTED:
         case XFER_RESULT_FAILED:
             mp_raise_usb_core_USBError(NULL);
             break;
@@ -201,6 +202,7 @@ static size_t _handle_timed_transfer_callback(tuh_xfer_t *xfer, mp_int_t timeout
     switch (result) {
         case XFER_RESULT_SUCCESS:
             return _actual_len;
+        case XFER_RESULT_ABORTED:
         case XFER_RESULT_FAILED:
             mp_raise_usb_core_USBError(NULL);
             break;

shared-module/usb_audio/usb_audio_descriptors.h

@@ -45,7 +45,7 @@ size_t usb_audio_descriptor_length(void);
 #define USB_AUDIO_ISO_EP_NUM (0)
 #endif
 
-// Fixed UAC2 entity IDs baked into TUD_AUDIO_MIC_ONE_CH_DESCRIPTOR and the
+// Fixed UAC2 entity IDs baked into TUD_AUDIO20_MIC_ONE_CH_DESCRIPTOR and the
 // hand-rolled speaker descriptor (USB_AUDIO_SPEAKER_DESCRIPTOR in __init__.c).
 // The speaker reuses the same IDs as the mic; only the terminal roles reverse
 // (input terminal = USB streaming, output terminal = desktop speaker).
@@ -69,26 +69,26 @@ size_t usb_audio_descriptor_length(void);
 #define USB_AUDIO_HS_ENTITY_MIC_OUTPUT_TERMINAL (0x07)  // USB streaming out to host
 
 // Length of the no-feedback mono speaker descriptor. It uses the same set of
-// sub-descriptors as TUD_AUDIO_MIC_ONE_CH_DESCRIPTOR (one isochronous data
+// sub-descriptors as TUD_AUDIO20_MIC_ONE_CH_DESCRIPTOR (one isochronous data
 // endpoint, no feedback endpoint), so this is identical to
-// TUD_AUDIO_MIC_ONE_CH_DESC_LEN -- but spell it out independently so the two
+// TUD_AUDIO20_MIC_ONE_CH_DESC_LEN -- but spell it out independently so the two
 // can diverge later (e.g. stereo) without silently mis-sizing the descriptor.
-// These TUD_AUDIO_DESC_*_LEN macros come from TinyUSB's usbd.h; this expression
+// These TUD_AUDIO20_DESC_*_LEN macros come from TinyUSB's usbd.h; this expression
 // is only expanded where that header is already included (never at the point
 // tusb_config.h includes us), so the header stays dependency-free.
-#define USB_AUDIO_SPEAKER_DESC_LEN (TUD_AUDIO_DESC_IAD_LEN \
-    + TUD_AUDIO_DESC_STD_AC_LEN \
-    + TUD_AUDIO_DESC_CS_AC_LEN \
-    + TUD_AUDIO_DESC_CLK_SRC_LEN \
-    + TUD_AUDIO_DESC_INPUT_TERM_LEN \
-    + TUD_AUDIO_DESC_OUTPUT_TERM_LEN \
-    + TUD_AUDIO_DESC_FEATURE_UNIT_ONE_CHANNEL_LEN \
-    + TUD_AUDIO_DESC_STD_AS_INT_LEN \
-    + TUD_AUDIO_DESC_STD_AS_INT_LEN \
-    + TUD_AUDIO_DESC_CS_AS_INT_LEN \
-    + TUD_AUDIO_DESC_TYPE_I_FORMAT_LEN \
-    + TUD_AUDIO_DESC_STD_AS_ISO_EP_LEN \
-    + TUD_AUDIO_DESC_CS_AS_ISO_EP_LEN)
+#define USB_AUDIO_SPEAKER_DESC_LEN (TUD_AUDIO20_DESC_IAD_LEN \
+    + TUD_AUDIO20_DESC_STD_AC_LEN \
+    + TUD_AUDIO20_DESC_CS_AC_LEN \
+    + TUD_AUDIO20_DESC_CLK_SRC_LEN \
+    + TUD_AUDIO20_DESC_INPUT_TERM_LEN \
+    + TUD_AUDIO20_DESC_OUTPUT_TERM_LEN \
+    + TUD_AUDIO20_DESC_FEATURE_UNIT_LEN(1) \
+    + TUD_AUDIO20_DESC_STD_AS_LEN \
+    + TUD_AUDIO20_DESC_STD_AS_LEN \
+    + TUD_AUDIO20_DESC_CS_AS_INT_LEN \
+    + TUD_AUDIO20_DESC_TYPE_I_FORMAT_LEN \
+    + TUD_AUDIO20_DESC_STD_AS_ISO_EP_LEN \
+    + TUD_AUDIO20_DESC_CS_AS_ISO_EP_LEN)
 
 // Length of the combined headset descriptor (microphone + speaker both enabled): one IAD
 // wrapping a single AudioControl interface plus two AudioStreaming interfaces
@@ -99,29 +99,29 @@ size_t usb_audio_descriptor_length(void);
 // single-direction descriptors). See USB_AUDIO_HEADSET_DESCRIPTOR in __init__.c.
 // Expanded only where TinyUSB's usbd.h is already included (never at the point
 // tusb_config.h includes us), so this header stays dependency-free.
-#define USB_AUDIO_HEADSET_DESC_LEN (TUD_AUDIO_DESC_IAD_LEN \
-    + TUD_AUDIO_DESC_STD_AC_LEN \
-    + TUD_AUDIO_DESC_CS_AC_LEN \
-    + TUD_AUDIO_DESC_CLK_SRC_LEN \
+#define USB_AUDIO_HEADSET_DESC_LEN (TUD_AUDIO20_DESC_IAD_LEN \
+    + TUD_AUDIO20_DESC_STD_AC_LEN \
+    + TUD_AUDIO20_DESC_CS_AC_LEN \
+    + TUD_AUDIO20_DESC_CLK_SRC_LEN \
     /* speaker chain: USB-streaming input terminal -> feature unit -> speaker */ \
-    + TUD_AUDIO_DESC_INPUT_TERM_LEN \
-    + TUD_AUDIO_DESC_FEATURE_UNIT_ONE_CHANNEL_LEN \
-    + TUD_AUDIO_DESC_OUTPUT_TERM_LEN \
+    + TUD_AUDIO20_DESC_INPUT_TERM_LEN \
+    + TUD_AUDIO20_DESC_FEATURE_UNIT_LEN(1) \
+    + TUD_AUDIO20_DESC_OUTPUT_TERM_LEN \
     /* mic chain: microphone input terminal -> feature unit -> USB-streaming out */ \
-    + TUD_AUDIO_DESC_INPUT_TERM_LEN \
-    + TUD_AUDIO_DESC_FEATURE_UNIT_ONE_CHANNEL_LEN \
-    + TUD_AUDIO_DESC_OUTPUT_TERM_LEN \
+    + TUD_AUDIO20_DESC_INPUT_TERM_LEN \
+    + TUD_AUDIO20_DESC_FEATURE_UNIT_LEN(1) \
+    + TUD_AUDIO20_DESC_OUTPUT_TERM_LEN \
     /* speaker AudioStreaming interface (alt 0 + alt 1 with OUT endpoint) */ \
-    + TUD_AUDIO_DESC_STD_AS_INT_LEN \
-    + TUD_AUDIO_DESC_STD_AS_INT_LEN \
-    + TUD_AUDIO_DESC_CS_AS_INT_LEN \
-    + TUD_AUDIO_DESC_TYPE_I_FORMAT_LEN \
-    + TUD_AUDIO_DESC_STD_AS_ISO_EP_LEN \
-    + TUD_AUDIO_DESC_CS_AS_ISO_EP_LEN \
+    + TUD_AUDIO20_DESC_STD_AS_LEN \
+    + TUD_AUDIO20_DESC_STD_AS_LEN \
+    + TUD_AUDIO20_DESC_CS_AS_INT_LEN \
+    + TUD_AUDIO20_DESC_TYPE_I_FORMAT_LEN \
+    + TUD_AUDIO20_DESC_STD_AS_ISO_EP_LEN \
+    + TUD_AUDIO20_DESC_CS_AS_ISO_EP_LEN \
     /* mic AudioStreaming interface (alt 0 + alt 1 with IN endpoint) */ \
-    + TUD_AUDIO_DESC_STD_AS_INT_LEN \
-    + TUD_AUDIO_DESC_STD_AS_INT_LEN \
-    + TUD_AUDIO_DESC_CS_AS_INT_LEN \
-    + TUD_AUDIO_DESC_TYPE_I_FORMAT_LEN \
-    + TUD_AUDIO_DESC_STD_AS_ISO_EP_LEN \
-    + TUD_AUDIO_DESC_CS_AS_ISO_EP_LEN)
+    + TUD_AUDIO20_DESC_STD_AS_LEN \
+    + TUD_AUDIO20_DESC_STD_AS_LEN \
+    + TUD_AUDIO20_DESC_CS_AS_INT_LEN \
+    + TUD_AUDIO20_DESC_TYPE_I_FORMAT_LEN \
+    + TUD_AUDIO20_DESC_STD_AS_ISO_EP_LEN \
+    + TUD_AUDIO20_DESC_CS_AS_ISO_EP_LEN)

supervisor/shared/usb/tusb_config.h

@@ -144,7 +144,7 @@ extern "C" {
 #define CFG_TUD_AUDIO_FUNC_1_N_BYTES_PER_SAMPLE_TX  USB_AUDIO_N_BYTES_PER_SAMPLE
 #define CFG_TUD_AUDIO_FUNC_1_N_CHANNELS_TX          USB_AUDIO_N_CHANNELS
 // wMaxPacketSize, sized for the highest supported sample rate.
-#define CFG_TUD_AUDIO_FUNC_1_EP_IN_SZ_MAX           TUD_AUDIO_EP_SIZE(USB_AUDIO_MAX_SAMPLE_RATE, USB_AUDIO_N_BYTES_PER_SAMPLE, USB_AUDIO_N_CHANNELS)
+#define CFG_TUD_AUDIO_FUNC_1_EP_IN_SZ_MAX           TUD_AUDIO_EP_SIZE(TUD_OPT_HIGH_SPEED, USB_AUDIO_MAX_SAMPLE_RATE, USB_AUDIO_N_BYTES_PER_SAMPLE, USB_AUDIO_N_CHANNELS)
 // Deep software FIFO so the 1 ms refill keeps clear of the underrun floor.
 #define CFG_TUD_AUDIO_FUNC_1_EP_IN_SW_BUF_SZ        (16 * CFG_TUD_AUDIO_FUNC_1_EP_IN_SZ_MAX)
 
@@ -156,7 +156,7 @@ extern "C" {
 #define CFG_TUD_AUDIO_FUNC_1_N_BYTES_PER_SAMPLE_RX  USB_AUDIO_N_BYTES_PER_SAMPLE
 #define CFG_TUD_AUDIO_FUNC_1_N_CHANNELS_RX          USB_AUDIO_N_CHANNELS
 // wMaxPacketSize, sized for the highest supported sample rate.
-#define CFG_TUD_AUDIO_FUNC_1_EP_OUT_SZ_MAX          TUD_AUDIO_EP_SIZE(USB_AUDIO_MAX_SAMPLE_RATE, USB_AUDIO_N_BYTES_PER_SAMPLE, USB_AUDIO_N_CHANNELS)
+#define CFG_TUD_AUDIO_FUNC_1_EP_OUT_SZ_MAX          TUD_AUDIO_EP_SIZE(TUD_OPT_HIGH_SPEED, USB_AUDIO_MAX_SAMPLE_RATE, USB_AUDIO_N_BYTES_PER_SAMPLE, USB_AUDIO_N_CHANNELS)
 // Deep software FIFO so the 1 ms drain keeps clear of the overrun ceiling.
 #define CFG_TUD_AUDIO_FUNC_1_EP_OUT_SW_BUF_SZ       (16 * CFG_TUD_AUDIO_FUNC_1_EP_OUT_SZ_MAX)
 #endif

supervisor/shared/usb/usb_device.c

@@ -156,7 +156,7 @@ bool tud_vendor_control_xfer_cb(uint8_t rhport, uint8_t stage, tusb_control_requ
 // size. Setting CFG_TUD_CDC_RX_BUFSIZE to the endpoint size and then sending
 // any character will prevent ctrl-c from working. Require at least a 64
 // character buffer.
-#if CFG_TUD_CDC_RX_BUFSIZE < CFG_TUD_CDC_EP_BUFSIZE + 64
+#if CFG_TUD_CDC_RX_BUFSIZE < CFG_TUD_CDC_RX_EPSIZE + 64
 #error "CFG_TUD_CDC_RX_BUFSIZE must be 64 bytes bigger than endpoint size."
 #endif
 

supervisor/supervisor.mk

@@ -127,7 +127,6 @@ ifeq ($(CIRCUITPY_TINYUSB),1)
   SRC_SUPERVISOR += \
     lib/tinyusb/src/class/cdc/cdc_device.c \
     lib/tinyusb/src/device/usbd.c \
-    lib/tinyusb/src/device/usbd_control.c \
     supervisor/shared/usb/usb_desc.c \
     supervisor/shared/usb/usb_device.c \