RGB.NET.Devices.Wooting/Enum/WootingDeviceType.cs

@@ -7,14 +7,10 @@ namespace RGB.NET.Devices.Wooting.Enum
     /// </summary>
     public enum WootingDeviceType
     {
-        /// <summary>
         /// 10 Keyless Keyboard. E.g. Wooting One
-        /// </summary>
         KeyboardTKL = 1,
 
-        /// <summary>
         /// Full Size keyboard. E.g. Wooting Two
-        /// </summary>
         Keyboard = 2
     }
 }

RGB.NET.Devices.Wooting/Enum/WootingDevicesIndexes.cs

@@ -0,0 +1,21 @@
+// ReSharper disable InconsistentNaming
+// ReSharper disable UnusedMember.Global
+
+using System.ComponentModel;
+
+#pragma warning disable 1591 // Missing XML comment for publicly visible type or member
+
+namespace RGB.NET.Devices.Wooting.Enum
+{
+    /// <summary>
+    /// Contains a list of available device-indexes.
+    /// </summary>
+    public enum WootingDevicesIndexes
+    {
+        [Description("Wooting One")]
+        WootingOne = 0,
+
+        [Description("Wooting Two")]
+        WootingTwo = 1
+    }
+}

RGB.NET.Devices.Wooting/Generic/WootingRGBDeviceInfo.cs

@@ -1,7 +1,6 @@
 using RGB.NET.Core;
 using RGB.NET.Devices.Wooting.Enum;
 using RGB.NET.Devices.Wooting.Helper;
