diff options
author | Eric S. Raymond <esr@thyrsus.com> | 2011-10-09 13:16:15 -0400 |
---|---|---|
committer | Eric S. Raymond <esr@thyrsus.com> | 2011-10-09 13:16:15 -0400 |
commit | afea16b0761da830578a5da712ea94495c5845f1 (patch) | |
tree | 0730d6dbcf40e13c7097b761386996d19803a9be /driver_sirf.c | |
parent | 8c399a07c0ad4e782b65abe6b871f4492413008d (diff) | |
download | gpsd-afea16b0761da830578a5da712ea94495c5845f1.tar.gz |
In SiRF, don't set the fix timestamp on a MID4.
Longer collection runs show that this often results in bad timestamps
from skyviews not good enough for a fix. There was a comment here
explaining that this was a way to pass a timestamp to NTP, but that
isn't necessary; if there's a fix, NTP will get notified by the MID2
sentence in each cycle.
Diffstat (limited to 'driver_sirf.c')
-rw-r--r-- | driver_sirf.c | 14 |
1 files changed, 1 insertions, 13 deletions
diff --git a/driver_sirf.c b/driver_sirf.c index b6d58b11..ecb6a219 100644 --- a/driver_sirf.c +++ b/driver_sirf.c @@ -472,7 +472,6 @@ static gps_mask_t sirf_msg_svinfo(struct gps_device_t *session, unsigned char *buf, size_t len) { int st, i, j, cn; - gps_mask_t mask = 0; if (len != 188) return 0; @@ -529,22 +528,11 @@ static gps_mask_t sirf_msg_svinfo(struct gps_device_t *session, session->driver.sirf.time_seen, session->gpsdata.skyview_time, session->context->leap_seconds); - /* - * Might be better if we didn't change the fix timestamp here, - * but presently there's no other way to pass the time to NTP. - */ - session->newdata.time = session->gpsdata.skyview_time; - mask |= TIME_SET | PPSTIME_IS; - /* - * This time stamp, at 4800bps, is so close to 1 sec old as to - * be confusing to ntpd, but ntpshm_put() will ignore it if a better - * time already seen - */ } #endif /* NTPSHM_ENABLE */ gpsd_report(LOG_DATA, "SiRF: MTD 0x04: visible=%d mask={SATELLITE}\n", session->gpsdata.satellites_visible); - return SATELLITE_SET | mask; + return SATELLITE_SET; } #ifdef NTPSHM_ENABLE |