Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

usbh_print_intf_desc prints wrong value for tbInterfaceClass (IDFGH-7422) #9000

Closed
nickbits1024 opened this issue May 21, 2022 · 4 comments
Closed

usbh_print_intf_desc prints wrong value for tbInterfaceClass (IDFGH-7422) #9000

nickbits1024 opened this issue May 21, 2022 · 4 comments
Labels
Resolution: Done Issue is done internally Status: Done Issue is done internally

Comments

@nickbits1024
Copy link
Contributor

usbh_print_intf_desc in components\usb\usb_helpers.c prints the value of bInterfaceProtocol for its output of bInterfaceClass. It is also does not print the values of bInterfaceSubClass or bInterfaceProtocol.
@espressif-bot espressif-bot added the Status: Opened Issue is new label May 21, 2022
@github-actions github-actions bot changed the title usbh_print_intf_desc prints wrong value for tbInterfaceClass usbh_print_intf_desc prints wrong value for tbInterfaceClass (IDFGH-7422) May 21, 2022
@Alvin1Zhang
Copy link
Collaborator

@nickbits1024 Thanks for reporting. Would you please help provide more details as suggested in the issue template? Information like elf, sdk configuration, backtrace, log outputs, commit ID, hardware and etc. would help us debug further. Thanks.

@nickbits1024
Copy link
Contributor Author

Sorry I didn't include most of that because I fixed it in a PR. It is an obvious bug/fix that doesn't really need any context.
But it is a bug off master/latest and my fix is in sha=0f645e59b68e98852fcfab628a24e117bbbc2479

@Alvin1Zhang
Copy link
Collaborator

Thanks for reporting and your contribution at #9001, we have put it under internal review queue, will update once there are progress, thanks.

@espressif-bot espressif-bot added Resolution: NA Issue resolution is unavailable Status: Done Issue is done internally Resolution: Done Issue is done internally and removed Status: Opened Issue is new Resolution: NA Issue resolution is unavailable labels May 24, 2022
@Alvin1Zhang
Copy link
Collaborator

PR #9001 has been accepted and merged, thanks for reporting and contribution again, feel free to reopen.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
Resolution: Done Issue is done internally Status: Done Issue is done internally
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@nickbits1024 @Alvin1Zhang @espressif-bot