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.
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
Add historical EP compatibility to bth class #2279
Conversation
3dd4cdb
to
8bac68d
Compare
hihi @kasjer do you have time to help reviewing this. Thank you. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good.
Verified not to break existing code with CFG_TUD_BTH_ISO_ALT_COUNT requirement.
Historical compatibility looks good (not tested though)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
thank you @kasjer for reviewing, I will make some tweak before merging this.
…_COUNT to tusb_option.h and mandatory for BTH driver
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
thank you for your PR. I made some tweaks
- rename to CFG_TUD_BTH_HISTORICAL_COMPATIBLE
- move CFG_TUD_BTH_ISO_ALT_COUNT to tusb_option.h and mandatory for BTH driver
will merge when ci passed
Than you very much! |
Describe the PR
Two improvements in this PR for the BTH class:
bRequest
is always0xe0
. See Bluetooth Core v5.3, Vol. 4, Part B, Section 2.2