Skip to content

Conversation

@ayedm1
Copy link
Contributor

@ayedm1 ayedm1 commented Jan 18, 2026

This pull request adds host HID API functions to get and set the HID protocol (boot vs report mode) and introduces an optional callback for the device side when the protocol changes.

  • Added new host-side APIs ux_host_class_hid_protocol_set() and ux_host_class_hid_protocol_get() with error checking wrappers.
  • Added device-side optional callback ux_device_class_hid_set_protocol_callback that is invoked when the protocol changes
  • Added comprehensive test to verify the new protocol callback functionality

fdesbiens and others added 2 commits January 14, 2026 11:01
Merging changes ahead of hotfix release 202504a
… set/get protocol

This pull request adds host HID API functions to get and set the HID protocol (boot vs report mode) and introduces an optional callback for the device side when the protocol changes.

- Added new host-side APIs ux_host_class_hid_protocol_set() and ux_host_class_hid_protocol_get() with error checking wrappers.
- Added device-side optional callback ux_device_class_hid_set_protocol_callback that is invoked when the protocol changes
- Added comprehensive test to verify the new protocol callback functionality
@ayedm1 ayedm1 requested review from ABOUSTM and rahmanih January 18, 2026 13:35
@ayedm1 ayedm1 moved this to In review in ThreadX Roadmap Jan 18, 2026
@ayedm1 ayedm1 added the feature New feature or enhancement request label Jan 18, 2026
@fdesbiens fdesbiens changed the base branch from master to dev February 5, 2026 21:36
Copy link
Contributor

@rahmanih rahmanih left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

IIRC, this PR implements 2 features in HID Device and Host Classes.
please split them in separate commits.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

feature New feature or enhancement request

Projects

Status: In review

Development

Successfully merging this pull request may close these issues.

3 participants