From 16789ef1f10c919031679cb44c0b6a347c90d84d Mon Sep 17 00:00:00 2001
From: Adam Honse <calcprogrammer1@gmail.com>
Date: Fri, 8 Oct 2021 23:13:02 -0500
Subject: [PATCH] Alloy FPS: Don't send extended packet, increase time between
 packets to 10ms to eliminate flickering

---
 .../HyperXAlloyFPSController.cpp              | 74 +------------------
 .../HyperXAlloyFPSController.h                | 14 +---
 2 files changed, 4 insertions(+), 84 deletions(-)

diff --git a/Controllers/HyperXKeyboardController/HyperXAlloyFPSController.cpp b/Controllers/HyperXKeyboardController/HyperXAlloyFPSController.cpp
index fbe00b9a1..c593a01e3 100644
--- a/Controllers/HyperXKeyboardController/HyperXAlloyFPSController.cpp
+++ b/Controllers/HyperXKeyboardController/HyperXAlloyFPSController.cpp
@@ -139,7 +139,7 @@ void HyperXAlloyFPSController::SetLEDsDirect(std::vector<RGBColor> colors)
         red_color_data
         );
 
-    std::this_thread::sleep_for(5ms);
+    std::this_thread::sleep_for(10ms);
 
     SendDirect
         (
@@ -147,20 +147,13 @@ void HyperXAlloyFPSController::SetLEDsDirect(std::vector<RGBColor> colors)
         grn_color_data
         );
 
-    std::this_thread::sleep_for(5ms);
+    std::this_thread::sleep_for(10ms);
 
     SendDirect
         (
         HYPERX_ALLOY_FPS_COLOR_CHANNEL_BLUE,
         blu_color_data
         );
-
-    std::this_thread::sleep_for(5ms);
-
-    SendDirectExtended
-        (
-        ext_color_data
-        );
 }
 
 void HyperXAlloyFPSController::SetLEDs(std::vector<RGBColor> colors)
@@ -201,14 +194,6 @@ void HyperXAlloyFPSController::SetLEDs(std::vector<RGBColor> colors)
         HYPERX_ALLOY_FPS_COLOR_CHANNEL_BLUE,
         blu_color_data
         );
-
-    std::this_thread::sleep_for(5ms);
-    
-    SendExtendedColor
-        (
-        0x01,
-        ext_color_data
-        );
 }
 
 /*-------------------------------------------------------------------------------------------------*\
@@ -381,33 +366,6 @@ void HyperXAlloyFPSController::SendColor
     hid_send_feature_report(dev, buf, 264);
 }
 
-void HyperXAlloyFPSController::SendExtendedColor
-    (
-    unsigned char   profile,
-    unsigned char*  color_data
-    )
-{
-    unsigned char buf[264];
-
-    /*-----------------------------------------------------*\
-    | Zero out buffer                                       |
-    \*-----------------------------------------------------*/
-    memset(buf, 0x00, sizeof(buf));
-
-    /*-----------------------------------------------------*\
-    | Set up Color packet                                   |
-    \*-----------------------------------------------------*/
-    buf[0x00]   = 0x07;
-    buf[0x01]   = HYPERX_ALLOY_FPS_PACKET_ID_SET_COLOR;
-    buf[0x02]   = profile;
-    buf[0x03]   = HYPERX_ALLOY_FPS_COLOR_CHANNEL_EXTENDED;
-
-    /*-----------------------------------------------------*\
-    | Send packet                                           |
-    \*-----------------------------------------------------*/
-    hid_send_feature_report(dev, buf, 264);
-}
-
 void HyperXAlloyFPSController::SendDirect
     (
     unsigned char   color_channel,
@@ -441,30 +399,4 @@ void HyperXAlloyFPSController::SendDirect
     | Send packet                                           |
     \*-----------------------------------------------------*/
     hid_send_feature_report(dev, buf, 264);
-}
-
-void HyperXAlloyFPSController::SendDirectExtended
-    (
-    unsigned char*  color_data
-    )
-{
-    unsigned char buf[264];
-
-    /*-----------------------------------------------------*\
-    | Zero out buffer                                       |
-    \*-----------------------------------------------------*/
-    memset(buf, 0x00, sizeof(buf));
-
-    /*-----------------------------------------------------*\
-    | Set up Direct packet                                  |
-    \*-----------------------------------------------------*/
-    buf[0x00]   = 0x07;
-    buf[0x01]   = HYPERX_ALLOY_FPS_PACKET_ID_DIRECT;
-    buf[0x02]   = HYPERX_ALLOY_FPS_COLOR_CHANNEL_EXTENDED;
-    buf[0x03]   = 0xA0;
-
-    /*-----------------------------------------------------*\
-    | Send packet                                           |
-    \*-----------------------------------------------------*/
-    hid_send_feature_report(dev, buf, 264);
-}
+}
\ No newline at end of file
diff --git a/Controllers/HyperXKeyboardController/HyperXAlloyFPSController.h b/Controllers/HyperXKeyboardController/HyperXAlloyFPSController.h
index 751d70a4d..5dbae1736 100644
--- a/Controllers/HyperXKeyboardController/HyperXAlloyFPSController.h
+++ b/Controllers/HyperXKeyboardController/HyperXAlloyFPSController.h
@@ -58,8 +58,7 @@ enum
 {
     HYPERX_ALLOY_FPS_COLOR_CHANNEL_RED              = 0x01,
     HYPERX_ALLOY_FPS_COLOR_CHANNEL_GREEN            = 0x02,
-    HYPERX_ALLOY_FPS_COLOR_CHANNEL_BLUE             = 0x03,
-    HYPERX_ALLOY_FPS_COLOR_CHANNEL_EXTENDED         = 0x04
+    HYPERX_ALLOY_FPS_COLOR_CHANNEL_BLUE             = 0x03
 };
 
 class HyperXAlloyFPSController
@@ -120,20 +119,9 @@ private:
                 unsigned char*  color_data
                 );
 
-    void    SendExtendedColor
-                (
-                unsigned char   profile,
-                unsigned char*  color_data
-                );
-
     void    SendDirect
                 (
                 unsigned char   color_channel,
                 unsigned char*  color_data
                 );
-
-    void    SendDirectExtended
-                (
-                unsigned char*  color_data
-                );
 };