ESP32S3 - HID USBHID SendReport error and USB Replug

[RichardsATcn]

Board

ESP32-S3-WROOM-1 Moudle

Device Description

ESP32-S3-WROOM-1

Hardware Configuration

only use USB ；

Version

v2.0.3

IDE Name

Arduino 1.8.19

Operating System

Windows 10

Flash frequency

QIO 80

PSRAM enabled

no

Upload speed

115200

Description

ESP32-S3-USB.zip

src.zip

USB device will appear error after using for a while.

If the CDC serial port is enabled, it will be easier to send at fixed time.

Devices will be re enumerated repeatedly

The error code segment is as follows

` bool res = ready();
if (!res)
{
log_e("not ready");
}
else
{
res = tud_hid_n_report(0, id, data, len);
if (!res)
{
log_e("report %u failed", id);
}
else
{
xSemaphoreTake(tinyusb_hid_device_input_sem, 0);
if (xSemaphoreTake(tinyusb_hid_device_input_sem, timeout_ms / portTICK_PERIOD_MS) != pdTRUE)
{
log_e("report %u wait failed", id);
res = false;
}
}
}
xSemaphoreGive(tinyusb_hid_device_input_mutex);

//Added code snippet
if (tinyusb_hid_device_input_sem)
{
xSemaphoreGive(tinyusb_hid_device_input_sem);
}
log_d("report %u wait over %s", id, res == true ? "true" : "false");
//Added code snippet

return res;`

usb设备使用一会儿会出现错误。
如果cdc串口开启定时发送时候会更容易出现。
设备会反复的 重新枚举

Sketch

//Added code snippet
    if (tinyusb_hid_device_input_sem)
    {
        xSemaphoreGive(tinyusb_hid_device_input_sem);
    }
    log_d("report %u wait over %s", id, res == true ? "true" : "false");
//Added code snippet

Debug Message

17:27:54.138 -> [ 34283][E][USBHID.cpp:410] SendReport(): report 3 wait failed
17:27:54.138 -> [ 34284][D][USBHID.cpp:420] SendReport(): report 3 wait over false
17:27:54.234 -> USB PLUGGED
17:27:54.330 -> [ 34475][E][USBHID.cpp:410] SendReport(): report 3 wait failed

Other Steps to Reproduce

No response

I have checked existing issues, online documentation and the Troubleshooting Guide

• I confirm I have checked existing issues, online documentation and Troubleshooting guide.

[VojtechBartoska]

This will be triage by @SuGlider, in next 2 weeks we will specify next steps.

[SuGlider]

@RichardsATcn - It seems that the code you posted is some modified version of "official" USBHID.cpp - Not sure how to help you. Please confirm.

Can you try the sketch using Arduino Core 2.0.4?
Could you please post a Arduino Sketch that reproduces the issue, using only official Arduino Core?

Thanks.

[VojtechBartoska]

@RichardsATcn any updates?

[RichardsATcn]

I will try with sdk 2.0.6，thank you。

[hoantv]

same issue with esps3 wroom 1U. Use lastest version 2.0.7
Just use gamepad example, PC doesnt recognize gamepad after sendReport command

[ 7621][E][USBHID.cpp�ESP-ROM:esp32s3-20210327
Build:Mar 27 2021
rst:0x1 (POWERON),boot:0x8 (SPI_FAST_FLASH_BOOT)
SPIWP:0xee
mode:DIO, clock div:1
load:0x3fce3808,len:0x44c
load:0x403c9700,len:0xbe4
load:0x403cc700,len:0x2a38
entry 0x403c98d4
[ 93][D][esp32-hal-tinyusb.c:675] tinyusb_enable_interface(): Interface CDC enabled
[ 94][D][esp32-hal-tinyusb.c:675] tinyusb_enable_interface(): Interface MSC enabled
[ 98][D][esp32-hal-tinyusb.c:675] tinyusb_enable_interface(): Interface HID enabled
[ 105][D][USBHID.cpp:61] tinyusb_enable_hid_device(): Device[0] len: 68
[ 160][D][FirmwareMSC.cpp:94] msc_update_get_required_disk_sectors(): APP size: 336928 (659 sectors)
[ 160][D][FirmwareMSC.cpp:100] msc_update_get_required_disk_sectors(): OTA size: 1310720 (2560 sectors)
[ 167][D][FirmwareMSC.cpp:113] msc_update_get_required_disk_sectors(): USING FAT12
[ 174][D][FirmwareMSC.cpp:116] msc_update_get_required_disk_sectors(): FAT sector size: 512
[ 183][D][FirmwareMSC.cpp:117] msc_update_get_required_disk_sectors(): FAT data sectors: 3235
[ 191][D][FirmwareMSC.cpp:118] msc_update_get_required_disk_sectors(): FAT table sectors: 10
[ 199][D][FirmwareMSC.cpp:119] msc_update_get_required_disk_sectors(): FAT total sectors: 3247 (1623KB)
[ 209][D][esp32-hal-tinyusb.c:564] tinyusb_load_enabled_interfaces(): Load Done: if_num: 4, descr_len: 130, if_mask: 0x15
[ 220][D][esp32-hal-cpu.c:244] setCpuFrequencyMhz(): PLL: 480 / 3 = 160 Mhz, APB: 80000000 Hz
[ 236][E][USBHID.cpp:346] SendReport(): not ready
[ 236][E][USBHID.cpp:346] SendReport(): not ready
[ 237][E][USBHID.cpp:346] SendReport(): not ready
[ 242][E][USBHID.cpp:346] SendReport(): not ready
[ 246][E][USBHID.cpp:346] SendReport(): not ready
[ 251][E][USBHID.cpp:346] SendReport(): not ready
[ 256][E][USBHID.cpp:346] SendReport(): not ready
[ 260][E][USBHID.cpp:346] SendReport(): not ready
[ 265][E][USBHID.cpp:346] SendReport(): not ready

[SuGlider]

@hoantv - I have tested it with Win11. The problem is that Win doesn't know what is BUTTON_START.

I tested it with an ESP32-S3 and when pressing the BOOT button on the board, the Joystick moves to the right bottom corner and returns back to the center.

This is the code that I have tested:

#if ARDUINO_USB_MODE
#warning This sketch should be used when USB is in OTG mode
void setup() {}
void loop() {}
#else
#include "USB.h"
#include "USBHIDGamepad.h"
USBHIDGamepad Gamepad;

const int buttonPin = 0;
int previousButtonState = HIGH;

void setup() {
  Serial.begin(115200);
  pinMode(buttonPin, INPUT_PULLUP);
  Gamepad.begin();
  USB.begin();
}

void loop() {
  int buttonState = digitalRead(buttonPin);
  if ((buttonState != previousButtonState) && (buttonState == LOW)) {
    Gamepad.leftStick(100, 100);
    delay(500);
    Gamepad.leftStick(0, 0);
    Serial.println("Button Pressed - gamePad should have sent event.");
  }
  previousButtonState = buttonState;
}
#endif /* ARDUINO_USB_MODE */

[SuGlider]

@RichardsATcn - I used Arduino Core 2.0.7 and it works fine also when USB is unplugged and plugged back.

This issue is solved.

[SuGlider]

@hoantv - Check PR #8058

One problem I had with the "TinyUSB HID Properties" Window was that it had to be Active/Selected (by clicking it), otherwise no gamepad information was displayed. The example of the PR #8058 works perfectly fine with Win10/11.