Make power button work. Detect CoAuth if it's alone on the bus.

Author: nseidle

Firmware/RTK_Everywhere/Begin.ino

@@ -94,7 +94,7 @@ void identifyBoard()
         // 0x08 - HUSB238 - USB C PD Sink Controller
         bool husb238Present = i2cIsDevicePresent(i2c_0, 0x08);
 
-        // 0x10 -MFI343S00177 Authentication Coprocessor
+        // 0x10 - MFI343S00177 Authentication Coprocessor
         bool mfiPresent = i2cIsDevicePresent(i2c_0, 0x10);
 
         i2c_0->end();
@@ -447,7 +447,6 @@ void beginBoard()
         // Facet V2 is based on the ESP32-WROVER
         // ZED-F9P is interfaced via I2C and UART1
         // NEO-D9S is interfaced via I2C. UART2 TX is also connected to ESP32 pin 4
-        // TODO: pass PMP over serial to save I2C traffic?
 
         // Specify the GNSS radio
 #ifdef COMPILE_ZED
@@ -872,6 +871,8 @@ void beginBoard()
         present.beeper = true;
         present.gnss_to_uart = true;
         present.needsExternalPpl = true; // Uses the PointPerfect Library
+        present.fastPowerOff = true;
+        present.invertedFastPowerOff = true; // Driving PWRKILL high will cause powerdown
 
         // We can't enable GNSS features here because we don't know if lg290pFirmwareVersion is >= v05
         // present.minElevation = true;
@@ -884,7 +885,7 @@ void beginBoard()
         pin_GnssUart_TX = 17;
         pin_GNSS_DR_Reset = 22; // Push low to reset GNSS/DR.
 
-        pin_GNSS_TimePulse = 39; // PPS on UM980
+        pin_GNSS_TimePulse = 39; // PPS on LG290P
 
         pin_usbSelect = 12;          // Controls U18 switch between ESP UART0 to USB or GNSS UART1
         pin_powerAdapterDetect = 36; // Goes low when USB cable is plugged in
@@ -896,12 +897,14 @@ void beginBoard()
         pin_beeper = 33;
 
         pin_powerButton = 34;
-        // pin_powerSenseAndControl = 18; // PWRKILL
+        pin_powerFastOff = 18; // PWRKILL
 
         pin_loraRadio_power = 19; // LoRa_EN
         // pin_loraRadio_boot = 23;  // LoRa_BOOT0
         // pin_loraRadio_reset = 5;  // LoRa_NRST
 
+        pinMode(pin_powerFastOff, INPUT); // Leave this as an input. powerDown() will drive high for fast power off
+
         DMW_if systemPrintf("pin_bluetoothStatusLED: %d\r\n", pin_bluetoothStatusLED);
         pinMode(pin_bluetoothStatusLED, OUTPUT);
 
@@ -1808,6 +1811,12 @@ bool i2cBusInitialization(TwoWire *i2cBus, int sda, int scl, int clockKHz)
         // SDA/VCC shorted: 1000ms, response 5
         // SDA/GND shorted: 14ms, response 5
         timer = millis();
+
+        // If there is nothing else on the bus, the authentication coprocessor can be asleep
+        // Ping it twice to be sure
+        if (addr == 0x10)
+            i2cIsDevicePresent(i2cBus, addr); // Throw away result. Just wake it up.
+
         if (i2cIsDevicePresent(i2cBus, addr))
         {
             if (deviceFound == false)

Firmware/RTK_Everywhere/menuMessages.ino

@@ -809,9 +809,13 @@ void checkGNSSArrayDefaults()
 #ifdef COMPILE_LG290P
     else if (present.gnss_lg290p)
     {
+        // This setting is not supported on the Torch X2 nor is it in the command array
+        // so it does not get used nor recorded to NVM leading to the defaults being
+        // applied at every boot. 
         if (settings.enableExtCorrRadio == 254)
         {
-            defaultsApplied = true;
+            if(productVariant != RTK_TORCH_X2) // Prevent defaults from being applied if this *is* a Torch X2
+                defaultsApplied = true;
             settings.enableExtCorrRadio = false;
         }
 

Firmware/RTK_Everywhere/settings.h

@@ -1779,13 +1779,13 @@ const RTK_Settings_Entry rtkSettingsEntries[] =
 //    i  d  i  v  V  i  c  n  r  e    X
 //    g  s  x  k  2  c  h  d  d  x    2  Type       Qual                Variable                  Name              afterSetCmd
 
-    // UM980 GNSS Receiver - TODO these apply to more than UM980
     { 1, 1, 0, 0, 0, 0, 1, 0, 1, HAS, 1, _bool,     0, & settings.enableGalileoHas, "enableGalileoHas", nullptr, },
     { 1, 1, 0, 0, 0, 0, 1, 0, 0, ALL, 0, _bool,     3, & settings.enableMultipathMitigation, "enableMultipathMitigation", nullptr, },
     { 0, 0, 0, 0, 0, 0, 1, 0, 0, ALL, 0, _bool,     0, & settings.enableImuCompensationDebug, "enableImuCompensationDebug", nullptr, },
     { 0, 0, 0, 0, 0, 0, 1, 0, 0, ALL, 0, _bool,     0, & settings.enableImuDebug, "enableImuDebug", nullptr, },
     { 1, 1, 0, 0, 0, 0, 1, 0, 0, ALL, 0, _bool,     0, & settings.enableTiltCompensation, "enableTiltCompensation", nullptr, },
 
+    // UM980 GNSS Receiver
 #ifdef  COMPILE_UM980
     { 1, 1, 1, 0, 0, 0, 1, 0, 0, U98, 0, tUmConst,  MAX_UM980_CONSTELLATIONS, & settings.um980Constellations, "constellation_", gnssCmdUpdateConstellations, },
     { 0, 1, 1, 0, 0, 0, 1, 0, 0, U98, 0, tUmMRNmea, MAX_UM980_NMEA_MSG, & settings.um980MessageRatesNMEA, "messageRateNMEA_", gnssCmdUpdateMessageRates, },