Skip to content

Add Corsair Commander DUO captures#21

Open
ndizazzo wants to merge 1 commit into
liquidctl:masterfrom
ndizazzo:add-corsair-commander-duo-data
Open

Add Corsair Commander DUO captures#21
ndizazzo wants to merge 1 commit into
liquidctl:masterfrom
ndizazzo:add-corsair-commander-duo-data

Conversation

@ndizazzo
Copy link
Copy Markdown

Summary

Add Corsair Commander DUO USBPcap captures and helper analyzers used while developing liquidctl support for the device.

Contents

  • Adds a new Corsair Commander DUO/ device directory.
  • Includes 26 renamed captures grouped into initial hardware-mode captures, Device Memory preview/apply captures, and focused Device Memory commit/readback captures.
  • Adds Python analyzer scripts for summarizing HID endpoint writes and extracting HID packet CSVs from USBPcap captures.

Notes

  • Captures cover software/live lighting previews, Device Memory fixed/off/rainbow commits, fan/cooling preset traffic, startup, and readback behavior.
  • Focused readback confirms Commander DUO Device Memory fixed-color payload order ff bb gg rr on endpoint 65 6d.
  • The analyzer scripts require tshark and read HID payloads from usbhid.data.

Verification

  • Organized files according to existing Corsair device directory conventions in this repository.
  • Renamed capture files consistently as commander-duo-NN-<scenario>.pcap.
  • Ran duo_hid_summary.py on the focused readback capture and confirmed it reports the expected 65 6d / 7e 20 payload.
  • Ran duo_extract_hid_csv.py on the same capture for host/device HID endpoints.
  • Ran Python syntax checks for both analyzer scripts.

Copilot AI review requested due to automatic review settings May 13, 2026 23:05
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

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

Copilot encountered an error and was unable to review this pull request. You can try again by re-requesting a review.

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants