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.

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

esp_hid: fix output report char declaration (IDFGH-3402) #5369

Closed
wants to merge 3 commits into from

Conversation

robinkrens
Copy link
Contributor

esp_hid: fix output report characteristics declaration

Current issue: output reports sent by Windows not received.
The report characteristic declaration should also support write without
response

As specified by HIDs profile https://www.bluetooth.com/specifications/GATT/
See page 14:

Report Type | Read | Write | Write without Response | Notify | 
------------+------+-------+------------------------+--------|
Input       |  M   |   O   |           X            |    M   |
------------+------+-------+------------------------+--------|
Output      |  M   |   M   |           M	     |    X   |
------------+------+-------+------------------------+--------|
Feature     |  M   |   M   |           X            |    X   | 
------------+------+-------+------------------------+--------|
M=mandatory, O=optional, X=none

See pull request for suggested changes
Tested under Windows 10 with hidapi

Note: did not test under Linux, because the example at (examples/bluetooth/esp_hid_device)
did not work properly (gap related). Will post additional issue for that.

Current issue: output reports sent by Windows not received.
The report characteristic declaration should also support write without
response as specified by HIDS profile:
See https://www.bluetooth.com/specifications/GATT/ (page 14)
@github-actions github-actions bot changed the title esp_hid: fix output report char declaration esp_hid: fix output report char declaration (IDFGH-3402) May 30, 2020
@Alvin1Zhang
Copy link
Collaborator

@robinkrens Thanks for your contribution.

benjaminaigner added a commit to asterics/esp32_mouse_keyboard that referenced this pull request Jul 20, 2020
…(LED report), closing #38 :

- Added report ID for LEDs
- Added callback event type for LED
- Fixed the characteristic & added write no response (according to espressif/esp-idf#5369 THX @robinkrens for the hint!)
@ghost
Copy link

ghost commented Sep 16, 2021

Hi, @robinkrens

Could you please checkout to the 2078dfe and see whether the issue still exist?
If yes, please rebase your PR on the latest master branch, we will handle this PR.

Thanks

@espressif-bot espressif-bot added the Status: In Progress Work is in progress label Sep 16, 2021
@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: In Progress Work is in progress Resolution: NA Issue resolution is unavailable labels Jan 7, 2022
@Alvin1Zhang
Copy link
Collaborator

Thanks for your contribution, changes merged with 5f28c0c.

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

Successfully merging this pull request may close these issues.

5 participants