sinowealth: don't enable LED support if LED type is not defined

This commit is contained in:
Yaroslav Chvanov
2022-01-28 15:55:46 +03:00
committed by Peter Hutterer
parent 1edb6df25b
commit 03d2d71f82

View File

@@ -614,15 +614,6 @@ sinowealth_init_profile(struct ratbag_device *device)
if (rc)
return rc;
/* LED count. */
drv_data->led_count = 0;
if (config->rgb_effect != RGB_NOT_SUPPORTED) {
drv_data->led_count += 1;
}
/* We may want to account for the DPI LEDs in the future.
* We don't support them yet, so it's not a priority now.
*/
if (strncmp(fw_version, "V102", 4) == 0) {
log_info(device->ratbag, "Found a Glorious Model O (old firmware) or a Glorious Model D\n");
@@ -662,6 +653,15 @@ sinowealth_init_profile(struct ratbag_device *device)
log_info(device->ratbag, "Found an unknown SinoWealth mouse\n");
}
/* LED count. */
drv_data->led_count = 0;
if (config->rgb_effect != RGB_NOT_SUPPORTED && drv_data->led_type != LED_NONE) {
drv_data->led_count += 1;
}
/* We may want to account for the DPI LEDs in the future.
* We don't support them yet, so it's not a priority now.
*/
/* number of DPIs = all DPIs from min to max (inclusive) and "0 DPI" as a special value
* to signal a disabled DPI step.
*/