History log of /frameworks/base/services/core/jni/com_android_server_location_GnssLocationProvider.cpp
Revision Date Author Comments (<<< Hide modified files) (Show modified files >>>)
77ca4f8508c276de7ca9bd012fe7db71122101b0 01-Jul-2017 Wyatt Riley <wyattriley@google.com> Improved GNSS state debug logs

Started & Legend for GnssDebug

Bug: 62628286
Test: adb shell dumpsys location works
Change-Id: I0471c4409ef79fe3441b441811c99446c0b92c93
/frameworks/base/services/core/jni/com_android_server_location_GnssLocationProvider.cpp
2171781a7ca12f6a8c9093a682210d0eb489b628 28-Apr-2017 Yipeng Cao <tccyp@google.com> Fixed the hasCarrierFrequency flag in GnssStatus (part1)

Bug: 37757180
Fixes: 37757180
Test: 1. built img locally and flash the device
make -j 40
vendor/google/tools/flashall
2. run cts
cts-tradefed run cts -m CtsLocationTestCases --test
android.location.cts.GnssStatusTest

Change-Id: I5046dbe28518812192de20daca91a8511a1558c0
/frameworks/base/services/core/jni/com_android_server_location_GnssLocationProvider.cpp
2520888be38a40f82620ef391d4939a208de46cf 15-Apr-2017 gomo <gomo@google.com> If gnss service dies, then, upon receiving the death notification on system-server side:
Proactively take down the whole system_server.

Bug: 36291658
Test: System server crashing when killing the Gnss process.
Change-Id: I0d2bf6bf66baff0fd048a64e8940e8ba5fdfe4dd
/frameworks/base/services/core/jni/com_android_server_location_GnssLocationProvider.cpp
268c6e04859cde5e2eacd7a4431bbd533342b5d4 29-Mar-2017 Wyatt Riley <wyattriley@google.com> IGnssDebug.hal change field outputs for dump()

Aligning the dump() string build to match the new
API fields in IGnssDebug.hal

Bug: 30955615
Test: Builds and non-implementation message reported on Pixel '16

Change-Id: I9e60c87e9c5a5ee2bc20e3f54eb878bd472515c1
/frameworks/base/services/core/jni/com_android_server_location_GnssLocationProvider.cpp
388e968a08b080d696de7ea6e477fb674583ed0a 08-Mar-2017 Hridya Valsaraju <hridya@google.com> Fix bug in error checking in GnssLocationProvider

Needed to check if gnssNiIface was a nullptr
before calling setCallback on it.

Bug: 36052760
Test: make
Change-Id: I4d725bd30310353fb7d248f9bf247ac55f94c765
/frameworks/base/services/core/jni/com_android_server_location_GnssLocationProvider.cpp
f6423ba1eff741f92dc7ebbd4cbd039b52c26dab 28-Feb-2017 Hridya Valsaraju <hridya@google.com> Check hwbinder status before checking the returned result

Test: Able to get a location lock on Google Maps on Pixel.

Change-Id: I1e3689ba139bec7d21c12be91bd46fe008ff8278
/frameworks/base/services/core/jni/com_android_server_location_GnssLocationProvider.cpp
a608207fc389d6bba7d40d1c11a52030308ee124 24-Feb-2017 TreeHugger Robot <treehugger-gerrit@google.com> Merge "gnss HAL uses "default" service name"
5d22983acfa5c3c429e21198b7848597e7606d9b 11-Feb-2017 Wyatt Riley <wyattriley@google.com> Common code for 4 jni Location conversions

Cleans up a TODO after GNSS location batching work

Bug: 35273058
Change-Id: Ibb9220269940377a7065b5df3e5672c2a49a50cf
Fixes: 35273058
Test: GnssLogger.apk, Maps & 3rd party app work as expected
/frameworks/base/services/core/jni/com_android_server_location_GnssLocationProvider.cpp
cf879db3666523404ff7e15e77b11434732b4a06 12-Jan-2017 Wyatt Riley <wyattriley@google.com> GNSS Batching - Default Implementation

