Commit graph

59 commits

Author SHA1 Message Date
Sasha Kuznetsov
7fd5cc3a4b Fix 2.0 VTS and 2.1 HAL/default implementation
Test: atest VtsHalGnssV2_0TargetTest && atest VtsHalGnssV2_1TargetTest
Bug: 145830353, 146216289
Change-Id: I77d614783ee5451368a0e93eddc5e71c571193ed
2019-12-17 12:17:01 -08:00
TreeHugger Robot
85fd31c951 Merge "Remove duplicate constants." 2019-12-12 05:29:58 +00:00
Sasha Kuznetsov
fdfd6f9de3 Remove duplicate constants.
Test: in progress
Change-Id: I7d266a9378a679ed30b6ee2af2dbcc4e79df91a5
2019-12-11 14:11:17 -08:00
Sasha Kuznetsov
17e69212c9 Fix comment in 1.1 GNSS HAL VTS tests
Test: n/a
Change-Id: I12cbf004f6b7d494b066478d62a62af22f67c1ac
2019-12-11 12:38:07 -08:00
Yu-Han Yang
b983f10362 Stop location to avoid timing issue
Before setting constellation blacklist, the location report should
be stopped first. Otherwise, the sv status which contains blacklist
constellation is still reporting and may be counted in checking
SvList. According to the comments of this case, location should be
turned off as 2a step.

Also adding another test that a blacklist request that comes in
while tracking is started, takes effect after a stop/start cycle.

Bug: 144675840
Test: on device
Change-Id: I04894426e8384d25f2b5e269961a9cf051cab40f
2019-12-06 13:55:56 -08:00
Yu-Han Yang
c06b536662 Add GNSS HAL 2.1
Bug: 136136192
Bug: 141758837

Test: atest VtsHalGnssV2_1TargetTest
Change-Id: I8e6d52695c8ab0aeacacb0107a6c0c8271983ed3
2019-12-03 13:42:22 -08:00
Dan Shi
4652a0e565 Convert VtsHalGnssV1_1TargetTest to be parameterized test
Bug: 142397658
Test: atest VtsHalGnssV1_1TargetTest
atest VtsHalGnssV1_1Target

Change-Id: I575630c8cbbf8bbd341425c5bbe04ca49c2c1758
2019-10-16 14:32:28 -07:00
Steven Moreland
38c46d0b98 Merge "Remove libhwbinder/libhidltransport deps" am: 6b62c58a9a am: 8c22c3862f am: 6684ee8c36
am: 9a5ffa3196

Change-Id: I3a80895d10767fe68e90a907312a83ded976cbcc
2019-09-06 16:06:13 -07:00
Steven Moreland
6684ee8c36 Merge "Remove libhwbinder/libhidltransport deps" am: 6b62c58a9a
am: 8c22c3862f

Change-Id: Iaedfceb49f81387f7deaad434613b7c9f1c502a0
2019-09-06 14:28:04 -07:00
Steven Moreland
b3a4d3832e Remove libhwbinder/libhidltransport deps
Since these were combined into libhidlbase.

Bug: 135686713
Test: build only (libhwbinder/libhidltransport are empty)
Change-Id: I075670b64eebbbbd6a6ae0e84ad51bf1c6f5ba36
2019-09-06 01:07:02 +00:00
Anil Admal
d3bef8732c Address GNSS VTS code duplication (part 1)
-Moved GNSS callback event queue class from VTS 2.0 code
 into common library that is shared with all GNSS VTS tests.
-Modified VTS 1.1 code to reuse GNSS callback event queue in
 order to address potential issues noted in b/131869042 and
 also to later extract common code from VTS 1.1 and VTS 2.0
 into common library.

Bug: 136515339
Test: atest VtsHalGnssV2_0TargetTest
      atest VtsHalGnssV1_1TargetTest

Change-Id: I9356e7653f9784a9149b2c2bd6307750e9cc56b3
2019-07-30 18:25:29 -07:00
Anil Admal
79f9484c6b Merge "VTS 1.1 TestGnssMeasurementCallback test must allow 1.1 or 1.0 versions" into qt-r1-dev
am: f22b39c457

