Skip to content

Draft: Add support for Corsair Vengeance Pro SL memory to Vengeance Pro controller

wedgius requested to merge wedgius/OpenRGB:corsair_veng_sl_add into master

The Corsair Vengeance Pro SL modules are recognized by the Corsair Dominator Platinum controller. However, the SL modules have only 10 LEDs—like their non-slimline Vengeance Pro cousins—versus the 12 of the Dominator Platinum modules. This may cause unsatisfactory results with certain visual effects, and is generally not preferred.

This update adds an additional valid value of 0x1B at SMBus register 0x43. This value is also valid for Dominator Platinum modules, hence the weirdness.

As yet there doesn't seem to be an additional way to distinguish between these modules without being recognized by both controllers. If this is merged, it should be recommended to only enable the controller for the memory that the user actually has.

Edited by Adam Honse

Merge request reports