Connections from .hal layer, through to
Location Manager System APIs

Bug: 31974439
Test: builds, boots, ordinary GNSS & new GNSS batching
basic functional checks on Marlin
Change-Id: If75118c27b5ed34cfc16c9f817b60a3be5485095
/frameworks/base/services/core/jni/com_android_server_location_GnssLocationProvider.cpp
f9133783ae78f7181807820ab1e8d467e7250669 24-Jan-2017 Chris Phoenix <cphoenix@google.com> gnss HAL uses "default" service name

The getService() and registerAsService() methods of interface objects
now have default parameters of "default" for the service name. HALs
will not have to use any service name unless they want to register
more than one service.

Test: builds

Bug: 33844934
Change-Id: Ie6d52854ef3cf0695e3c57292683183bab008f2d
/frameworks/base/services/core/jni/com_android_server_location_GnssLocationProvider.cpp
127ba323d9ae2107ed58039f24ffade2c27ff7c9 16-Jan-2017 gomo <gomo@google.com> Unhide public methods for API Council review.
Test: Existing unit tests still pass.
Change-Id: I49b81f99e4e8d8300fbc6b3d74f336d4f291d171
/frameworks/base/services/core/jni/com_android_server_location_GnssLocationProvider.cpp
4402af6be9de4782d218170203033618f079551d 11-Jan-2017 gomo <gomo@google.com> GNSS O Features according to go/o-gps-hal

Added "Tow Known" as a possible gnss measurement state. As well added Automatic Gain Control (AGC)
to allow jammer detection. Also added the GNSS carrier frequeny to SV status. Also added vertical
GPS position uncertainty, speed uncertainty and bearing uncertainty. Also propagate locaton new
fields to geofence engine.
Test: Existing unit tests still pass.

Change-Id: I472b2fd2516cb7614877dea4bb054a34f50844dc
/frameworks/base/services/core/jni/com_android_server_location_GnssLocationProvider.cpp
b39eb40587e3164141211436b9bdfdd2b5981a07 11-Jan-2017 Hridya Valsaraju <hridya@google.com> Use bitfields in GNSS HAL

Test: mm
Bug: 31974439
Change-Id: Ic14715d7bc55a3460756b07190bf02d6cebe3c52
/frameworks/base/services/core/jni/com_android_server_location_GnssLocationProvider.cpp
b5b6ffe042d298ff44d3f8e49d1c71e623ec0447 06-Jan-2017 Hridya Valsaraju <hridya@google.com> JNI attach/detach threads executing GNSS HAL callback methods

Bug: 31974439
Test: Verified that threads that are not attached to JNI
gets attached and on exit, gets detached.

Change-Id: Ib089720998131ceb107fa5b318a0fcbdf91076ec
/frameworks/base/services/core/jni/com_android_server_location_GnssLocationProvider.cpp
3a3a4acae49e5305455add537a093b1ccd12daa6 04-Jan-2017 Martijn Coenen <maco@google.com> Update GNSS to use elided interface return values.

Test: mma
Bug: 31830743
Change-Id: I315723bc50f682d90c7afd75357a10ed5d4db03f
/frameworks/base/services/core/jni/com_android_server_location_GnssLocationProvider.cpp
d002a8b07becc7ce1b0978b837bd61f29f8c4d18 04-Jan-2017 Steven Moreland <smoreland@google.com> Return<*> getStatus().isOk() -> isOk()

Bug: 31348667
Test: compiles
Change-Id: I7b99a655aefab8c5129b35799e3897a31166be91
/frameworks/base/services/core/jni/com_android_server_location_GnssLocationProvider.cpp
37455587e9fcc7521a0cd027291e92dbebdb4455 29-Dec-2016 Martijn Coenen <maco@google.com> Remove obsolete calls to IPCThreadState/ProcessState.

