-
Notifications
You must be signed in to change notification settings - Fork 399
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
BASS & GATT fixes. #1779
BASS & GATT fixes. #1779
Conversation
@sjanc @KKopyscinski FYI |
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.
as discussed offline BLE_GATT_CHR_F_WRITE_ENC defines permissions for write so both flags needs to be present
This commit changes BASS audio scan control point and BASS broadcast receive state characteristics properties. Previous properties were not correct with specification.
776a14a
to
966fcad
Compare
Change keyword and description from PACS to BASS.
#define BLE_GATT_CHR_F_AUTH_SIGN_WRITE 0x0040 | ||
|
||
/** GATT Characteristic Flag: Reliable Writes. */ |
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.
reliable writes are (extended) property, not permission
This commit changes GATT flags descriptions, making them more readable regarding permissions and properties.
f8af0ed
to
3341671
Compare
This commit modifies GATT functions description to get them up to speed with current doxygen rules.
BASS related changes LGTM |
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.
BASS part LGTM too
Fixed characteristic properties.
Changed naming.