Merge pull request #105 from OpenAstroTech/remove-obsolete-boards

Removed support for esp8266, Arduino UNO, integrated heating

Author: ClutchplateDude

Software/Arduino code/OpenAstroTracker/Configuration_adv.hpp

@@ -190,29 +190,21 @@
 #endif  // DISPLAY_TYPE
 
 // Enable Meade protocol communication over serial
-#ifdef __AVR_ATmega328P__
-#define SUPPORT_SERIAL_CONTROL 0
-#else 
 #define SUPPORT_SERIAL_CONTROL 1
-#endif
 
 
-#if defined(ESP8266) || defined(ESP32)
+#if defined(ESP32)
 /////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////
 //                //////////
 // WIFI SETTINGS  //////////
 //                //////////
 ////////////////////////////
-// These settings are valid only for ESP8266 or ESP32
+// These settings are valid only for ESP32
 //
 // Define some things, dont change: ///
 #define ESPBOARD
 // #define BLUETOOTH_ENABLED
 #define WIFI_ENABLED 
-#if defined(ESP8266) 
-  #undef RUN_STEPPERS_IN_MAIN_LOOP
-  #define RUN_STEPPERS_IN_MAIN_LOOP 1
-#endif
 ///////////////////////////////////////
 //
 // SETTINGS
@@ -293,10 +285,3 @@
 #if RA_STEPPER_TYPE != STEPPER_TYPE_28BYJ48 && ESP32
 #error "Sorry, ESP32 only supports 28BYJ48 steppers at this moment"
 #endif
-#if RA_STEPPER_TYPE != STEPPER_TYPE_28BYJ48 && __AVR_ATmega328P__
-#error "Sorry, Arduino Uno only supports 28BYJ48 steppers."
-#endif
-
-// Set this to 1 this to enable the heating menu
-// NOTE: Heating is currently not supported!
-#define SUPPORT_HEATING 0

Software/Arduino code/OpenAstroTracker/Configuration_pins.hpp

@@ -99,29 +99,3 @@
         #error GPS module not currently configured/supported in ESP32
     #endif
 #endif
-
-////////////////////////////////////////////////////////////////////
-// Arduino UNO
-////////////////////////////////////////////////////////////////////
-#ifdef __AVR_ATmega328P__ // Arduino Uno
-  #if (RA_STEPPER_TYPE != STEPPER_TYPE_28BYJ48) || (DEC_STEPPER_TYPE != STEPPER_TYPE_28BYJ48)
-    #error Only 28BJY-48 stepper motors are supported on the UNO (for now)
-  #endif
-
-  #define RA_IN1_PIN  12
-  #define RA_IN2_PIN  11
-  #define RA_IN3_PIN  3
-  #define RA_IN4_PIN  2
-
-  #define DEC_IN1_PIN 18
-  #define DEC_IN2_PIN 17
-  #define DEC_IN3_PIN 16
-  #define DEC_IN4_PIN 15
-
-  #if AZIMUTH_ALTITUDE_MOTORS == 1
-    #error Azimuth / Altitude motors not currently configured/supported in ESP32
-  #endif
-  #if USE_GPS == 1
-    #error GPS module not currently configured/supported in ESP32
-  #endif
-#endif

Software/Arduino code/OpenAstroTracker/platformio.ini

@@ -31,19 +31,6 @@ upload_protocol = wiring
 lib_deps = 
 	${common.lib_deps}
 
-[env:uno]	
-platform = atmelavr	
-board = uno	
-lib_deps = 
-	${common.lib_deps}
-
-[env:esp8266]	
-platform = espressif8266	
-board = esp12e
-lib_deps = 
-	${common.lib_deps}
-	ESP8266WiFi
-
 [env:esp32]
 platform = espressif32
 board = esp32dev

Software/Arduino code/OpenAstroTracker/src/Core.cpp

@@ -8,7 +8,6 @@
 #include "c725_menuHOME.hpp"
 #include "c72_menuHA.hpp"
 #include "c72_menuHA_GPS.hpp"