Change-Id: Ic7bd9b309d7e9fe7a2317c4dcba85bd87fa1431f
2019-07-19 11:25:34 -07:00
Anil Admal
6abdaac714 VTS 1.1 TestGnssMeasurementCallback test must allow 1.1 or 1.0 versions
Fixes: 137839372
Test: atest VtsHalGnssV1_1Target
Change-Id: Ie3b1fa8b4738c03bff7924396b5eb5c6774841f1
2019-07-18 18:40:53 -07:00
Anil Admal
dbef8877c0 Merge "Add GetLocationLowPower VTS 2.0 test" into qt-r1-dev
am: 954f4fef65

Change-Id: Id553c7b77c0b87e29c8055eb9421823b198a3a9d
2019-07-17 14:57:09 -07:00
Anil Admal
03bc98e028 Add GetLocationLowPower VTS 2.0 test
- Run test only if LOW_POWER_MODE capability is supported.
- Modify the corresponding VTS 1.1 test to run only if the GNSS
  HAL version is 1.1.

Fixes: 127434062
Test: atest VtsHalGnssV2_0Target
Change-Id: Icbd18d80468e9e5a542f2eace5bfbe45af08ca47
2019-07-15 18:56:37 -07:00
Felix
bf481c99ed Merge "Add interface info to .rc files" am: 98d0f4d52c am: 37298a7616
am: 3531ec0183

Change-Id: I9a48f01064e6b64fdbd3dcae3623163a27998703
2019-07-08 12:42:48 -07:00
Felix
37298a7616 Merge "Add interface info to .rc files"
am: 98d0f4d52c

Change-Id: Ie6cfbacd523c4b09f77bee3369ec8e88b3fa0419
2019-07-08 12:22:23 -07:00
Anil Admal
b821d8e59f Merge "Add GNSS satellite blacklist VTS 2.0 test" into qt-r1-dev
am: 27c44d047a

Change-Id: Id41c14d75616be6ae4650a1188acd253ac96cc39
2019-06-28 15:44:01 -07:00
Anil Admal
f42602678f Add GNSS satellite blacklist VTS 2.0 test
- Use IGnssCallback@2.0::gnssSvStatusCb_2_0() method to receive SV
  info.
- Run the test only if SATELLITE_BLACKLIST capability is supported.
- IRNSS is excluded from blacklisting as IGnssConfiguration@2.0 is
  not updated to support the blacklisting new constellation type.
- Modify the corresponding VTS 1.1 test to run only if the GNSS
  HAL version is 1.1.

Bug: 128028791
Test: atest VtsHalGnssV2_0Target
Change-Id: Idd44a5a1c9d54d7434da945ecbd1ee90007f5054
2019-06-26 18:26:36 -07:00
Felix
551b8d15ce Add interface info to .rc files
Signed-off-by: Felix <google@ix5.org>
Change-Id: I6d70bbdb66c3dce280bf6908c3750316a6f6cf70
2019-06-25 20:00:07 +02:00
Anil Admal
bc5fcc4019 Merge "Clarify when IGnss.hal cleanup() and setCallback() methods are called" into qt-dev
am: 88d9d2570e

Change-Id: I775555a6e8394ca0954ab90fa0e08d50f73ab8cf
2019-05-10 09:05:47 -07:00
Anil Admal
2f28bb0c19 Clarify when IGnss.hal cleanup() and setCallback() methods are called
Update IGnss.hal cleanup() and setCallback() method documentation to
clarify when the framework calls these methods so that the GNSS engine
knows when to shut down for power savings.

This CL updates @1.1::IGnss.hal and @1.0::IGnss.hal interfaces only.
@2.0::IGnss.hal is updated in another CL.

Bug: 124104175
Test: Existing tests pass
Change-Id: I8181251677dab78ce0619fa1e2a667b36e115f25
2019-05-09 16:32:06 -07:00
Steven Moreland
6d494b2346 Merge "Update hidl makefiles for bpfmt" am: ff0bd741ca
am: 96f40f7b02

Change-Id: Idbf030e4993067bdb8181321bca2de00c9b6f7ef
2019-04-18 14:34:45 -07:00
Steven Moreland
1ae4615d9f Update hidl makefiles for bpfmt
hidl-generated makefiles are now generated such that bpfmt(file) == file.

Bug: 67417008
Test: enable bpfmt hook
Change-Id: I1f69d292bc23a7cc293a66110cb02d597e1019ad
2019-04-17 09:38:50 -07:00
Yu-Han Yang
08642f98a4 Add VTS test for gnss.measurement_corrections@1.0
- also implemented default implementation