Threadpools are started automatically, if necessary.

Bug: 31226656
Test: mma
Change-Id: Ib3542ee6f1530ee9c08eb7e9608db741d2c1560d
/frameworks/base/services/core/jni/com_android_server_location_GnssLocationProvider.cpp
e8650320c55869b8ab7430be5aabca3f0c77fe38 06-Dec-2016 Hridya Valsaraju <hridya@google.com> Fix bug in GnssLocationProvider.

Perform nullptr checking correctly before calling setCallback()
on gnssXtraIface.

Bug: 33334137
Test: Checked no crash happens when GNSS HAL returns a null
GnssXtraInfc.

Change-Id: I763d96443a0a6c0f036ecfc86ac442b297560efa
/frameworks/base/services/core/jni/com_android_server_location_GnssLocationProvider.cpp
2ea296029f115376aaa7f4336d0082bdf47eee29 13-Sep-2016 Hridya Valsaraju <hridya@google.com> Modify GnssLocationProvider to use Treble Gnss HAL

Test: GPS functionality tested in binderized and pass-through modes on
Angler. All interfaces except AGnssRil, GnssNavigationMessage and
GnssNi. AGnssRil and AGnssNavigationMessage are not implemented by conventional
GPS HALs in Google devices that would be upgrading to O.

Change-Id: Ie4decb4c47892725c8c89bad3f48581550facff7
/frameworks/base/services/core/jni/com_android_server_location_GnssLocationProvider.cpp
fe427f24e5788612bc271bfff1b07b36ecebf3af 08-Oct-2016 Lifu Tang <lifu@google.com> Fixed the memory leak when receiving Nav messages

Bug: 32038182

Change-Id: Ie659dc945d6efc7feaf9285d3e7375c90cba7234
/frameworks/base/services/core/jni/com_android_server_location_GnssLocationProvider.cpp
f6527aebbce5e2b6b771fa42ae499c2e7584bd26 24-May-2016 Wyatt Riley <wyattriley@google.com> Adding SBAS offset to Gps/Gnss SvStatus converter

Opposite sign of Glonass & BDS offsets. Noticed after screenshot search & further testing.

Bug: 28623392
Change-Id: I6a7c98f455ee7df278eed985ba57ec1c12d888e0
/frameworks/base/services/core/jni/com_android_server_location_GnssLocationProvider.cpp
49d9891c573f8c4994841b24f8d90690c342e8ae 18-May-2016 Wyatt Riley <wyattriley@google.com> Mapping up/down of legacy Gps vs. Gnss Status

- Maps incoming (from HAL) GpsSvStatus for Glonass, Beidou, SBAS, and QZSS
from defacto (NMEA-like) numbers to internal platform with detected constellation.
- Maps outgoing (to Java API) GpsStatus getSatellite info back into the de-facto
overloaded (beyond 32) "GPS" ID numbers.
- Simplifies Glonass ID definition for N (comments only.)
- This enables GPS/GNSS Test apps a non-degrading upgrade path.

Bug: 28623392
Change-Id: I9a19db1f11267032c6927daed767df5afa51c770
/frameworks/base/services/core/jni/com_android_server_location_GnssLocationProvider.cpp
8ad4430c57539d8b4cb5856bd9cfc3ba06ca6343 20-Apr-2016 Lifu Tang <lifu@google.com> Fixed bug in upgrading GpsMeasurement

Bug: 28278410
Change-Id: I936e257d68c348654d6031eb3e8f6d63b28d5e0a
/frameworks/base/services/core/jni/com_android_server_location_GnssLocationProvider.cpp
96635bd0561172e7fd711bcb191a21205c6ce43e 25-Mar-2016 Mike Cailean <mcailean@codeaurora.org> Fixed bugs when deliver GnssMeasurement from N-HAL

There're two major problems that prevents GnssMeasurement from N-HAL from
delivering to Java-side correct:

1. The size check of HAL-provided GnssData is wrong. So native code rejects all
valid GnssData.

