From 4d8deeb5044101350847b65bf7846c941f713085 Mon Sep 17 00:00:00 2001
From: Dan <UnrealDanMail@gmail.com>
Date: Thu, 5 Dec 2024 14:55:34 -0800
Subject: [PATCH] fixes for spark

---
 .../src/Menus/MenuList/ModeSharing.cpp        | 58 ++++++-------------
 VortexEngine/src/Menus/MenuList/ModeSharing.h |  6 +-
 VortexEngine/src/Wireless/IRConfig.h          |  4 +-
 VortexEngine/src/Wireless/VLConfig.h          |  4 +-
 4 files changed, 25 insertions(+), 47 deletions(-)

diff --git a/VortexEngine/src/Menus/MenuList/ModeSharing.cpp b/VortexEngine/src/Menus/MenuList/ModeSharing.cpp
index 1906e4c573..9af54dda5c 100644
--- a/VortexEngine/src/Menus/MenuList/ModeSharing.cpp
+++ b/VortexEngine/src/Menus/MenuList/ModeSharing.cpp
@@ -34,10 +34,14 @@ bool ModeSharing::init()
   }
   // skip led selection
   m_ledSelected = true;
-  // start on receive because it's the more responsive of the two
-  // the odds of opening receive and then accidentally receiving
-  // a mode that is being broadcast nearby is completely unlikely
-  beginReceivingVL();
+  if (m_advanced) {
+    // start on receive because it's the more responsive of the two
+    // the odds of opening receive and then accidentally receiving
+    // a mode that is being broadcast nearby is completely unlikely
+    beginReceivingVL();
+  } else {
+    beginReceivingIR();
+  }
   DEBUG_LOG("Entering Mode Sharing");
   return true;
 }
@@ -78,7 +82,7 @@ Menu::MenuAction ModeSharing::run()
 }
 
 // handlers for clicks
-void ModeSharing::onShortClickM()
+void ModeSharing::onShortClick()
 {
   switch (m_sharingMode) {
   case ModeShareState::SHARE_RECEIVE_VL:
@@ -104,15 +108,9 @@ void ModeSharing::onShortClickM()
   Leds::clearAll();
 }
 
-// handlers for clicks
-void ModeSharing::onShortClickL()
-{
-  switchVLIR();
-}
-
-void ModeSharing::onShortClickR()
+void ModeSharing::onLongClick()
 {
-  switchVLIR();
+  leaveMenu();
 }
 
 void ModeSharing::switchVLIR()
@@ -132,12 +130,6 @@ void ModeSharing::switchVLIR()
   Leds::clearAll();
 }
 
-void ModeSharing::onLongClickM()
-{
-  Modes::updateCurMode(&m_previewMode);
-  leaveMenu(true);
-}
-
 void ModeSharing::beginSendingIR()
 {
   // if the sender is sending then cannot start again
@@ -208,11 +200,9 @@ void ModeSharing::receiveModeIR()
     return;
   }
   DEBUG_LOGF("Success receiving mode: %u", m_previewMode.getPatternID());
-  if (!m_advanced) {
-    Modes::updateCurMode(&m_previewMode);
-    // leave menu and save settings, even if the mode was the same whatever
-    leaveMenu(true);
-  }
+  Modes::updateCurMode(&m_previewMode);
+  // leave menu and save settings, even if the mode was the same whatever
+  leaveMenu(true);
 }
 
 void ModeSharing::beginSendingVL()
@@ -275,11 +265,9 @@ void ModeSharing::receiveModeVL()
     return;
   }
   DEBUG_LOGF("Success receiving mode: %u", m_previewMode.getPatternID());
-  if (!m_advanced) {
-    Modes::updateCurMode(&m_previewMode);
-    // leave menu and save settings, even if the mode was the same whatever
-    leaveMenu(true);
-  }
+  Modes::updateCurMode(&m_previewMode);
+  // leave menu and save settings, even if the mode was the same whatever
+  leaveMenu(true);
 }
 
 void ModeSharing::showSendModeVL()
@@ -302,11 +290,7 @@ void ModeSharing::showReceiveModeVL()
     Leds::clearAll();
     Leds::setRange(LED_FIRST, (LedPos)(VLReceiver::percentReceived() / 10), RGBColor(0, 1, 0));
   } else {
-    if (m_advanced) {
-      m_previewMode.play();
-    } else {
-      Leds::setAll(0x010101);
-    }
+    Leds::setAll(0x010101);
   }
 }
 
@@ -317,10 +301,6 @@ void ModeSharing::showReceiveModeIR()
     Leds::clearAll();
     Leds::setRange(LED_FIRST, (LedPos)(VLReceiver::percentReceived() / 10), RGBColor(0, 1, 0));
   } else {
-    if (m_advanced) {
-      m_previewMode.play();
-    } else {
-      Leds::setAll(RGB_CYAN0);
-    }
+    Leds::setAll(RGB_CYAN0);
   }
 }
diff --git a/VortexEngine/src/Menus/MenuList/ModeSharing.h b/VortexEngine/src/Menus/MenuList/ModeSharing.h
index ed15e7c8af..aae4198d81 100644
--- a/VortexEngine/src/Menus/MenuList/ModeSharing.h
+++ b/VortexEngine/src/Menus/MenuList/ModeSharing.h
@@ -13,10 +13,8 @@ class ModeSharing : public Menu
   MenuAction run() override;
 
   // handlers for clicks
-  void onShortClickM() override;
-  void onShortClickL() override;
-  void onShortClickR() override;
-  void onLongClickM() override;
+  void onShortClick() override;
+  void onLongClick() override;
 
 private:
   void beginSendingVL();
diff --git a/VortexEngine/src/Wireless/IRConfig.h b/VortexEngine/src/Wireless/IRConfig.h
index 2a8e68ddf2..e763b6b17d 100644
--- a/VortexEngine/src/Wireless/IRConfig.h
+++ b/VortexEngine/src/Wireless/IRConfig.h
@@ -39,7 +39,7 @@
 #define IR_DIVIDER_SPACE_MIN IR_HEADER_MARK_MIN
 #define IR_DIVIDER_SPACE_MAX IR_HEADER_MARK_MAX
 
-#define IR_SEND_PWM_PIN 0
-#define IR_RECEIVER_PIN 2
+#define IR_SEND_PWM_PIN 3
+#define IR_RECEIVER_PIN 4
 
 #endif
diff --git a/VortexEngine/src/Wireless/VLConfig.h b/VortexEngine/src/Wireless/VLConfig.h
index 00438aea9a..a2a310c6f3 100644
--- a/VortexEngine/src/Wireless/VLConfig.h
+++ b/VortexEngine/src/Wireless/VLConfig.h
@@ -8,7 +8,7 @@
 // Whether to enable the Visible Light system as a whole
 //
 #define VL_ENABLE_SENDER          1
-#define VL_ENABLE_RECEIVER        0
+#define VL_ENABLE_RECEIVER        1
 
 // the size of IR blocks in bits
 #define VL_DEFAULT_BLOCK_SIZE 256
@@ -42,6 +42,6 @@
 #define VL_DIVIDER_SPACE_MAX VL_HEADER_MARK_MAX
 
 #define VL_SEND_PWM_PIN 0
-#define VL_RECEIVER_PIN 0
+#define VL_RECEIVER_PIN 1
 
 #endif