From 320129633d9cab92cdf1c28da270155c256d83bc Mon Sep 17 00:00:00 2001
From: Adam Honse <calcprogrammer1@gmail.com>
Date: Mon, 20 Jul 2020 17:49:12 -0500
Subject: [PATCH] Add SMBus detection mode selection box

---
 qt/OpenRGBSystemInfoPage.cpp | 22 +++++++++-
 qt/OpenRGBSystemInfoPage.ui  | 85 +++++++++++++++++++++---------------
 2 files changed, 72 insertions(+), 35 deletions(-)

diff --git a/qt/OpenRGBSystemInfoPage.cpp b/qt/OpenRGBSystemInfoPage.cpp
index 435a581e..45f80572 100644
--- a/qt/OpenRGBSystemInfoPage.cpp
+++ b/qt/OpenRGBSystemInfoPage.cpp
@@ -28,6 +28,12 @@ OpenRGBSystemInfoPage::OpenRGBSystemInfoPage(std::vector<i2c_smbus_interface *>&
     }
 
     ui->SMBusAdaptersBox->setCurrentIndex(0);
+
+    ui->SMBusDetectionModeBox->addItem("Auto");
+    ui->SMBusDetectionModeBox->addItem("Quick");
+    ui->SMBusDetectionModeBox->addItem("Read");
+
+    ui->SMBusDetectionModeBox->setCurrentIndex(0);
 }
 
 OpenRGBSystemInfoPage::~OpenRGBSystemInfoPage()
@@ -47,7 +53,21 @@ void Ui::OpenRGBSystemInfoPage::on_DetectButton_clicked()
     if(busses.size() > current_index)
     {
         i2c_smbus_interface* bus = busses[current_index];
-        ui->SMBusDataText->setPlainText(i2c_detect(bus, MODE_QUICK).c_str());
+
+        switch(ui->SMBusDetectionModeBox->currentIndex())
+        {
+        case 0:
+            ui->SMBusDataText->setPlainText(i2c_detect(bus, MODE_AUTO).c_str());
+            break;
+
+        case 1:
+            ui->SMBusDataText->setPlainText(i2c_detect(bus, MODE_QUICK).c_str());
+            break;
+
+        case 2:
+            ui->SMBusDataText->setPlainText(i2c_detect(bus, MODE_READ).c_str());
+            break;
+        }
     }
 }
 
diff --git a/qt/OpenRGBSystemInfoPage.ui b/qt/OpenRGBSystemInfoPage.ui
index ce1795d7..a6edfd64 100644
--- a/qt/OpenRGBSystemInfoPage.ui
+++ b/qt/OpenRGBSystemInfoPage.ui
@@ -14,38 +14,10 @@
    <string>Frame</string>
   </property>
   <layout class="QGridLayout" name="gridLayout">
-   <item row="0" column="0">
-    <widget class="QLabel" name="SMBusAdaptersLabel">
-     <property name="text">
-      <string>SMBus Adapters:</string>
-     </property>
-    </widget>
+   <item row="3" column="0" colspan="4">
+    <widget class="QPlainTextEdit" name="SMBusDataText"/>
    </item>
-   <item row="0" column="1" colspan="2">
-    <widget class="QComboBox" name="SMBusAdaptersBox"/>
-   </item>
-   <item row="0" column="3">
-    <widget class="QPushButton" name="DetectButton">
-     <property name="text">
-      <string>Detect Devices</string>
-     </property>
-    </widget>
-   </item>
-   <item row="1" column="0">
-    <widget class="QLabel" name="SMBusDumpLabel">
-     <property name="text">
-      <string>SMBus Dumper:</string>
-     </property>
-    </widget>
-   </item>
-   <item row="1" column="1">
-    <widget class="QLabel" name="DumpAddressLabel">
-     <property name="text">
-      <string>Address:</string>
-     </property>
-    </widget>
-   </item>
-   <item row="1" column="2">
+   <item row="2" column="2">
     <widget class="QSpinBox" name="DumpAddressBox">
      <property name="suffix">
       <string/>
@@ -61,15 +33,60 @@
      </property>
     </widget>
    </item>
-   <item row="1" column="3">
+   <item row="2" column="3">
     <widget class="QPushButton" name="DumpButton">
      <property name="text">
       <string>Dump Device</string>
      </property>
     </widget>
    </item>
-   <item row="2" column="0" colspan="4">
-    <widget class="QPlainTextEdit" name="SMBusDataText"/>
+   <item row="2" column="1">
+    <widget class="QLabel" name="DumpAddressLabel">
+     <property name="text">
+      <string>Address:</string>
+     </property>
+    </widget>
+   </item>
+   <item row="2" column="0">
+    <widget class="QLabel" name="SMBusDumpLabel">
+     <property name="text">
+      <string>SMBus Dumper:</string>
+     </property>
+    </widget>
+   </item>
+   <item row="0" column="0">
+    <widget class="QLabel" name="SMBusAdaptersLabel">
+     <property name="text">
+      <string>SMBus Adapters:</string>
+     </property>
+    </widget>
+   </item>
+   <item row="1" column="3">
+    <widget class="QPushButton" name="DetectButton">
+     <property name="text">
+      <string>Detect Devices</string>
+     </property>
+    </widget>
+   </item>
+   <item row="0" column="1" colspan="3">
+    <widget class="QComboBox" name="SMBusAdaptersBox"/>
+   </item>
+   <item row="1" column="0">
+    <widget class="QLabel" name="SMBusDetectorLabel">
+     <property name="text">
+      <string>SMBus Detector:</string>
+     </property>
+    </widget>
+   </item>
+   <item row="1" column="1">
+    <widget class="QLabel" name="SMBusDetectModeLabel">
+     <property name="text">
+      <string>Detection Mode:</string>
+     </property>
+    </widget>
+   </item>
+   <item row="1" column="2">
+    <widget class="QComboBox" name="SMBusDetectionModeBox"/>
    </item>
   </layout>
  </widget>