-using RGB.NET.Devices.Wooting.Native;
 
 namespace RGB.NET.Devices.Wooting.Generic
 {
@@ -29,9 +28,9 @@ namespace RGB.NET.Devices.Wooting.Generic
         public object? LayoutMetadata { get; set; }
 
         /// <summary>
-        /// Gets the <see cref="WootingDeviceType"/> of the <see cref="WootingRGBDevice{TDeviceInfo}"/>.
+        /// Gets the <see cref="WootingDevicesIndexes"/> of the <see cref="WootingRGBDevice{TDeviceInfo}"/>.
         /// </summary>
-        public WootingDeviceType WootingDeviceType { get; }
+        public WootingDevicesIndexes DeviceIndex { get; }
 
         #endregion
 
@@ -41,13 +40,13 @@ namespace RGB.NET.Devices.Wooting.Generic
         /// Internal constructor of managed <see cref="WootingRGBDeviceInfo"/>.
         /// </summary>
         /// <param name="deviceType">The type of the <see cref="IRGBDevice"/>.</param>
-        /// <param name="deviceInfo">The <see cref="_WootingDeviceInfo"/> of the <see cref="IRGBDevice"/>.</param>
-        internal WootingRGBDeviceInfo(RGBDeviceType deviceType, _WootingDeviceInfo deviceInfo)
+        /// <param name="deviceIndex">The <see cref="WootingDevicesIndexes"/> of the <see cref="IRGBDevice"/>.</param>
+        internal WootingRGBDeviceInfo(RGBDeviceType deviceType, WootingDevicesIndexes deviceIndex)
         {
             this.DeviceType = deviceType;
-            this.WootingDeviceType = deviceInfo.DeviceType;
+            this.DeviceIndex = deviceIndex;
 
-            Model = deviceInfo.Model;
+            Model = deviceIndex.GetDescription();
             DeviceName = DeviceHelper.CreateDeviceName(Manufacturer, Model);
         }
 

RGB.NET.Devices.Wooting/Keyboard/WootingKeyboardLedMappings.cs

@@ -13,9 +13,9 @@ namespace RGB.NET.Devices.Wooting.Keyboard
     {
         #region Properties & Fields
 
-        #region TKL
+        #region Wooting One
 
-        private static readonly Dictionary<LedId, (int row, int column)> TKL_US = new()
+        private static readonly Dictionary<LedId, (int row, int column)> WootingOne_US = new()
                                                                                          {
             { LedId.Keyboard_Escape, (0,0) },
             { LedId.Keyboard_F1, (0,2) },
@@ -111,7 +111,7 @@ namespace RGB.NET.Devices.Wooting.Keyboard
             { LedId.Keyboard_ArrowRight, (5,16) }
         };
 
-        private static readonly Dictionary<LedId, (int row, int column)> TKL_UK = new()
+        private static readonly Dictionary<LedId, (int row, int column)> WootingOne_UK = new()
                                                                                          {
             { LedId.Keyboard_Escape, (0,0) },
             { LedId.Keyboard_F1, (0,2) },
@@ -178,7 +178,7 @@ namespace RGB.NET.Devices.Wooting.Keyboard
             { LedId.Keyboard_L, (3,9) },
             { LedId.Keyboard_SemicolonAndColon, (3,10) },
             { LedId.Keyboard_ApostropheAndDoubleQuote, (3,11) },
-            { LedId.Keyboard_NonUsTilde, (3,12) },
+            { LedId.Keyboard_NonUsTilde, (3,11) },
             { LedId.Keyboard_Enter, (3,13) },
 
             { LedId.Keyboard_LeftShift, (4,0) },
@@ -211,9 +211,9 @@ namespace RGB.NET.Devices.Wooting.Keyboard
 
         #endregion
 
-        #region Fullsize
+        #region Wooting Two
 
-        private static readonly Dictionary<LedId, (int row, int column)> Fullsize_US = new()
+        private static readonly Dictionary<LedId, (int row, int column)> WootingTwo_US = new()
                                                                                          {
             { LedId.Keyboard_Escape, (0,0) },
             { LedId.Keyboard_F1, (0,2) },
@@ -330,7 +330,7 @@ namespace RGB.NET.Devices.Wooting.Keyboard
             { LedId.Keyboard_NumPeriodAndDelete, (5,19) }
         };
 
-        private static readonly Dictionary<LedId, (int row, int column)> Fullsize_UK = new()
+        private static readonly Dictionary<LedId, (int row, int column)> WootingTwo_UK = new()
                                                                                          {
             { LedId.Keyboard_Escape, (0,0) },
             { LedId.Keyboard_F1, (0,2) },
@@ -454,20 +454,20 @@ namespace RGB.NET.Devices.Wooting.Keyboard
         /// <summary>
         /// Contains all the hardware-id mappings for Wooting devices.
         /// </summary>
-        public static readonly Dictionary<WootingDeviceType, Dictionary<WootingPhysicalKeyboardLayout, Dictionary<LedId, (int row, int column)>>> Mapping =
+        public static readonly Dictionary<WootingDevicesIndexes, Dictionary<WootingPhysicalKeyboardLayout, Dictionary<LedId, (int row, int column)>>> Mapping =
             new()
             {
-                { WootingDeviceType.KeyboardTKL, new Dictionary<WootingPhysicalKeyboardLayout, Dictionary<LedId, (int row, int column)>>
+                { WootingDevicesIndexes.WootingOne, new Dictionary<WootingPhysicalKeyboardLayout, Dictionary<LedId, (int row, int column)>>
                   {
-                    { WootingPhysicalKeyboardLayout.US, TKL_US },
-                    { WootingPhysicalKeyboardLayout.UK, TKL_UK }
+                    { WootingPhysicalKeyboardLayout.US, WootingOne_US },
+                    { WootingPhysicalKeyboardLayout.UK, WootingOne_UK }
                   }
                 },
 
-                { WootingDeviceType.Keyboard, new Dictionary<WootingPhysicalKeyboardLayout, Dictionary<LedId, (int row, int column)>>
+                { WootingDevicesIndexes.WootingTwo, new Dictionary<WootingPhysicalKeyboardLayout, Dictionary<LedId, (int row, int column)>>
                   {
-                    { WootingPhysicalKeyboardLayout.US, Fullsize_US },
-                    { WootingPhysicalKeyboardLayout.UK, Fullsize_UK }
+                    { WootingPhysicalKeyboardLayout.US, WootingTwo_US },
+                    { WootingPhysicalKeyboardLayout.UK, WootingTwo_UK }
                   }
                 }
             };

RGB.NET.Devices.Wooting/Keyboard/WootingKeyboardRGBDevice.cs

@@ -38,14 +38,14 @@ namespace RGB.NET.Devices.Wooting.Keyboard
         private void InitializeLayout()
         {
             //TODO DarthAffe 13.02.2021: Check how the mapping can work without knowing the physical layout
-            Dictionary<LedId, (int row, int column)> mapping = WootingKeyboardLedMappings.Mapping[DeviceInfo.WootingDeviceType][WootingPhysicalKeyboardLayout.US];
+            Dictionary<LedId, (int row, int column)> mapping = WootingKeyboardLedMappings.Mapping[DeviceInfo.DeviceIndex][WootingPhysicalKeyboardLayout.US];
 
             foreach (KeyValuePair<LedId, (int row, int column)> led in mapping)
                 AddLed(led.Key, new Point(led.Value.column * 19, led.Value.row * 19), new Size(19, 19));
         }
 
         /// <inheritdoc />
-        protected override object GetLedCustomData(LedId ledId) => WootingKeyboardLedMappings.Mapping[DeviceInfo.WootingDeviceType][WootingPhysicalKeyboardLayout.US][ledId];
+        protected override object GetLedCustomData(LedId ledId) => WootingKeyboardLedMappings.Mapping[DeviceInfo.DeviceIndex][WootingPhysicalKeyboardLayout.US][ledId];
 
         /// <inheritdoc />
         protected override void UpdateLeds(IEnumerable<Led> ledsToUpdate) => UpdateQueue.SetData(GetUpdateData(ledsToUpdate));

RGB.NET.Devices.Wooting/Keyboard/WootingKeyboardRGBDeviceInfo.cs

@@ -1,7 +1,6 @@
 using RGB.NET.Core;
 using RGB.NET.Devices.Wooting.Enum;
 using RGB.NET.Devices.Wooting.Generic;
-using RGB.NET.Devices.Wooting.Native;
 
 namespace RGB.NET.Devices.Wooting.Keyboard
 {
@@ -23,9 +22,9 @@ namespace RGB.NET.Devices.Wooting.Keyboard
         /// <summary>
         /// Internal constructor of managed <see cref="T:RGB.NET.Devices.Wooting.WootingKeyboardRGBDeviceInfo" />.
         /// </summary>
-        /// <param name="deviceInfo">The native <see cref="T:RGB.NET.Devices.Wooting.Native._WootingDeviceInfo" />.</param>
-        internal WootingKeyboardRGBDeviceInfo(_WootingDeviceInfo deviceInfo)
-            : base(RGBDeviceType.Keyboard, deviceInfo)
+        /// <param name="deviceIndex">The index of the <see cref="T:RGB.NET.Devices.Wooting.WootingKeyboardRGBDevice" />.</param>
+        internal WootingKeyboardRGBDeviceInfo(WootingDevicesIndexes deviceIndex)
+            : base(RGBDeviceType.Keyboard, deviceIndex)
         { }
 
         #endregion

RGB.NET.Devices.Wooting/Native/_WootingDeviceInfo.cs

@@ -20,7 +20,5 @@ namespace RGB.NET.Devices.Wooting.Native
         internal byte KeycodeLimit { get; private set; }
 
         internal WootingDeviceType DeviceType { get; private set; }
-
-        internal bool V2Interface { get; set; }
     }
 }

RGB.NET.Devices.Wooting/WootingDeviceProvider.cs

@@ -70,8 +70,17 @@ namespace RGB.NET.Devices.Wooting
                 if (_WootingSDK.KeyboardConnected())
                 {
                     _WootingDeviceInfo nativeDeviceInfo = (_WootingDeviceInfo)Marshal.PtrToStructure(_WootingSDK.GetDeviceInfo(), typeof(_WootingDeviceInfo))!;
+                    IWootingRGBDevice? device = nativeDeviceInfo.Model switch
+                    {
+                        "Wooting two" => new WootingKeyboardRGBDevice(new WootingKeyboardRGBDeviceInfo(WootingDevicesIndexes.WootingTwo), GetUpdateTrigger()),
+                        "Wooting one" => new WootingKeyboardRGBDevice(new WootingKeyboardRGBDeviceInfo(WootingDevicesIndexes.WootingOne), GetUpdateTrigger()),
+                        _ => null
+                    };
 
-                    yield return new WootingKeyboardRGBDevice(new WootingKeyboardRGBDeviceInfo(nativeDeviceInfo), GetUpdateTrigger());
+                    if (device == null)
+                        Throw(new RGBDeviceException("No supported Wooting keyboard connected"));
+                    else
+                        yield return device;
                 }
             }
         }