espressif / idf-extra-components

Additional components for ESP-IDF, maintained by Espressif

143 stars 86 forks source link

usb_host_hid v1.0.2: Refactor HID Interfaces parsing #302

Closed roma-jam closed 7 months ago

roma-jam commented 7 months ago

Checklist

[x] Component contains License
[x] Component contains README.md
[x] Component contains idf_component.yml file with url field defined
[x] Component was added to upload job
[x] Component was added to build job
[x] Optional: Component contains unit tests
[x] CI passing

Change description

Refactored parsing Interfaces descriptor of the USB HID device. Fixed the crash, when USB HID device has two interfaces with inconsistent numbers (e.g. Device has two Interfaces with bInterfaceNumber=0 and bInterfaceNumber=2).

github-actions[bot] commented 7 months ago

Unit Test Results

  15 files ±0   15 suites ±0 21m 9s :stopwatch: +11s   27 tests ±0   27 :heavy_check_mark: ±0 0 :zzz: ±0 0 :x: ±0 382 runs ±0 382 :heavy_check_mark: ±0 0 :zzz: ±0 0 :x: ±0

Results for commit e32bdc30. ± Comparison against base commit 66994c50.

roma-jam commented 7 months ago

Closed, as new PR in esp-usb repository has been created: https://github.com/espressif/esp-usb/pull/3