Use uint8_t for comparisons to avoid int conversions

[fw/altos] / src / ao_gps.c

diff --git a/src/ao_gps.c b/src/ao_gps.c
index 811ac2a8aae0aee4f1a3c0643da7c61e3aa34ff7..78957cb386dc6aeff05f2f19af1971128ff1a0d1 100644 (file)
--- a/src/ao_gps.c
+++ b/src/ao_gps.c
@@ -289,9 +289,9 @@ ao_gps(void) __reentrant
        }
        for (;;) {
                /* Locate the begining of the next record */
-               while (ao_sirf_byte() != 0xa0)
+               while (ao_sirf_byte() != (uint8_t) 0xa0)
                        ;
-               if (ao_sirf_byte() != 0xa2)
+               if (ao_sirf_byte() != (uint8_t) 0xa2)
                        continue;
 
                /* Length */
@@ -321,9 +321,9 @@ ao_gps(void) __reentrant
                cksum |= ao_sirf_byte();
                if (ao_sirf_cksum != cksum)
                        continue;
-               if (ao_sirf_byte() != 0xb0)
+               if (ao_sirf_byte() != (uint8_t) 0xb0)
                        continue;
-               if (ao_sirf_byte() != 0xb3)
+               if (ao_sirf_byte() != (uint8_t) 0xb3)
                        continue;
 
                switch (i) {
@@ -332,7 +332,7 @@ ao_gps(void) __reentrant
                        ao_gps_data.hour = ao_sirf_data.utc_hour;
                        ao_gps_data.minute = ao_sirf_data.utc_minute;
                        ao_gps_data.second = ao_sirf_data.utc_second / 1000;
-                       ao_gps_data.flags = (ao_sirf_data.num_sv << AO_GPS_NUM_SAT_SHIFT) & AO_GPS_NUM_SAT_MASK;
+                       ao_gps_data.flags = ((ao_sirf_data.num_sv << AO_GPS_NUM_SAT_SHIFT) & AO_GPS_NUM_SAT_MASK) | AO_GPS_RUNNING;
                        if ((ao_sirf_data.nav_type & NAV_TYPE_GPS_FIX_TYPE_MASK) >= NAV_TYPE_4_SV_KF)
                                ao_gps_data.flags |= AO_GPS_VALID;
                        ao_gps_data.latitude = ao_sirf_data.lat;