2. The svid's data type is different between HAL-side and Java-side. Therefore
an explicit type casting is required.

This CL fixes those problems, together with a few typo fixes.

Bug: 28169973
Change-Id: Id0c2cc120387970564b840cfcbdcb6f329c7e543
/frameworks/base/services/core/jni/com_android_server_location_GnssLocationProvider.cpp
5b49d8a00640ef2e6de953eb83a0cf81f27fa58c 12-Apr-2016 Lifu Tang <lifu@google.com> Removed all the references to deprecated class

Bug: 28144767
Change-Id: If14752a81f6662fb33aacc04d54bfee249221d74
/frameworks/base/services/core/jni/com_android_server_location_GnssLocationProvider.cpp
e50607efab69c62d1a7b5ba9bb3dff47449483c9 08-Apr-2016 Lifu Tang <lifu@google.com> Fixed the sign error in GnssClock

Bug: 28068514
Change-Id: I71b68acb70d4139f94f3829bd4a280d926e251de
/frameworks/base/services/core/jni/com_android_server_location_GnssLocationProvider.cpp
17350f925f5ee4d4df92c768c92e442179b1a420 22-Mar-2016 Lifu Tang <lifu@google.com> A few changes to the measurement implementation

- Fixed the device crashing bug in JNI code
- Fixed a typo ('lock' => 'clock')
- Added hardware clock discontinuity count in toString()

(cherry picked from commit 3a21b02ee2dc8f2c6c879109757d89a2be5d720a)

Change-Id: I6dbda85abda20da0240ac08b22b81729eae7b6f0
/frameworks/base/services/core/jni/com_android_server_location_GnssLocationProvider.cpp
ccb448802b99c5de03ad66e252e187fe8a826fc1 10-Mar-2016 Lifu Tang <lifu@google.com> Checked the flag not set in the correct way

The previous macro always returns false after expansion. Fixed the problem, and
also added a mechanism to catch such kind of mistakes during compile time.

Change-Id: I6870dfa80a8cbbaa0e145ae00f09dd0d78cff157
/frameworks/base/services/core/jni/com_android_server_location_GnssLocationProvider.cpp
0f1ab04516d3e9a72eda22c998910f49f2bd40bb 08-Mar-2016 Lifu Tang <lifu@google.com> Modified the API to make coverage test possible

Bug: 27462840
Change-Id: Idfd2c3c108b25f8be9046fd4bb72dbc57edc24bd
/frameworks/base/services/core/jni/com_android_server_location_GnssLocationProvider.cpp
235bbb1bb50b221da048063181f0f76a13e8014b 05-Mar-2016 Lifu Tang <lifu@google.com> Fixed the crash caused by type mismatch

Bug: 27503442
Change-Id: I019afc1f3dd872c8f9d79e9b6e589a77acd0cddc
/frameworks/base/services/core/jni/com_android_server_location_GnssLocationProvider.cpp
76a620f0482ba54a4921c7c7e7eeb8ea87c12d3e 27-Feb-2016 Lifu Tang <lifu@google.com> Addressed the review comments from API Council

Following are the review comments from API Council:

(Places below that reference "s/foo/bar/" below are suggesting that "foo" be
replaced with "bar".)

GnssClock
-- Class docs should not refer to "GPS" at the top level
-- s/InNs/Nanos/
-- s/Sec/Second/

GnssMeasurement
-- s/In// for "InUnitname"
-- s/Sec/Second/
-- s/Ms/Millis/
-- s/Deg/Degrees/
-- STATUS_GPS_LOCATION_DISABLED -> STATUS_GNSS_LOCATION_DISABLED

GnssMeasurementsEvent.Callback
-- "returns" -> "reports" in the method docs as the methods return void

LocationManager
-- register/unregisterGnssMeasurementEventCallback ->
register/unregisterGnssMeasurementsEventCallback (plural)

