Rev 0.6: Firmware fix: one-button mode initial frequency glitch due to potentiometer capacitor

Author: ScrapComputing

README.md

@@ -239,7 +239,7 @@ So I don't think there is a way to get the Throttle Blaster to work universally
 
 
 # Change Log
-- Rev 0.6: Firmware bug fixes: not saving MHz/Period adjustments to flash.
+- Rev 0.6: Firmware bug fixes: (i) fix saving MHz/Period adjustments to flash and (ii) one-button mode frequency glitch.
 - Rev 0.5: Replaces potentiometer capacitor with SMD and several firmware fixes.
 - Rev 0.4: Reverses knob direction and adds jumper JP3 for selecting direction.
 - Rev 0.3: Adds UART support in both firmware and PCB.

firmware/src/CMakeLists.txt

@@ -19,7 +19,7 @@ cmake_minimum_required(VERSION 3.13)
 # For example: minicom -b 115200 -D /dev/ttyACM0. Serial connection: 115200 8N1
 
 set(REVISION_MAJOR 0)
-set(REVISION_MINOR 4)
+set(REVISION_MINOR 6)
 
 message("PICO_SDK_PATH = ${PICO_SDK_PATH}")
 

firmware/src/PotentiometerLogic.cpp

@@ -32,8 +32,10 @@ void PotentiometerLogic::manual(int PotVal, ButtonState BtnState) {
     Disp.setFlash(true);
     return;
   }
-  int MaxKHz = Presets.getMaxKHz();
-  DC.setKHz(PotVal * MaxKHz / PotMaxVal);
+  if (EnablePot) {
+    int MaxKHz = Presets.getMaxKHz();
+    DC.setKHz(PotVal * MaxKHz / PotMaxVal);
+  }
 }
 
 void PotentiometerLogic::cyclePresets(int PotVal, ButtonState BtnState) {

firmware/src/main.cpp

@@ -51,7 +51,7 @@ const char *getUIModeStr(UIMode Mode) {
   case UIMode::TwoButton:
     return "TwoButton";
   case UIMode::Button:
-    return "Button";
+    return "OneButton";
   case UIMode::ButtonWithPot:
     return "ButtonWithPot";
   }