unmount function called during mount, and the mount function called during unmount (IEC-203)

[lijunru-hub]

Answers checklist.

• I have read the component documentation ESP-IDF Components and the issue is not addressed there.
• I am using target and esp-idf version as defined in component's idf_component.yml
• I have searched the issue tracker for a similar issue and not found any related issue.

Which component are you using? If you choose Other, provide details in More Information.

device/esp_tinyusb

ESP-IDF version.

IDF release/v5.2

Development Kit.

ESP32S3

Used Component version.

1.4.5

More Information.

The function calls are opposite to the intended logic.

[lijunru-hub]

https://github.com/espressif/esp-usb/blob/master/device/esp_tinyusb/tusb_msc_storage.c#L628C1-L639C2

[peter-marcisovsky]

It works like this:
tud_umount_cb is called when a USB device is unmounted from a USB host -> a storage is mounted on the application FW tinyusb_msc_storage_mount. Now the application FW owns the storage.

tud_mount_cb is called when a USB device is mounted to a USB host -> a storage is unmounted from the application FW tinyusb_msc_storage_unmount. Now the USB Host owns the storage.

https://github.com/espressif/esp-usb/blob/master/device/esp_tinyusb/include/tusb_msc_storage.h#L127

[DamienEspitallier]

Is calling tinyusb_msc_storage_unmount in tud_mount_cb really correct ?

It only does something if the firmware has previously mounted the storage (and probably do something with it). It will remove the access for the firmware with no way to refuse and can happen at any moment (user can plug the cable when he wants).

May be a good way would be to add a way to refuse the unmont with a custom return value of the TINYUSB_MSC_EVENT_PREMOUNT_CHANGED callback fired in tinyusb_msc_storage_unmount ? I can do a PR if you think this is a good fix.

[lijunru-hub]

There's another issue: tusb_msc_storage.c occupies the tud_mount and tud_unmount functions.

Although two callback interfaces are provided:

• tusb_msc_callback_t callback_mount_changed
• tusb_msc_callback_t callback_premount_changed

there is a logical bug during actual use. The callback is only triggered the first time, and subsequent plug-in or unplug events do not trigger the callback.

I believe a better approach would be to register a TinyUSB event and return the tud_mount and tud_unmount functions for user control.