AP_GPS: move blended-GPS functions into AP_GPS_Blended #26565
Merged
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
collects all of these together in preparation for making a backend
This is simply a 1:1 move of the code from one file to another.
Concept is that these functions will become backend methods, and we can retain history of those modifications made if we move these ahead of time.
Despite being 1:1 copy (I did the whole thing twice!), there are compiler output changes:
I've run
elf-diff
across this and it really does seem to come down to the compiler optimising things differently. The fact Durandal shrinks while others grow seems to back that conclusion up.Current plan for making the backend (and onwards):
new
the backend and treat it like the other GPSs as much as possible.GPS_MAX_BACKENDS > 2
here (e.g. RTK rover mapping / magic-rover-yaw-switching)GPS3_TYPE
will be set to be the blended backend. We will probably want to make the current "blend mask" parameter be a per-instance parameter.GPS_MAX_INSTANCES
vsGPS_MAX_RECEIVERS
at the same time, replacing withGPS_MAX_BACKENDS
for a clean break from the old semantics.GPS_MAX_BACKENDS > 2