-#include "c74_menuHEAT.hpp"
 #include "c75_menuCTRL.hpp"
 #include "c76_menuCAL.hpp"
 #include "c78_menuINFO.hpp"

Software/Arduino code/OpenAstroTracker/src/ESP8266TimerInterrupt.h

@@ -6,12 +6,9 @@
 // whatever timer is used for the hardware being run
 //////////////////////////////////////
 
-// NOTE: ESP8266 support is not complete and does not work. This code is never called.
-#ifdef ESP8266
-  #include "ESP8266TimerInterrupt.h"
-#elif defined ESP32
+#if defined ESP32
   // We don't support ESP32 boards in interrupt mode
-#elif defined __AVR_ATmega328P__ || defined __AVR_ATmega2560__   // Arduino Uno or Mega
+#elif defined __AVR_ATmega2560__   // Arduino Mega
   #define USE_TIMER_1     true
   #define USE_TIMER_2     true
   #define USE_TIMER_3     false
@@ -22,43 +19,7 @@
   #error Unrecognized board selected. Either implement interrupt code or define the board here.
 #endif
 
-#ifdef ESP8266
-ESP8266Timer interruptHandler;
-
-void* actualPayload = NULL;
-volatile interrupt_callback_p actualCallback = NULL;
-
-// This timer only supports a callback with no payload
-void ICACHE_RAM_ATTR esp8266callback(void)
-{
-  if (actualCallback != NULL) {
-    (*actualCallback)(actualPayload);
-  }
-}
-
-bool InterruptCallback::setInterval(float intervalMs, interrupt_callback_p callback, void* payload)
-{
-  // Since this timer only supports a callback with no payload, we store the requested callback and payload
-  // in static variables and install an intermediate callback for the interrupt to call. The intermerdiate
-  // The calls the actual requested callback with the payload.
-  actualPayload = payload;
-  actualCallback = callback;
-
-  // This timer library requires microsecond interval definitions
-  interruptHandler.setInterval(1000.0f * intervalMs, esp8266callback);
-
-  return true;
-}
-
-void InterruptCallback::start()
-{
-}
-
-void InterruptCallback::stop()
-{
-}
-
-#elif defined(ESP32)
+#if defined(ESP32)
 
 /*
 volatile bool _lock = false;
@@ -107,7 +68,7 @@ void InterruptCallback::start(){
 }
 */
 
-#elif defined __AVR_ATmega328P__ || defined __AVR_ATmega2560__
+#elif defined __AVR_ATmega2560__
 
 bool InterruptCallback::setInterval(float intervalMs, interrupt_callback_p callback, void* payload)
 {

Software/Arduino code/OpenAstroTracker/src/InterruptCallback.cpp

@@ -365,7 +365,7 @@ bool gpsAqcuisitionComplete(int & indicator); // defined in c72_menuHA_GPS.hpp
 // :XGM#
 //      Get Mount configuration settings 
 //      Returns: <board>,<RA Stepper Info>,<DEC Stepper Info>,<GPS info>,<AzAlt info>,<Gyro info>#
-//      Where <board> is one of the supported boards (currently Uno, Mega, ESP8266, ESP32)
+//      Where <board> is one of the supported boards (currently Mega, ESP32)
 //            <Stepper Info> is a pipe-delimited string of Motor type (NEMA or 28BYJ), Pulley Teeth, Steps per revolution)
 //            <GPS info> is either NO_GPS or GPS, depending on whether a GPS module is present
 //            <AzAlt info> is either NO_AZ_ALT or AUTO_AZ_ALT, depending on whether the AutoPA stepper motors are present
@@ -374,7 +374,7 @@ bool gpsAqcuisitionComplete(int & indicator); // defined in c72_menuHA_GPS.hpp
 //
 // :XGN#
 //      Get network settings
-//      Gets the current status of the Wifi connection. Reply only available when running on ESP8266 boards.
+//      Gets the current status of the Wifi connection. Reply only available when running on ESP boards.
 //      Returns: 1,<stats>,<hostname>,<ip>:<port>,<SSID>,<OATHostname>#     - if Wifi is enabled
 //      Returns: 0,#     - if Wifi is not enabled
 //