Skip to content

Conversation

marcg03
Copy link

@marcg03 marcg03 commented Aug 5, 2025

I'll be adding support for SteelSeries Arctis Nova 3P Wireless. Right now I managed to get the sidetone working.

I first tried seeing if the implementation of SteelSeries Arctis Nova 3 works for this model as well, but it doesn't.

@AnErrupTion
Copy link
Contributor

I first tried seeing if the implementation of SteelSeries Arctis Nova 3 works for this model as well, but it doesn't.

It seems like most (if not all) of the commands for each capability are the same, except without the 0x06 byte at the beginning. So you might be able to copy most of the code from the Arctis Nova 3 implementation for the Nova 3P (though I already saw the latter was a bit more capable, such as supporting more sidetone levels).

@marcg03
Copy link
Author

marcg03 commented Aug 8, 2025

Yeah, I copied most of the code

@marcg03 marcg03 force-pushed the feat/arctis-nova-3p-wireless branch from f3a551e to 16a6fd2 Compare August 11, 2025 10:50
@marcg03
Copy link
Author

marcg03 commented Aug 11, 2025

Support for Linux works fine on my end.

Unfortunately inside my Windows VM things don't work (I tried all Usage-Page + Interface combos as reported by headsetcontrol --dev -- --list). There seems to be an issue with how I use hid_send_feature_report because inside WireShark I see the packets have 1060 length.

@marcg03 marcg03 marked this pull request as ready for review August 11, 2025 10:52
@Sapd
Copy link
Owner

Sapd commented Sep 2, 2025

Support for Linux works fine on my end.

Unfortunately inside my Windows VM things don't work (I tried all Usage-Page + Interface combos as reported by headsetcontrol --dev -- --list). There seems to be an issue with how I use hid_send_feature_report because inside WireShark I see the packets have 1060 length.

Windows support is a bit tricky, it expects the exact same length.
Here it seems to be overwritten:
The FeatureReport seems to be supposed to be longer

https://github.com/libusb/hidapi/blob/f18d2c0768bed300d13758dd9f52b57163c0498f/windows/hid.c#L1256

I think we can merge anyways, we just need to maybe include some message or at least a comment that windows seems to not quite work yet with that implementation

@marcg03
Copy link
Author

marcg03 commented Sep 2, 2025

Alright, where should I write the message/comment?

@Sapd
Copy link
Owner

Sapd commented Sep 2, 2025

Are all functions erroring out or only specific?

@marcg03
Copy link
Author

marcg03 commented Sep 3, 2025

From what I recall, nothing works on windows and they don't error out but instead just don't have any effects.

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