V1.7.07 - Updates

- Eliminated all warnings under PlatformIO compilation for all boards

Author: ClutchplateDude

Software/Arduino code/OpenAstroTracker/InterruptCallback.cpp

@@ -46,6 +46,8 @@ bool InterruptCallback::setInterval(float intervalMs, interrupt_callback_p callb
 
   // This timer library requires microsecond interval definitions
   interruptHandler.setInterval(1000.0f * intervalMs, esp8266callback);
+
+  return true;
 }
 
 void InterruptCallback::start()
@@ -89,6 +91,7 @@ bool InterruptCallback::setInterval(float intervalMs, interrupt_callback_p callb
     Serial.println("Setup ESP32 Timer");
 #endif // DEBUG_MODE
 
+  return true;
 }
 
 void InterruptCallback::stop(){
@@ -112,7 +115,7 @@ void InterruptCallback::start(){
 
 #elif defined __AVR_ATmega328P__ || defined __AVR_ATmega2560__
 
-  bool InterruptCallback::setInterval(float intervalMs, interrupt_callback_p callback, void* payload)
+bool InterruptCallback::setInterval(float intervalMs, interrupt_callback_p callback, void* payload)
 {
   // We have requested to use Timer2 (see above)
   ITimer2.init();

Software/Arduino code/OpenAstroTracker/Mount.cpp

@@ -58,7 +58,7 @@ const char* formatStringsRA[] = {
 // This is the callback function for the timer interrupt. It does very minimal work,
 // only stepping the stepper motors as needed.
 /////////////////////////////////
-void IRAM_ATTR mountLoop(void* payload) {
+void mountLoop(void* payload) {
   Mount* mount = reinterpret_cast<Mount*>(payload);
   mount->interruptLoop();
 }
@@ -1120,7 +1120,7 @@ void Mount::delay(int ms) {
 //
 // This function is run in an ISR. It needs to be fast and do little work.
 /////////////////////////////////
-void IRAM_ATTR Mount::interruptLoop()
+void Mount::interruptLoop()
 {
   if (_mountStatus & STATUS_GUIDE_PULSE) {
     if (_mountStatus & STATUS_GUIDE_PULSE_RA) {

Software/Arduino code/OpenAstroTracker/OpenAstroTracker.hpp

@@ -1,7 +1,7 @@
 /*
   =======================================================================================================================================
 
-    Version 1.6
+    Version 1.7
 
     1. Connect your Arduino, under tools choose "Arduino Uno", set the right Port and set "Arduino ISP" as the Programmer.
     2. Hit upload (Ctrl-U)
@@ -16,7 +16,7 @@
 */
 #include "Globals.hpp"
 
-String version = "V1.7.06";
+String version = "V1.7.07";
 
 ///////////////////////////////////////////////////////////////////////////
 // Please see the Globals.h file for configuration of the firmware.

Software/Arduino code/OpenAstroTracker/WifiControl.hpp

@@ -2,8 +2,8 @@
 #include "Globals.hpp"
 
 #ifdef WIFI_ENABLED
-#include "Mount.hpp";
-#include "LcdMenu.hpp";
+#include "Mount.hpp"
+#include "LcdMenu.hpp"
 #include "MeadeCommandProcessor.hpp"
 #include "WiFiServer.h"
 #include "WiFiUDP.h"
@@ -39,8 +39,8 @@ class WifiControl {
     WiFiUDP* _udp;
     WiFiClient client;
 
-    long _infraStart = 0;
-    int _infraWait = 30000; // 30 second timeout for 
+    unsigned long _infraStart = 0;
+    unsigned long _infraWait = 30000; // 30 second timeout for 
 };
 
 extern WifiControl wifiControl;

Software/Arduino code/OpenAstroTracker/b_setup.hpp

@@ -24,7 +24,6 @@ void finishSetup();
 #if defined(ESP32) && !defined(RUN_STEPPERS_IN_MAIN_LOOP)
 // Forward declare the two functions we run in the main loop
 void serialLoop();
-void finishSetup();
 
 TaskHandle_t StepperTask;
 TaskHandle_t  CommunicationsTask;
@@ -134,8 +133,9 @@ void finishSetup()
   lcdMenu.printMenu("OpenAstroTracker");
   lcdMenu.setCursor(0, 1);
   lcdMenu.printMenu("     " + version);
+#ifndef HEADLESS_CLIENT
   unsigned long now = millis();
-
+#endif
   // Create the command processor singleton
   MeadeCommandProcessor::createProcessor(&mount, &lcdMenu);
 

Software/Arduino code/OpenAstroTracker/c_buttons.hpp

@@ -16,10 +16,11 @@ byte lastKey = btnNONE;
 
 #endif
 
+byte lcd_key;
+
 void loop() {
 
 #ifdef LCD_BUTTON_TEST
-  byte lcd_key;
   int adc_key_in;
 
   lcdMenu.setCursor(0, 0);