Fix timing issues in USB drivers

In combination with slow USB hubs or KVM switches, some drivers
apparently fail to initialize due to too strict timing constraints.
So be more lenient with timings, and increase the time we wait between
issuing of commands

Referencs #161

eruption-hwutil/src/hwdevices/roccat_kone_pro_air.rs

@@ -21,6 +21,7 @@ use std::collections::HashMap;
 use std::time::Duration;
 use std::{cell::RefCell, thread};
 
+use crate::constants::DEVICE_SETTLE_MILLIS;
 #[allow(unused)]
 use crate::{constants, eprintln_v, println_v};
 
@@ -302,7 +303,7 @@ impl DeviceTrait for RoccatKoneProAir {
                 Err(_) => return Err(HwDeviceError::InvalidResult {}.into()),
             }
 
-            thread::sleep(Duration::from_millis(70));
+            thread::sleep(Duration::from_millis(DEVICE_SETTLE_MILLIS));
 
             Ok(())
         }
@@ -404,7 +405,7 @@ impl DeviceTrait for RoccatKoneProAir {
                     _ => { /* do nothing */ }
                 }
 
-                thread::sleep(Duration::from_millis(15));
+                thread::sleep(Duration::from_millis(DEVICE_SETTLE_MILLIS));
             }
 
             Ok(DeviceStatus(table))

eruption/src/constants.rs

@@ -104,7 +104,7 @@ pub const MAX_EVENTS_PER_ITERATION: u64 = 128;
 pub const EVENTS_UPCALL_RATE_LIMIT_MILLIS: u64 = 25;
 
 /// Amount of time that has to pass before we retry sending a command to the LED control device
-pub const DEVICE_SETTLE_MILLIS: u64 = 15;
+pub const DEVICE_SETTLE_MILLIS: u64 = 25;
 
 /// Update sensors every n seconds
 /// It is recommended to use a prime number value here

eruption/src/hwdevices/roccat_kone_pro_air.rs

@@ -22,12 +22,13 @@ use evdev_rs::enums::EV_KEY;
 use hidapi::{HidApi, HidDevice};
 use log::*;
 use parking_lot::{Mutex, RwLock};
+use std::time::Duration;
 // use std::sync::atomic::Ordering;
-use std::any::Any;
 use std::collections::HashMap;
 use std::sync::Arc;
+use std::{any::Any, thread};
 
-use crate::constants;
+use crate::constants::{self, DEVICE_SETTLE_MILLIS};
 
 use super::{
     DeviceCapabilities, DeviceInfoTrait, DeviceStatus, DeviceTrait, HwDeviceError, MouseDevice,
@@ -256,6 +257,8 @@ impl RoccatKoneProAir {
                 Err(_) => { /* do nothing */ }
             }
 
+            thread::sleep(Duration::from_millis(DEVICE_SETTLE_MILLIS));
+
             Ok(())
         }
     }
@@ -507,6 +510,7 @@ impl DeviceInfoTrait for RoccatKoneProAir {
             } */
 
             let result = super::DeviceInfo::new(0_i32);
+
             Ok(result)
         }
     }

eruption/src/hwdevices/roccat_vulcan_1xx.rs

@@ -27,7 +27,7 @@ use std::thread;
 use std::time::Duration;
 use std::{any::Any, mem::size_of};
 
-use crate::constants;
+use crate::constants::{self, DEVICE_SETTLE_MILLIS};
 
 use super::{
     DeviceCapabilities, DeviceInfoTrait, DeviceStatus, DeviceTrait, HwDeviceError, KeyboardDevice,
@@ -469,6 +469,8 @@ impl RoccatVulcan1xx {
 
                     Err(_) => return Err(HwDeviceError::InvalidResult {}.into()),
                 }
+
+                thread::sleep(Duration::from_millis(DEVICE_SETTLE_MILLIS));
             }
         }
     }

eruption/src/hwdevices/roccat_vulcan_pro.rs

@@ -25,7 +25,7 @@ use std::collections::HashMap;
 use std::{any::Any, mem::size_of, time::Duration};
 use std::{sync::Arc, thread};
 
-use crate::constants;
+use crate::constants::{self, DEVICE_SETTLE_MILLIS};
 
 use super::{
     DeviceCapabilities, DeviceInfoTrait, DeviceStatus, DeviceTrait, HwDeviceError, KeyboardDevice,
@@ -325,7 +325,7 @@ impl RoccatVulcanPro {
             //     }
             // }
 
-            thread::sleep(Duration::from_millis(70));
+            thread::sleep(Duration::from_millis(DEVICE_SETTLE_MILLIS));
 
             Ok(())
         }

eruption/src/hwdevices/roccat_vulcan_pro_tkl.rs

@@ -26,7 +26,7 @@ use std::time::Duration;
 use std::{any::Any, mem::size_of};
 use std::{sync::Arc, thread};
 
-use crate::constants;
+use crate::constants::{self, DEVICE_SETTLE_MILLIS};
 
 use super::{
     DeviceCapabilities, DeviceInfoTrait, DeviceStatus, DeviceTrait, HwDeviceError, KeyboardDevice,
@@ -322,7 +322,7 @@ impl RoccatVulcanProTKL {
             //     }
             // }
 
-            thread::sleep(Duration::from_millis(70));
+            thread::sleep(Duration::from_millis(DEVICE_SETTLE_MILLIS));
 
             Ok(())
         }

eruption/src/hwdevices/roccat_vulcan_tkl.rs

@@ -25,7 +25,7 @@ use std::collections::HashMap;
 use std::{any::Any, mem::size_of, time::Duration};
 use std::{sync::Arc, thread};
 
-use crate::constants;
+use crate::constants::{self, DEVICE_SETTLE_MILLIS};
 
 use super::{
     DeviceCapabilities, DeviceInfoTrait, DeviceStatus, DeviceTrait, HwDeviceError, KeyboardDevice,
@@ -420,7 +420,7 @@ impl RoccatVulcanTKL {
             //     }
             // }
 
-            thread::sleep(Duration::from_millis(70));
+            thread::sleep(Duration::from_millis(DEVICE_SETTLE_MILLIS));
 
             Ok(())
         }