Updated the example

Author: PaulZC

examples/Example20_SendCustomCommand/Example20_SendCustomCommand.ino

@@ -85,11 +85,10 @@ void setup()
   // SFE_UBLOX_STATUS_DATA_RECEIVED if the data we requested was read / polled successfully
   // SFE_UBLOX_STATUS_DATA_SENT     if the data we sent was writted successfully (ACK'd)
   // Other values indicate errors. Please see the sfe_ublox_status_e enum for further details.
-  // If you see a failure you can of course simply try sending the same command again.
 
   // Referring to the u-blox M8 Receiver Description and Protocol Specification we see that
   // the dynamic model is configured using the UBX-CFG-NAV5 message. So let's load our
-  // custom packet with the correct information so we can read (poll) the current settings.
+  // custom packet with the correct information so we can read (poll / get) the current settings.
 
   customCfg.cls = UBX_CLASS_CFG; // This is the message Class
   customCfg.id = UBX_CFG_NAV5; // This is the message ID
@@ -102,18 +101,9 @@ void setup()
   // Now let's read the current navigation model settings. The results will be loaded into customCfg.
   if (myGPS.sendCommand(&customCfg, maxWait) != SFE_UBLOX_STATUS_DATA_RECEIVED) // We are expecting data and an ACK
   {
-    Serial.println(F("sendCommand (poll) failed! Trying again..."));
-    // We need to reset the packet before we try again as the values could have changed
-    customCfg.cls = UBX_CLASS_CFG;
-    customCfg.id = UBX_CFG_NAV5;
-    customCfg.len = 0;
-    customCfg.startingSpot = 0;
-    if (myGPS.sendCommand(&customCfg, maxWait) != SFE_UBLOX_STATUS_DATA_RECEIVED) // We are expecting data and an ACK
-    {
-      Serial.println(F("sendCommand (poll) failed again! Freezing."));
-      while (1)
-        ;
-    }
+    Serial.println(F("sendCommand (poll) failed! Freezing..."));
+    while (1)
+      ;
   }
 
   // Referring to the message definition for UBX-CFG-NAV5 we see that we need to change

src/SparkFun_Ublox_Arduino_Library.cpp

@@ -128,7 +128,7 @@ const char *SFE_UBLOX_GPS::statusString(sfe_ublox_status_e stat)
     return "Timeout";
     break;
   case SFE_UBLOX_STATUS_COMMAND_UNKNOWN:
-    return "Command Unknown";
+    return "Command Unknown (NACK)";
     break;
   case SFE_UBLOX_STATUS_OUT_OF_RANGE:
     return "Out of range";
@@ -154,6 +154,9 @@ const char *SFE_UBLOX_GPS::statusString(sfe_ublox_status_e stat)
   case SFE_UBLOX_STATUS_I2C_COMM_FAILURE:
     return "I2C Comm Failure";
     break;
+  case SFE_UBLOX_STATUS_DATA_OVERWRITTEN:
+    return "Data Packet Overwritten";
+    break;
   default:
     return "Unknown Status";
     break;
@@ -2059,18 +2062,11 @@ boolean SFE_UBLOX_GPS::setNavigationFrequency(uint8_t navFreq, uint16_t maxWait)
   if (sendCommand(&packetCfg, maxWait) != SFE_UBLOX_STATUS_DATA_RECEIVED) // We are expecting data and an ACK
     return (false); //If command send fails then bail
 
-  if (_printDebug == true)
-  {
-    _debugSerial->print(F("setNavigationFrequency packetCfg.len is "));
-    _debugSerial->println(packetCfg.len);
-  }
-
   uint16_t measurementRate = 1000 / navFreq;
 
   //payloadCfg is now loaded with current bytes. Change only the ones we need to
   payloadCfg[0] = measurementRate & 0xFF; //measRate LSB
   payloadCfg[1] = measurementRate >> 8;   //measRate MSB
-  //packetCfg.len = 6;
 
   return ((sendCommand(&packetCfg, maxWait)) == SFE_UBLOX_STATUS_DATA_SENT); // We are only expecting an ACK
 }