espressif / esp-usb

Other
30 stars 16 forks source link

feature(esp_tinyusb): Added tusb_teardown() call while tinyusb_driver_uninstall() #39

Open roma-jam opened 5 months ago

roma-jam commented 5 months ago

esp_tinyusb (Not a Release)

Requirements

Description

Changes

Notes

  1. Test case only covers the enumeration and configuration for a class. There is no communication in between tinyusb driver tinyusb_driver_install() and tinyusb_driver_uninstall() calls.

Resolve https://github.com/espressif/esp-idf/issues/13788

roma-jam commented 5 months ago

@tore-espressif , @peter-marcisovsky This changes unlock the possibility to run usb device tests one by one (https://github.com/espressif/tinyusb/pull/27 should be merged first, though).

I posted two questions for both of you, because it seems that we can make it better and we can use it.

I need that for enumeration driver testing (but maybe I will eliminate esp_tinyusb component from the chain, but anyway), so feel free to check it out when possible.

Meanwhile, I will go and cover the Enum Driver as much as I can.

finger563 commented 2 months ago

Is there a timeline for this feature to be finalized?

roma-jam commented 2 months ago

Hi @finger563 ,

This feature allows the re-configure the driver with these changes: https://github.com/espressif/tinyusb/pull/27 But as far as the same changes are already in the upstream, so we will rebase these changes after updating version to v0.17.0. For current feature there is no available timeline for now, but the one thing for sure: the update to v0.17.0 will be done first.

Meanwhile, this feature should be available in the upstream of TinyUSB, if anything it is possible to use it.

Sorry for the inconvenience.