From 3a3c13a143c9120bae648d1b5ddb8e12483e8695 Mon Sep 17 00:00:00 2001
From: Scott Powell <ripple_biz@protonmail.com>
Date: Tue, 15 Sep 2026 18:41:11 +1000
Subject: [PATCH] * refactor to support new Board::handleCommand(), and
 "get/set radio.fem.*" commands

---
 .../wio-tracker-l1-1w/WioTrackerL1Board.cpp   | 51 ++++++++++++++++++-
 .../wio-tracker-l1-1w/WioTrackerL1Board.h     | 10 ++--
 2 files changed, 55 insertions(+), 6 deletions(-)

diff --git a/variants/wio-tracker-l1-1w/WioTrackerL1Board.cpp b/variants/wio-tracker-l1-1w/WioTrackerL1Board.cpp
index 8210c24e..c6dbdbf5 100644
--- a/variants/wio-tracker-l1-1w/WioTrackerL1Board.cpp
+++ b/variants/wio-tracker-l1-1w/WioTrackerL1Board.cpp
@@ -15,7 +15,6 @@ void WioTrackerL1Board::begin() {
   pinMode(PIN_BUTTON4, INPUT_PULLUP);
   pinMode(PIN_BUTTON5, INPUT_PULLUP);
   pinMode(PIN_BUTTON6, INPUT_PULLUP);
-  
 
   #if defined(PIN_WIRE_SDA) && defined(PIN_WIRE_SCL)
     Wire.setPins(PIN_WIRE_SDA, PIN_WIRE_SCL);
@@ -32,9 +31,57 @@ void WioTrackerL1Board::begin() {
 
   delay(10);   // give sx1262 some time to power up
 }
-
+ 
 bool WioTrackerL1Board::setLoRaFemPaGainEnabled(bool enable) {
   _is_pa_enabled = enable;
   digitalWrite(SX126X_POWER_EN, enable ? HIGH : LOW);   // enable/disable the 1W PA
   return true;
 }
+
+void WioTrackerL1Board::attachDynamicPrefs(KeyValueStore* prefs) {
+  _prefs = prefs;
+
+  char gain[8];
+
+  gain[0] = 0;
+  _prefs->getByKey("fem_txgain", gain, 7);  // get initial values
+  setLoRaFemPaGainEnabled(strcmp(gain, "1") == 0);
+}
+
+bool WioTrackerL1Board::handleCommand(const char* command, uint32_t sender_timestamp, char* reply) {
+  if (strcmp(command, "get radio.fem.rxgain") == 0) {
+    strcpy(reply, "Error: unsupported");
+    return true;
+  }
+  if (memcmp(command, "set radio.fem.rxgain ", 21) == 0) {
+    strcpy(reply, "Error: unsupported");
+    return true;
+  }
+
+  if (strcmp(command, "get radio.fem.txgain") == 0) {
+    sprintf(reply, "> %s", isLoRaFemPaGainEnabled() ? "on" : "off");
+    return true;
+  }
+  if (memcmp(command, "set radio.fem.txgain ", 21) == 0) {
+    if (memcmp(&command[21], "on", 2) == 0) {
+      if (setLoRaFemPaGainEnabled(true)) {
+        _prefs->setByKey("fem_txgain", "1");
+        strcpy(reply, "OK - LoRa FEM TX gain on");
+      } else {
+        strcpy(reply, "Error: failed to apply LoRa FEM TX gain");
+      }
+    } else if (memcmp(&command[21], "off", 3) == 0) {
+      if (setLoRaFemPaGainEnabled(false)) {
+        _prefs->setByKey("fem_txgain", "0");
+        strcpy(reply, "OK - LoRa FEM TX gain off");
+      } else {
+        strcpy(reply, "Error: failed to apply LoRa FEM TX gain");
+      }
+    } else {
+      strcpy(reply, "Error: state must be on or off");
+    }
+    return true;
+  }
+
+  return false; // not handled
+}
diff --git a/variants/wio-tracker-l1-1w/WioTrackerL1Board.h b/variants/wio-tracker-l1-1w/WioTrackerL1Board.h
index c5a60944..798145db 100644
--- a/variants/wio-tracker-l1-1w/WioTrackerL1Board.h
+++ b/variants/wio-tracker-l1-1w/WioTrackerL1Board.h
@@ -8,10 +8,16 @@ class WioTrackerL1Board : public NRF52BoardDCDC {
 protected:
   uint8_t btn_prev_state;
   bool _is_pa_enabled = false;
+  KeyValueStore* _prefs = NULL;
+
+  bool setLoRaFemPaGainEnabled(bool enable);
+  bool isLoRaFemPaGainEnabled() const { return _is_pa_enabled; }
 
 public:
   WioTrackerL1Board() : NRF52Board("WioTrackerL1 OTA") {}
   void begin();
+  void attachDynamicPrefs(KeyValueStore* prefs);
+  bool handleCommand(const char* command, uint32_t sender_timestamp, char* reply) override;
 
 #if defined(P_LORA_TX_LED)
   void onBeforeTransmit() override {
@@ -35,10 +41,6 @@ public:
     return "Seeed Wio Tracker L1";
   }
 
-  bool canControlLoRaFemPaGain() const override { return true; }
-  bool setLoRaFemPaGainEnabled(bool enable) override;
-  bool isLoRaFemPaGainEnabled() const override { return _is_pa_enabled; }
-
   void powerOff() override {
     setLoRaFemPaGainEnabled(false);   // turn Off PA
     NRF52Board::powerOff();