Allow for negative GPS speed and altitude values

Author: r3n33

command_interface.c

@@ -263,7 +263,7 @@ void command_interface_process_byte(char incoming)
         }
         else if(strncmp(command_input_buffer, "version", 7) == 0)
         {
-            sprintf((char *)command_response_buffer, "version,0.10.0,beta");
+            sprintf((char *)command_response_buffer, "version,0.10.1,beta");
             m_ble_tx_logbuffer(command_response_buffer, strlen((const char *)command_response_buffer));
         }
         else if(strncmp(command_input_buffer, "getcfg", 6) == 0)

datatypes.h

@@ -93,8 +93,8 @@ typedef struct {
 	time_t dt;
 	uint8_t satellites;
 	//NOTE: 8 bit padding
-	uint16_t altitude; // Div/10
-	uint16_t speed; // Div/10
+	int16_t altitude; // Div/10
+	int16_t speed; // Div/10
 	//NOTE: 16 bit padding
 	int32_t latitude; // Div/100000
 	int32_t longitude; // Div/100000

main.c

@@ -1753,8 +1753,8 @@ static void logging_timer_handler(void *p_context) {
 				// Or we have drifted too far from the last record we must write a full GPS message
 				(
 					currentTime - log_message_gps.dt > 255 ||
-					fabs(hgps.altitude) * 10 - log_message_gps.altitude > 127 ||
-					fabs(hgps.speed) * 10 - log_message_gps.speed > 127 ||
+					hgps.altitude * 10 - log_message_gps.altitude > 127 ||
+					hgps.speed * 10 - log_message_gps.speed > 127 ||
 					fabs(hgps.latitude) * 10000 - abs(log_message_gps.latitude) > 32767 ||
 					fabs(hgps.longitude) * 10000 - abs(log_message_gps.longitude) > 32767
 				)
@@ -1763,8 +1763,8 @@ static void logging_timer_handler(void *p_context) {
 			//TODO: duplicated code
 			log_message_gps.dt = currentTime;
 			log_message_gps.satellites = hgps.sats_in_view;
-			log_message_gps.altitude = fabs(hgps.altitude) * 10;
-			log_message_gps.speed = fabs(hgps.speed) * 10;
+			log_message_gps.altitude = hgps.altitude * 10;
+			log_message_gps.speed = hgps.speed * 10;
 			log_message_gps.latitude = hgps.latitude * 100000;
 			log_message_gps.longitude = hgps.longitude * 100000;
 
@@ -1776,17 +1776,17 @@ static void logging_timer_handler(void *p_context) {
 			// Update delta message
 			log_message_gps_delta.dt = currentTime - log_message_gps.dt;
 			log_message_gps_delta.satellites = hgps.sats_in_view - log_message_gps.satellites;
-			log_message_gps_delta.altitude = fabs(hgps.altitude) * 10 - log_message_gps.altitude;
-			log_message_gps_delta.speed = fabs(hgps.speed) * 10 - log_message_gps.speed;
+			log_message_gps_delta.altitude = (int16_t)(hgps.altitude * 10) - log_message_gps.altitude;
+			log_message_gps_delta.speed = (int16_t)(hgps.speed * 10) - log_message_gps.speed;
 			log_message_gps_delta.latitude = (int32_t)(hgps.latitude * 100000) - log_message_gps.latitude;
 			log_message_gps_delta.longitude = (int32_t)(hgps.longitude * 100000) - log_message_gps.longitude;
 
 			// Update full message
 			//TODO: duplicated code
 			log_message_gps.dt = currentTime;
 			log_message_gps.satellites = hgps.sats_in_view;
-			log_message_gps.altitude = fabs(hgps.altitude) * 10;
-			log_message_gps.speed = fabs(hgps.speed) * 10;
+			log_message_gps.altitude = hgps.altitude * 10;
+			log_message_gps.speed = hgps.speed * 10;
 			log_message_gps.latitude = hgps.latitude * 100000;
 			log_message_gps.longitude = hgps.longitude * 100000;