feat(hid): support LIGHTSPEED receivers and RGB effects#351
Draft
LuisUrrutia wants to merge 4 commits into
Draft
feat(hid): support LIGHTSPEED receivers and RGB effects#351LuisUrrutia wants to merge 4 commits into
LuisUrrutia wants to merge 4 commits into
Conversation
Why: G502 X PLUS ships on a LIGHTSPEED receiver and reports RGB Effects 0x8071 plus Per Key Lighting V2 0x8081, so the existing Bolt/Unifying and older RGB paths could not surface or drive its lighting. Changes: - Recognize LIGHTSPEED receiver PIDs and enumerate them with the Unifying-style HID++ 1.0 receiver registers. - Try RGB Effects 0x8071 for solid colors before falling back to per-key lighting.
Why: G502 X Plus devices paired through LIGHTSPEED can expose RGB and host-hook mouse buttons without the MX-line ReprogControls table, so they need model-specific identity and UI handling. Changes: - detect G502-family model IDs and names, including the live G502 X Plus IDs - add a two-perspective G502 button layout with read-only native controls - keep display-only gaming controls out of runtime binding projection until their protocol is implemented - document follow-up protocol and diagnostics work in IMPROVEMENTS.md
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Adds support for Logitech LIGHTSPEED gaming receivers and the RGB effect path used by the G502 X PLUS.
Changes
046d:c547, and probe them with the HID++ 1.0 receiver registers used for Unifying-style slot enumeration.DeviceRoute::Unifyingshape to avoid changing the serialized route wire format.0x8071/0x8081as lighting-capable and tryRgbEffects(0x8071) before falling back to per-key lighting.Validation
cargo fmt --all -- --checkcargo test -p openlogi-core -p openlogi-hidpp -p openlogi-hid -p openlogi-cliOPENLOGI_LOG=debug cargo run -- listbuilt and ran, but the local macOS USB device tree did not show a Logitech receiver at that time (system_profiler SPUSBDataTypealso found none), so real G502 X PLUS hardware behavior still needs a connected-device check.Notes
046d:c547is the LIGHTSPEED receiver PID for the G502 X PLUS case this patch targets. The device exposesRGB EFFECTS(0x8071) andPER KEY LIGHTING V2(0x8081), so the lighting path needs the newer RGB effects family in addition to the older0x8070/0x8080paths.