[USB][v5.1.4] Yamaha P-525 returns HUB: Bad transfer status 1: CHECK_SHORT_DEV_DESC (IDFGH-14542)

[chipweinberger]

IDF version.

v5.1.4

Espressif SoC revision.

esp32-s3

Development Kit.

custom pcb

Power Supply used.

USB

What is the expected behavior?

USB communication should succeed.

What is the actual behavior?

    (103239) HUB: Bad transfer status 1: CHECK_SHORT_DEV_DESC
    (103243) HUB: Stage failed: CHECK_SHORT_DEV_DESC

Steps to reproduce.

1. plug in Yamaha P-525

More Information.

This is all the relevant information I have. This is a customer report.

[chipweinberger]

@tore-espressif
@peter-marcisovsky
@roma-jam

Thanks for your time.

[roma-jam]

Hi @chipweinberger,

Thanks for reporting the problem.

Could you try to increase the root port delays in menuconfig?
Component config -> USB-OTG -> Root Hub configuration

Step 1. Increase only Reset recovery delay to 3000 ms. If that helps, you can decrease the value and find out which one works. If not:
Step 2. Increase Reset hold to 1000 ms. If not helps:
Step 3. Increase SetAddress() recovery time to 1000 ms.

Let me know if that helps.

[chipweinberger]

Thanks for the suggestions. i'll try it!

Question - My product needs to work with hundreds if not thousands of different piano models.

Could changing these hurt compatibility with other models?

I need one set of values that works with all digital pianos.

imo, we should use the most compatible values by default. and we should probably use the values most similar to a Windows PC

Thanks for your help.

[chipweinberger]

I'll also need a way to change these at runtime, so that the same firmware can support all models.

[roma-jam]

Hi @chipweinberger,

Yes, that's right indeed.
This will increase the overall time from plugging the device to the port and getting the enumeration done. It won't affect any devices, which works with less delay.

Right now there is a tendency to move from build-time configuration to the run-time and as a part of this process the delay values as a configurable parameter also could be eliminated. Unfortunately there is no information about when this could be implemented.