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

UefiHidDxe: Change HID descriptor read algorithm #339

Conversation

joschock
Copy link
Contributor

@joschock joschock commented Oct 19, 2023

Resolves #338

Description

Updates the algorithm used to read the HID descriptor from HID devices. Empirical testing indicates that some devices do not support reading the HID descriptor via the class-specific Get_Report() method described in USB HID 1.11. This changes the HID read to read the entire configuration descriptor and parse the HID descriptor out of the larger structure, and gives compatibility with a broader range of devices.

  • Impacts functionality?
    Supports a broader range of devices.
  • Impacts security?
  • Breaking change?
  • Includes tests?
  • Includes documentation?

How This Was Tested

Verified against emulated USB devices in QEMU.

Integration Instructions

N/A

@makubacki makubacki self-requested a review October 19, 2023 21:53
@makubacki makubacki added the type:enhancement New feature or pull request label Oct 19, 2023
@joschock joschock force-pushed the personal/joschock/hid_descriptor_read_improvement branch from 2655bfd to 7106736 Compare October 20, 2023 02:05
@makubacki makubacki enabled auto-merge (squash) October 20, 2023 19:17
@makubacki makubacki merged commit 310592e into microsoft:release/202302 Oct 20, 2023
31 checks passed
ProjectMuBot referenced this pull request in microsoft/mu_tiano_platforms Oct 20, 2023
Introduces 1 new commits in [Common/MU](https://github.com/microsoft/mu_plus.git).

<details>
<summary>Commits</summary>
<ul>
<li><a href="https://github.com/microsoft/mu_plus/commit/310592e057a25a9c737a35ea5c19c005352e347e">310592</a> UefiHidDxe: Change HID descriptor read algorithm (<a href="https://github.com/microsoft/mu_plus/pull/339">#339</a>)</li>
</ul>
</details>

Signed-off-by: Project Mu Bot <mubot@microsoft.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type:enhancement New feature or pull request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Bug]: Improve HID Descriptor Parsing Compatibility
3 participants