GnssNavigationMessage
-- MESSAGE_TYPE_ constants -> change to just TYPE_; the "MESSAGE" is redundant
with the class name
-- is a signed byte/short ok based on the standard here? If not consider using
int in the public api

GnssStatus
-- Make sure @IntDef exists for the constellation type
-- Please append unit types for values returned to match GnssMeasurement methods

GnssStatusCallback
-- Please document time base/units for the time parameter of onFirstFix

Bug: 27385557
Change-Id: Ifed6a2eca3fd7ba89b3ded6964a70376235af8f4
/frameworks/base/services/core/jni/com_android_server_location_GnssLocationProvider.cpp
38bce7925c58fe585144f25ea6c954bab4f5da42 25-Feb-2016 Lifu Tang <lifu@google.com> Removed clock type from GnssClock

Bug: 27341583
Change-Id: I5ffd891737c3ddf83d9c6a78aa36c94858591199
/frameworks/base/services/core/jni/com_android_server_location_GnssLocationProvider.cpp
9363b949a104786c87cd2cd18f206ff3c1957e5e 17-Feb-2016 Lifu Tang <lifu@google.com> Finalized GPS=>GNSS changes with documents

Bug: 27098594
Change-Id: I24529e60d8eb9b48b644cf01e5efc684e6f04839
/frameworks/base/services/core/jni/com_android_server_location_GnssLocationProvider.cpp
a2b74fd268ebce83eb1e3572695bb86b89f057ec 11-Feb-2016 Lifu Tang <lifu@google.com> Fixed the build break

Change-Id: I02ee9b3f44d79510f174a1041f28b9c60450aa9c
/frameworks/base/services/core/jni/com_android_server_location_GnssLocationProvider.cpp
120480f17ae18bbe8c6fa7ec4854281fdf349b59 08-Feb-2016 Lifu Tang <lifu@google.com> Optimized the memory consumption and renamed PRN

Bug: 27058251
Change-Id: I2309943b1a4e7565868ed0f247b84edc631fbd59
/frameworks/base/services/core/jni/com_android_server_location_GnssLocationProvider.cpp
818aa2c2c46addae934fb6cd42f889affc9ef747 01-Feb-2016 Lifu Tang <lifu@google.com> Rename Gps to Gnss for measurement-related APIs

Bug: 26895757
Change-Id: I12d89dc251e356cf976a4bcd3589a8925a8ad745
/frameworks/base/services/core/jni/com_android_server_location_GnssLocationProvider.cpp
e5a0e21b6fcba32e6d7486c5e7736d7d0348d55c 26-Jan-2016 Lifu Tang <lifu@google.com> Framework support to read newly added fields

Change-Id: Ie43395addd102656a4dbf9c25111ac50ec9afe41
/frameworks/base/services/core/jni/com_android_server_location_GnssLocationProvider.cpp
82f893d6df236d257ab2142dd63e72865a5e4f00 22-Jan-2016 Lifu Tang <lifu@google.com> Added an API to query GPS hardware version info

Change-Id: Ic45357d30da350759f56c9d061e60196acb3255b
/frameworks/base/services/core/jni/com_android_server_location_GnssLocationProvider.cpp
a8b7bb5a50d8ad197c6f32eb308919c11b52ce74 26-Jan-2016 Lifu Tang <lifu@google.com> GPS Measurement and Navigation APIs go public

Those APIs are already used by first-party Apps for a while. We now make them
available to third-party Apps as well.

Change-Id: I87d9f0dbb04831fc849228d7df03a2686e2eb383
/frameworks/base/services/core/jni/com_android_server_location_GnssLocationProvider.cpp
30f95a7d67870c28d2aaa88ee989c099f5670225 08-Jan-2016 Lifu Tang <lifu@google.com> Supported GNSS multi-constellation in frameworks

Change-Id: I82574afd3697c47e806d2cfaedc33756d0353b73
/frameworks/base/services/core/jni/com_android_server_location_GnssLocationProvider.cpp