Fix - ESP8266 Platform Build

Author: brentru

src/components/gps/model.cpp

@@ -57,8 +57,7 @@ wippersnapper_gps_GPSConfig *GPSModel::GetGPSConfigMsg() {
 void GPSModel::CreateGPSEvent() {
   // Zero-out whatever was previously in the GPSEvent message
   memset(&_msg_gps_event, 0, sizeof(_msg_gps_event));
-  // Create new GPSEvent message with initializer
-  _msg_gps_event = wippersnapper_gps_GPSEvent_init_zero;
+  // Already zeroed out by memset, just set the counts
   _msg_gps_event.gga_responses_count = 0;
   _msg_gps_event.rmc_responses_count = 0;
 }
@@ -124,7 +123,7 @@ bool GPSModel::AddGpsEventRMC(wippersnapper_gps_GPSDateTime datetime,
 
   // Always store at index 0, overwriting any previous response
   wippersnapper_gps_GPSRMCResponse rmc_response;
-  rmc_response = wippersnapper_gps_GPSRMCResponse_init_zero;
+  memset(&rmc_response, 0, sizeof(rmc_response));
   rmc_response.has_datetime = true;
   rmc_response.datetime = datetime;
 
@@ -164,7 +163,7 @@ bool GPSModel::AddGpsEventGGA(wippersnapper_gps_GPSDateTime datetime,
     return false;
 
   wippersnapper_gps_GPGGAResponse gga_response;
-  gga_response = wippersnapper_gps_GPGGAResponse_init_zero;
+  memset(&gga_response, 0, sizeof(gga_response));
   gga_response.has_datetime = true;
   gga_response.datetime = datetime;
 

src/components/uart/drivers/drvUartPm25.h

@@ -57,8 +57,8 @@ class drvUartPm25 : public drvUartBase {
   drvUartPm25(SoftwareSerial *sw_serial,
               wippersnapper_uart_UartDeviceType device_type,
               const char *driver_name, uint32_t port_num)
-      : drvUartBase(sw_serial, device_type, driver_name, port_num) {
-    // Handled by drvUartBase constructor
+      : drvUartBase(sw_serial, driver_name, port_num) {
+    _device_type = device_type; // Set device type after base constructor
   }
 #endif // HAS_SW_SERIAL
 

src/components/uart/hardware.cpp

@@ -144,7 +144,13 @@ bool UARTHardware::ConfigureSerial() {
 #else
     // ESP8266, SAMD, and other platforms
     // take the default Arduino/Wiring API arguments
+    #ifdef ARDUINO_ARCH_ESP8266
+    // We want to pass the cfg as an enum for ESP8266
+    _hwSerial->begin((unsigned long)_config.baud_rate, (SerialConfig)cfg);
+    #else
+    // We want to cast the cfg for other platforms
     _hwSerial->begin((unsigned long)_config.baud_rate, (uint32_t)cfg);
+    #endif
 #endif
     _baud_rate = _config.baud_rate;
   }