Bug: 120529158
Fixes: 120529158
Test: tested on cuttlefish
Change-Id: I9e63dc35284effff351eabedade05ed147d1ef17
2019-03-05 19:30:39 -08:00
Yu-Han Yang
a509861a59 Report mock locations in 2.0 default implementation
- Create android.hardware.gnss@common-default-lib for sharing common
default implementation code.
- Create android.hardware.gnss@common-vts-lib for sharing common VTS
code.

Bug: 121217686
Change-Id: I35c127c23d97ab9a5c6ee13b36fbfe9c3708e3f3
Fixes: 121217686
Test: tested on cuttlefish
2019-02-15 09:42:43 -08:00
Keun Soo YIM
68ae05dd2d pack VTS cc_test binaries as general-tests
Test: make general-tests
Bug: 120093339
Merged-In: I363450d205868f900e4925ccff1430e2a569f2a4
Change-Id: I363450d205868f900e4925ccff1430e2a569f2a4
2018-12-07 10:49:56 -08:00
Keun Soo YIM
ff84c37bc1 pack VTS cc_test binaries as general-tests
Test: make general-tests
Bug: 120093339
Change-Id: I363450d205868f900e4925ccff1430e2a569f2a4
2018-11-27 16:11:41 -08:00
WyattRiley
074a27e28f Allow more time to un-blacklist strongest satelite
Also fixes a few intermediate values and confusing
logging discovered during weak GPS signal testing.

Change-Id: I2d3c5113956132ef4b60e28653aacf89f1159d83
Fixes: 116170859
Test: Deep indoor fail and windowledge pass of VTS on
test phone, and logcat values check.
2018-09-28 07:12:40 -07:00
Yu-Han Yang
45f0a8bb67 Set the floor value of time estimate as 2017 Jan 01 00:00:00 GMT
- The VTS test checks if the time estimate is a sane value. Thus, any
arbitrary sane value is acceptable. Here we make it consistent to
IGnssDebug.hal, where a comment mentions 2017 Jan 1 as an example of
the value.

Bug: 110094003
Test: make vts, and tested on device.
Merged-In: Ic4f6b597a718ea3e98a67c7939f9e6930d44d224
Change-Id: Ic4f6b597a718ea3e98a67c7939f9e6930d44d224
(cherry picked from commit 5fa43c83e9)
(cherry picked from commit 4f0d19ebff)
2018-08-29 17:56:51 +00:00
WyattRiley
2d86da7b28 Improve VTS GNSS 1.1 reliability.
Allow VTS test for low power mode to handle
certain vendor issues that supply one extra location
at the start of low power mode.

Allow more time for GNSS first fix, and warmup as
needed given limited VTS access to AGPS.

Bug: 110626730
Bug: 110987651
Test: (TODO) Passes on device.
Merged-In: Ieeefd7fcd45890b03dffbbee965e1d9b17805c4c
Change-Id: Ieeefd7fcd45890b03dffbbee965e1d9b17805c4c
(cherry picked from commit 5ea5dda2c3)
2018-08-28 21:17:02 +00:00
Yu-Han Yang
75b0c4bba2 Inject last location in InjectBestLocation VTS test
Bug: 110626730
Test: run VTS on device.
Merged-In: I6f57dd7f9164dacf011f25412e55fcd22821aca0
Change-Id: I6f57dd7f9164dacf011f25412e55fcd22821aca0
(cherry picked from commit bb372e5984)
2018-08-28 21:13:38 +00:00
Yu-Han Yang
398b272b30 Tolerate 1 sv status in GNSS blacklist VTS test
Bug: 110969497
Test: run VTS test on device.
Merged-In: I26345bb62513da8530cb48672e5f59a915817cb4
Change-Id: I26345bb62513da8530cb48672e5f59a915817cb4
(cherry picked from commit 91cc8f4832)
2018-08-28 18:18:47 +00:00
Yu-Han Yang
6b21448262 Set lowPowerMode as false in StartAndGetLocations
Bug: 80544245
Change-Id: Icf85e9b8ed433dbdb759b54c4ba7b5e7a21b7a26
Fixes: 80544245
Test: VtsHalGnssV1_1TargetTest --gtest_filter=*Blacklist*
2018-05-31 15:04:04 -07:00
Yu-Han Yang
ca63cbf6e2 Update VTS test to blacklist only non-GPS satellites
Bug: 79481978
Change-Id: Ieef04d816448d37ae54c61375db4955d4d3a24d7
Fixes: 79481978
Test: make vts
2018-05-22 11:26:21 -07:00
Yu-Han Yang
5d21ab2cc1 Update doc in Gnss Hal v1.1
Bug: 74949950
Fixes: 74949950
Test: N/A
Change-Id: I863445417d785c0b346ec96383adfd2366dbd71b
2018-04-02 14:28:33 -07:00
Wyatt Riley
e5c92bb789 Comment updated for privacy clarity
Clarifying model name does not allow device specific
indentification.

