summaryrefslogtreecommitdiff
path: root/subframe.c
Commit message (Collapse)AuthorAgeFilesLines
* LICENSE: Update licenses for consistency. No functional changes.Gary E. Miller2018-11-191-1/+1
|
* SPDXify the licerse references.Eric S. Raymond2018-03-081-1/+1
|
* Trailing whitespace fixup.Gary E. Miller2015-04-301-1/+1
|
* Retire splint from our set of static analyzers.Eric S. Raymond2015-03-301-9/+1
| | | | | | | | | | | | | | | | | | | The proximate cause was that we've been seing emission of error messages that were randomly and disturbingly variable across different environments - notably Raspbian and Gentoo splint gave nontrivially different results than Ubuntu 14.10 splint. And this was *not* due to Ubuntu patches! A pristine splint built from the 3.1.2 tarball on Ubuntu didn't match the Raspbian and Gentoo results either. But this has been coming for a while. Easy access to more modern static analyzers such as coverity, scan-build and cppcheck has been decreasing the utility of splint, which is unmaintained and somewhat buggy and not easy to use. Only file not cleaned is ppsthread.c, because Gary has been working on it during this cleanup. All regression tests pass. PPS observed live on GR601-W.
* Fix cppcheck complaints about not enough parenthesis.Gary E. Miller2015-03-091-1/+1
| | | | | | | Change this: A & B ? 0 : 1 To this: (A & B) ? 0 : 1 No logic or functional change.
* gpsd-report() -> gpsd_log()Eric S. Raymond2015-03-071-184/+184
| | | | | | | | | | | | | | | | This change is done so we can add a "log" hook to the pps_thread_t structure (this is not done yet) and harmonize with the name of the outer logging function. If that name had been left as gpsd_report() there would have been scope for bad confusion with the report_hook member. Also, remove two stray duplicative printf calls from the NMEA2000 driver (drivers shouldn't have printfs!) and fix one typo. This is a step towards factoring out ntplib. For that to happen, the PPS thread code needs to be decoupled from the core session structure. No logic changes. Object compatibility preserved. All regression tests pass.
* FIXME from Hal Murray, only allow leap second notifications in Jun & Dec.Gary E. Miller2015-03-061-0/+1
|
* cppcheck cleanup.Eric S. Raymond2015-01-221-1/+1
|
* Now that the transition is done we can restore the gpsd_report name.Eric S. Raymond2014-08-271-19/+19
| | | | All regression tests pass.
* All gpsd_reportcalls are gone. Only the unused definitins are left.Eric S. Raymond2014-08-271-4/+4
| | | | | | Next, implement labeled reporting and fix up gpson to do the right thing. All regression tests pass.
* The big leap. Most instances of gpsd_report are replaced...Eric S. Raymond2014-08-271-18/+18
| | | | | | | | | | | | | | | | ...with calls to gps_notify(), which indirects to its output method through a slot in an errout structure. Usually the errout structure lives inside the session context, but because struct errout_t is its own thing this does not have to be the case. One large clique of gpsd_notify() calls, in packet.c and isgps.c, looks at a struct errout_t copy in a lexer control block This change is not complete. Remnant gpsd_report calls need to be changed, and gpsmon needs to be tweaked so that the struct_error_t in its context is a non-defaukt hook updating the cuses display rather than simply dumping to stderr. Also the report label functionality needs to be added. All regression tests pass.
* Introduce struct errout_t to encapsulate error-reporting hooks.Eric S. Raymond2014-08-271-18/+18
| | | | | | A major step towards eliminating reverse linkage. All regression tests pass.
* Another step away from reverse linkage. No logic changes.Eric S. Raymond2014-08-271-10/+10
| | | | All regression tests pass.
* Reduce gpsd log verbosity a lot.Eric S. Raymond2013-11-121-4/+4
| | | | | | | | | | Way too much logging was happening at LOG_IO level, which is intended for watching data traffic in and out of the daemon rather than all the minutiae of data analysis - that's LOG_DATA. Also, LOG_DATA gets pushed down two levels. The effect is that -D 5 means exactly what it did, but for purposes other than driver debugging -D 4 now suffices.
* More cppcheck cleanup. All regression tests pass. PPS is live.Eric S. Raymond2013-11-061-1/+1
|
* Contain the assumption that time hinting is done to NTPD.Eric S. Raymond2013-10-311-2/+2
| | | | | | | Because it might be going to chrony. Or something else. No logic changes, just name changes and one new derived #define, TIMEHINT_ENABLE.
* Revert "Separate ntpd support from chrony support, step 1."Eric S. Raymond2013-10-301-2/+2
| | | | We'll have to sneak up on this change more slowly.
* Separate ntpd support from chrony support, step 1.Eric S. Raymond2013-10-301-2/+2
| | | | | | | | | | | | | ntp_offset becomes time_offset. There is a new config symbol CHRONY_ENABLE; most time service code is npw controlled by TIMESERVICE_ENABLE. The file ntpshm.c becomes timeexport.c This change is not complete. More disentanglement has to be done inside timeexport.c itself; at the moment enabling one but not both of ntpshm or chrony will probably break its compile. The point of getting this commit out is so Gary will see the new baseline code ASAP. All regression tests pass.
* Cleanup after the big change.Eric S. Raymond2013-09-291-1/+2
| | | | | The fact that this was necessary may indicate a build systenm issue that's leaving stale binaries in place.
* Change gpsd_report to no longer use a global. All regression tests pass.Eric S. Raymond2013-09-291-60/+70
| | | | | | | | This is a large, ugly change. But without it we can't troubleshoot the ICP/IP-source initialization bug properly - colliding definitions of gpsd_report() were interfering with error reporting early in gpsd runs. More cleanup work remains to be done, but at least this is working.
* Nuke trailing whitespace in C source.Eric S. Raymond2012-09-071-1/+1
|
* Change the API so the yermios strucytture is not publicly exposed.Eric S. Raymond2012-09-071-1/+1
| | | | | | | | Becauser this changed the libgpsd API but not the libgps one, libgpsd now has its own version number. (Doing otherwise would have inflicted a gratuitous binary-compatibility break on application developers). All regression tests pass. Audit check runs clean.
* Another fix the leap seconds notify.Michael Tatarinov2012-07-011-2/+1
| | | | Signed-off-by: Eric S. Raymond <esr@thyrsus.com>
* Fix the leap seconds notify.Michael Tatarinov2012-06-261-1/+5
| | | | Signed-off-by: Eric S. Raymond <esr@thyrsus.com>
* Fix numerous typos in the subframe decoderGary E. Miller2012-05-091-6/+6
| | | | Patch by: Alexey Illarionov <littlesavage@orionet.ru>
* Typo fix.Eric S. Raymond2012-04-201-1/+1
|
* Leap-second warning status from GPS subframe info it's passed to ntpd.Eric S. Raymond2012-04-141-8/+15
| | | | | Patch concept from Michael Tatarinov. Modified to use a session context member rather than a global.
* The attached patch fixes decoding of several subframe fields:Alexey Illarionov2012-03-041-6/+6
| | | | | | | | | sub1.l2 and sub1.af0 have wrong offsets sub2.M0 is 32bit signed sub3.Cid is 16bit signed sub3.IDOT is 14bit signed Signed-off-by: Eric S. Raymond <esr@thyrsus.com>
* scan-build cleanup. All regression tests pass, code splints clean.Eric S. Raymond2011-03-291-1/+1
|
* Re-identify most *_IS flags with corresponding *_SET flags.Eric S. Raymond2011-03-271-1/+1
| | | | | | | | | | | Required so clients looking at flag masks in data exported via the shared-memory interface will see the right thing. These were separated originally in order to avoid pushing the reqyuired width of the gpsdata.set flag mask over 32 bits. It became 64 bits in the Version 5 API change, so that constraint went away. All regression tests pass.
* Clear up some confusion about *_SET vs.. *_IS defines.Eric S. Raymond2011-03-261-1/+0
| | | | | | | | | *_SET masks are library-side and for the set member of the fisx structure. *_IS masks are daemon-side and used only in the status mask returned by the packet analyzer methods, and code in the core library that uses same. Never the twain shall meet. All regression tests pass.
* Move a warning about speeds < 38400 that triggered too often.Eric S. Raymond2011-02-221-0/+7
| | | | | I have profiled. For GPS data, 9600 is sufficient. The exception is if subframe data is enabled.
* splint simplification and cleanup.Eric S. Raymond2011-02-041-27/+27
|
* gemiller says the LEAP_SECONDS check is no longer needed...Eric S. Raymond2011-01-181-35/+20
| | | | | | | | ...because we have a working parity check on the data now. I'm trying to get rid of this magic constant. To do: Check for and log suspcious jumps in the value. Probably means implementing an internal method function to set it.
* Merge branch 'master' of ssh://git.berlios.de/gitroot/gpsdChris Kuethe2011-01-111-1/+1
|\
| * Fix visibilize() for chars with the top bit set.Gary E. Miller2011-01-111-1/+1
| |
* | whitespaceChris Kuethe2011-01-111-61/+61
| |
* | compose sv health with loops rather than snprintfChris Kuethe2011-01-111-12/+12
|/
* All the subframe data now has scaled/unscaled.Gary E. Miller2011-01-101-1/+3
|
* More splint cleanup.Eric S. Raymond2011-01-081-28/+28
|
* Partial splint cleanup. All regression tests pass.Eric S. Raymond2011-01-061-30/+33
|
* The last subframe decoded to JSON.Gary E. Miller2011-01-051-0/+3
| | | | Whoops, I thought this was already pushed.
* Subframe 5, page 25 to JSONGary E. Miller2011-01-051-7/+7
|
* Merge branch 'master' of ssh://git.berlios.de/gitroot/gpsdGary E. Miller2011-01-041-2/+2
|\
| * scale M0 (mean anomaly) to match published almanacChris Kuethe2011-01-041-2/+2
| |
* | Add two more significant digits to sqrtA.Gary E. Miller2011-01-041-1/+1
|/
* Dump subframe 4 almanac to JSON too.Gary E. Miller2011-01-041-0/+3
|
* Add TOW17 to JSON SUBFRAMEGary E. Miller2011-01-041-1/+1
|
* Merge branch 'master' of ssh://git.berlios.de/gitroot/gpsdChris Kuethe2011-01-041-307/+12
|\
| * Subframe 5 almanc part output to JSON.Gary E. Miller2011-01-031-4/+4
| | | | | | | | So how do I enable that JSON output?