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

Fix: Add ID for Arctis Nova 7X revision #352

Merged
merged 3 commits into from
May 31, 2024

Conversation

rpbaptist
Copy link
Contributor

Changes made

The ID for Arctis Nova 7X seems to be incorrect.

❯ headsetcontrol -b
No supported device found

❯ headsetcontrol --dev -- --list --device 0x1038:0x2258
Device Found
  VendorID: 0x1038
 ProductID: 0x2258
  path: /dev/hidraw7
  serial_number: 
  Manufacturer: SteelSeries
  Product:      Arctis Nova 7X
  Interface:    3
  Usage-Page: 0xffc0 Usageid: 0x1

Device Found
  VendorID: 0x1038
 ProductID: 0x2258
  path: /dev/hidraw8
  serial_number: 
  Manufacturer: SteelSeries
  Product:      Arctis Nova 7X
  Interface:    4
  Usage-Page: 0xc Usageid: 0x1

Device Found
  VendorID: 0x1038
 ProductID: 0x2258
  path: /dev/hidraw9
  serial_number: 
  Manufacturer: SteelSeries
  Product:      Arctis Nova 7X
  Interface:    5
  Usage-Page: 0xff00 Usageid: 0x1

After changing the ID and recompiling and installing:

❯ headsetcontrol -b
Found SteelSeries Arctis Nova 7 (Arctis Nova 7X)!

Battery:
	Status: BATTERY_AVAILABLE
	Level: 100%

Checklist

  • I adjusted the README (if needed)
  • For new features in HeadsetControl: I discussed it beforehand in Issues or Discussions and adhered to the wiki

@Sapd
Copy link
Owner

Sapd commented May 31, 2024

There seems to be multiple revisions/ids of the same device: #233
So instead of replacing it you will probably have to add yours.

@rpbaptist rpbaptist changed the title Fix: Correct ID for Arctis Nova 7X Fix: Add ID for Arctis Nova 7X revision May 31, 2024
@rpbaptist
Copy link
Contributor Author

There seems to be multiple revisions/ids of the same device: #233 So instead of replacing it you will probably have to add yours.

Thanks. I gave it a shot by simply adding _v2 to it. All seems to be in order, but I'm not familiar with the code here, so I might be overlooking something.

Found SteelSeries Arctis Nova 7 (Arctis Nova 7X)!

Battery:
	Status: BATTERY_AVAILABLE
	Level: 100%

@Sapd Sapd merged commit f64ac48 into Sapd:master May 31, 2024
5 checks passed
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.

3 participants