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

[XAP] finalise the HID protocol packet format #15619

Closed
tzarc opened this issue Dec 28, 2021 · 1 comment
Closed

[XAP] finalise the HID protocol packet format #15619

tzarc opened this issue Dec 28, 2021 · 1 comment

Comments

@tzarc
Copy link
Member

tzarc commented Dec 28, 2021

XAP Task Info

Original Issue
Original spec document
Current XAP Definitions
Current XAP Generated Docs
Placeholder PR

Description

Any work in this area needs to be discussed with QMK Collaborators first, and in this case primarily with @tzarc due to their prior work in this area.

  • The XAP protocol when communicating over HID only works with fixed-length packets.
  • The original specification defines a request/response pattern, with a correlation token in order to determine when a response is received.
  • The request/response pattern is not fully synchronous, in the sense that there is no guarantee that the next received packet is the corresponding response
  • The request/response pattern is somewhat synchronous, in the sense that there is a guarantee that there is only one "in-flight" request being handled by the firmware at any one time, and as such the received responses will be in the same order as the original requests (interspersed with other packets as required)
  • Broadcast messages (like logging output) may be received before the actual response is received by the host app
  • Host apps will not receive copies of any outbound packets made by other host apps
  • Host apps will receive all responses/broadcasts regardless of whether or not they were the initiator of the request
  • Need to decide if the packet format is capable of catering for all possible usages
  • Need to decide if a randomly-generated 16-bit token is enough, or if it should be bumped to 24- or 32-bit
  • Need to investigate whether or not feature reports should be used instead
@tzarc tzarc added enhancement help wanted xap XAP-related issues/PRs labels Dec 28, 2021
@stale
Copy link

stale bot commented Apr 16, 2022

This issue has been automatically marked as stale because it has not had activity in the last 90 days. It will be closed in the next 30 days unless it is tagged properly or other activity occurs.
For maintainers: Please label with bug, in progress, on hold, discussion or to do to prevent the issue from being re-flagged.

@stale stale bot added the stale Issues or pull requests that have become inactive without resolution. label Apr 16, 2022
@stale stale bot removed the stale Issues or pull requests that have become inactive without resolution. label Apr 16, 2022
@tzarc tzarc closed this as completed Jan 3, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: Done
Development

No branches or pull requests

1 participant