Bug: 73951626
Test: Build only (comment only)
Change-Id: Ib10176ec4db935fe87330b39977b7ab6b281808f
2018-03-21 17:29:33 -07:00
Yu-Han Yang
f2b1866a13 Fix breakage by removing #include gps.h
Bug: 74951218
Fixes: 74951218

Test: aosp_arm64-eng builds
Change-Id: Iae66fc56b5cba01a3a1cb363f62dabd223743438
2018-03-15 17:35:35 +00:00
Wyatt Riley
4bd3e45305 Merge "Mock blacklisting satellites in default implementation" into pi-dev 2018-03-15 15:19:15 +00:00
Yu-Han Yang
8c7a0309ab Call SetGnssCallback in SetUp
- Each VTS test requires a valid GnssCallback

Bug: 73845705

Test: tested Gnss v1.1 VTS on gce_x86
Change-Id: I2ca8e6b86aeadc60d7ceacbc2ffaf05ea75c0f40
2018-03-15 01:51:27 +00:00
Yu-Han Yang
78652a09fd Update OWNERS
- Add yuhany to 1.1/vts/OWNERS
- Add OWNER file to 1.1/default/

Bug: 73845705
Test: N/A

Change-Id: Id83805edd841d161f1dbfe1907af97d5d7cb2b06
2018-03-14 21:26:41 +00:00
Yu-Han Yang
a2f77322b4 Mock blacklisting satellites in default implementation
- Mock GnssDebug to pass the sanity check.

Bug: 73845705

Test: All Gnss v1.1 VTS tests are passing on gce_x86

Change-Id: I258fb1671d2b682f471207192b8a0feb138c16ab
2018-03-14 14:23:30 -07:00
Yu-Han Yang
584ad32e73 Implement Gnss::start() and Gnss::stop()
- calling start() creates a thread that periodically receives locations

Bug: 73845705

Test: ran VtsHalGnssV1_1TargetTest. SetCallbackResponses and
GetLocationLowPower are passing

Change-Id: I6a544c2832037a8b507c0ab030a8b766be499669
2018-03-08 13:54:07 -08:00
TreeHugger Robot
b7724bec9e Merge "Add boilerplate default implementation for Gnss Hal v1.1" 2018-02-28 22:44:38 +00:00
Yu-Han Yang
73f16adf92 Add boilerplate default implementation for Gnss Hal v1.1
Bug: 73845705

Test: gce_x86 device builds and runs. Debug logging shows default
implementation is called by GnssLocationProvider.

Change-Id: I58db6b11ad8b67d0364d8945801165cf6e499e55
2018-02-27 18:02:20 -08:00
Yu-Han Yang
0e745916b9 Update documentation of gnssRequestLocationCb
Bug: 73716887
Test: This change updates comment only. Verified it builds.

Change-Id: I580de91d95b25075af4b8e9fd7d3ed6536303616
2018-02-26 10:01:48 -08:00
Wyatt Riley
631308a783 Merge "Adding additional enum value for Half Cycle" 2018-02-21 21:39:43 +00:00
TreeHugger Robot
f9651f3d3b Merge "Call deleteAidingData to clean up after injectBestLocation" 2018-02-21 20:51:03 +00:00
Wyatt Riley
ed90e4dd9e Adding additional enum value for Half Cycle
Adds Enum, and documents the deprecated fields.

Bug: 71908529
Test: Builds for device and vts, with corresponding
framework and vts changes.
Change-Id: I232239f583f820c5d0538e1ae9567f01944f2399
2018-02-20 09:58:14 -08:00
Yu-Han Yang
b493a6a212 Call deleteAidingData to clean up after
injectBestLocation

Bug: 73318250

Test: Copy the same test to v1.0 and manually run vts tests. All passed.
Change-Id: Ie5196c3ce197c39f57444b89d1ae6b75cedf2b69
2018-02-16 15:27:20 -08:00