commit be7f077122deebc468cf609cb5739a845ce7beec
author Bhavna Sharma <sbhavna@codeaurora.org> Mon Jul 03 18:46:41 2017 -0700
committer Bhavna Sharma <sbhavna@codeaurora.org> Thu Jul 13 15:16:59 2017 -0700
tree ce30dcb28fb808f98ecea06a41f16966d9d7e770
parent 86786d166a8ee8a8a8b389f942c08b70eab5c4dd

Fix to report bearing and speed accuracy

1. Validity bit for vertical uncertainity was conflicting
with the validity bit of location source, hence had
to change the bit.
2. Modify ZPP API to report speed and bearing accuracy.

CRs-Fixed:2064017

Change-Id: I8b09ecab3d67557ad7c2451fee71dbc0e1f14371

core/LocApiBase.cpp

diff --git a/core/LocApiBase.cpp b/core/LocApiBase.cpp
index 2dcd4cd..2a0d205 100644
--- a/core/LocApiBase.cpp
+++ b/core/LocApiBase.cpp
@@ -541,10 +541,12 @@
 }
 
 enum loc_api_adapter_err LocApiBase::
-   getBestAvailableZppFix(LocGpsLocation & zppLoc, LocPosTechMask & tech_mask)
+   getBestAvailableZppFix(LocGpsLocation & zppLoc, GpsLocationExtended & locationExtended,
+           LocPosTechMask & tech_mask)
 {
    memset(&zppLoc, 0, sizeof(zppLoc));
    memset(&tech_mask, 0, sizeof(tech_mask));
+   memset(&locationExtended, 0, sizeof (locationExtended));
    DEFAULT_IMPL(LOC_API_ADAPTER_ERR_SUCCESS)
 }
 

core/LocApiBase.h

diff --git a/core/LocApiBase.h b/core/LocApiBase.h
index 86610e7..aa09be5 100644
--- a/core/LocApiBase.h
+++ b/core/LocApiBase.h
@@ -209,7 +209,8 @@
     virtual enum loc_api_adapter_err
         getBestAvailableZppFix(LocGpsLocation & zppLoc);
     virtual enum loc_api_adapter_err
-        getBestAvailableZppFix(LocGpsLocation & zppLoc, LocPosTechMask & tech_mask);
+        getBestAvailableZppFix(LocGpsLocation & zppLoc, GpsLocationExtended & locationExtended,
+                LocPosTechMask & tech_mask);
     virtual int initDataServiceClient(bool isDueToSsr);
     virtual int openAndStartDataCall();
     virtual void stopDataCall();

gnss/GnssAdapter.cpp

diff --git a/gnss/GnssAdapter.cpp b/gnss/GnssAdapter.cpp
index 31e992b..25577ce 100644
--- a/gnss/GnssAdapter.cpp
+++ b/gnss/GnssAdapter.cpp
@@ -1616,7 +1616,8 @@
             GpsLocationExtended locationExtended = {};
             locationExtended.size = sizeof(locationExtended);
 
-            mApi.getBestAvailableZppFix(location.gpsLocation, techMask);
+            mApi.getBestAvailableZppFix(location.gpsLocation, locationExtended,
+                    techMask);
             //Mark the location source as from ZPP
             location.gpsLocation.flags |= LOCATION_HAS_SOURCE_INFO;
             location.position_source = ULP_LOCATION_IS_FROM_ZPP;

utils/loc_gps.h

diff --git a/utils/loc_gps.h b/utils/loc_gps.h
index e3f2182..77ebe54 100644
--- a/utils/loc_gps.h
+++ b/utils/loc_gps.h
@@ -99,7 +99,7 @@
 /** LocGpsLocation has valid accuracy. */
 #define LOC_GPS_LOCATION_HAS_ACCURACY   0x0010
 /** LocGpsLocation has valid vertical uncertainity */
-#define LOC_GPS_LOCATION_HAS_VERT_UNCERTAINITY   0x0020
+#define LOC_GPS_LOCATION_HAS_VERT_UNCERTAINITY   0x0040
 
 /** Flags for the loc_gps_set_capabilities callback. */
 
@@ -541,7 +541,7 @@
      * Represents altitude in meters above the WGS 84 reference ellipsoid.
      */
     double          altitude;
-    /** Represents speed in meters per second. */
+    /** Represents horizontal speed in meters per second. */
     float           speed;
     /** Represents heading in degrees. */
     float           bearing;