This reverts commit 8ac1971678.
Reason for revert: Didn't remove automotive changes from this CL.
Merged-In: I8608c8f636c35f21e4246a805a9eff6d14124e0a
Change-Id: I1c660cffc8817ad0b33da9f6eceb3d88e7c48416
And use VtsHalTargetTestDefaults.
Bug: 64040096
Test: crop out all non-affected tests from vts-hal-hidl.xml and run
vts-tradefed run commandAndExit vts-hal-hidl --skip-all-system-status-check
--skip-preconditions
2 failures, same as without this change:
GatekeeperHidlTest.DeleteAllUsersTest
GatekeeperHidlTest.DeleteUserTest
Change-Id: I8f6995e9536a9aefe283ee3effec9f5a7f03b620
Update the Android.bp generated with hidl-gen.
Test: build with and without BOARD_VNDK_VERSION=current
Bug: 63866913
Change-Id: I1a9db1df49e0f13c5790da2b118ae9ec63ba34a7
Allow HAL definition libs to be static.
Bug: 32920003
Bug: 64040096
Test: update-all-google-makefiles.sh
Change-Id: I1483d572bea6799717d1614fb7d52fe225e31104
Notifiation count now cleared in Setup (and
discarded messages warned about in Teardown.)
Fixes: 62645164
Test: passes on device
Change-Id: I7e88da7251464f281d7c98140d7a9b5a73d79542
(cherry picked from commit 6ec696b7fe)
Notifiation count now cleared in Setup (and
discarded messages warned about in Teardown.)
Fixes: 62645164
Test: passes on device
Change-Id: I7e88da7251464f281d7c98140d7a9b5a73d79542
GPS prefix; also made some comment fixes in IGnssMeasurementCallback.hal
BUG: 37946308
Test: Existing tests still pass.
Change-Id: Ia29c3f3943f0373e18634ddeede2ff5eb8998050
android.hidl.base@1.0 and android.hidl.manager@1.0 are built into libhidltransport.
Test: links
Bug: 33276472
Merged-In: I08aaad80f7e2fc262aa3a8b66fe932e8133a928d
Change-Id: I08aaad80f7e2fc262aa3a8b66fe932e8133a928d
android.hidl.base@1.0 and android.hidl.manager@1.0 are built into libhidltransport.
Test: links
Bug: 33276472
Change-Id: I08aaad80f7e2fc262aa3a8b66fe932e8133a928d
**Note, hashes are being modified in place ONLY
because this interface has not yet been released
in a release of Android.
Updated comments for clarity nearby
some removed todo's on telephony APN
and GNSS demodulated satellite data bits.
Fixes: 34108918
Test: Built & run on Pixel, GPS & XTRA pass.
Change-Id: Ia67be4c4c095479bbbf001987c1629b57ca864a5
Bug: 36084078
Fixes: 36084078
Test: Delete-all GPS tests at desk - Xtra downloads fine
and GPS works
Change-Id: I58fa7d970178577cf177a337117b259dbd9fd266
Both system and gps groups might be relevant to the implementation of a
GNSS HAL.
b/36651420 [QCOM Treble] marlin/sailfish location daemon uses Binder
b/36085168 STOPSHIP - HAL (GNSS) cannot have internet access
b/35757613 STOPSHIP: O GNSS TTFF - Regressed to Cold Start (XTRA
broken) (revert SELinux Change)
Test: marlin
Change-Id: I8aa5b18ef094b8540e7838174154f060a0c3fa9a
Merged-In: I8aa5b18ef094b8540e7838174154f060a0c3fa9a
Signed-off-by: Iliyan Malchev <malchev@google.com>
Files relying on transitive include of utils/Log.h (and things that it
includes) from MQDescriptor.h
Test: pass
Merged-In: Iff316b21bef556bb026378b7f89e97ded3febef4
Change-Id: Iff316b21bef556bb026378b7f89e97ded3febef4
Was compile time, making command line for
easier test team usage of the stronger
test you can do with GNSS signal available.
Also ensuring location calculated correctly after
a delete all (when -agps present), and faster tests.
Bug: 36356863
Test: Verified command line switches work as expected
on a Pixel
Change-Id: I4aa92ff2ca664b1c93bbc9ac154d27063160acf2
Clarifying comment changes only, no functional
changes.
Bug: 30955615
Test: Builds and GPS still works on device
Change-Id: I073527c11d095651d712b41fc5b301ac478e4082
Modifying some fields in this API to align with values
that work with partner implementations, and better
capture the intent of this new API.
Bug: 30955615
Test: Builds and works with hidl-gen
Change-Id: Ied53cbeed2703ce757a246b45339ff2b9dfeb410
Calling all extension APIs, and checking some exist
Calling delete and inject methods
Checking measurement capabilties
Bug:32022567
Test: Passes on Pixel XL with O
Change-Id: I86d5416b08c91e4d96eff6ba8f88e50e0544540b
Stops all GNSS HAL operations, as system doesn't need them any more:
normal, measurements, navigation message, batching
Update default implementation to resend capabilities and system info
and corresponding VTS to enforce
Minor fixes completing the location check
Softens GPS VTS to pass with no signal (still applies better checks
if there is signal)
Bug:36291274
Bug:36066672
Bug:35678469
Bug:35799723
Fixes:36291274
Fixes:36066672
Fixes:35799723
Test: VTS test passes, GPS stops after system server is killed
Basic GPS tests (including delete Xtra) work (before VTS is run)
Change-Id: Ic2ab0f8a79b4aff26eef468615bfee97a83e672f
Adding group inet permission
for the GNSS HAL default implementation
for the XTRA-assisted-GPS file
(from internet) download use case
Bug: 35768061
Bug: 35757613
Bug: 34208553
Test: On Marlin - manual GPS tests
including delete all then restats
with xtra throttle off
Change-Id: I54090979eff53e83a267ce8f6ad84b2aceae290a
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: I0c0cafff5079e49edbfd1f4858760f0c53d3fcfa
Initial VTS tests for GNSS HAL
Basic connection, and GNSS Location Received
Bug: 32022567
Test: VTS runs & passes on a Marlin with build from OPR1.170222
Change-Id: Ied3c203add46013ad6f2df33bbc40cd2ec630d42
Also, reset static state in destructors of interfaces.
Until now, new interfaces were being created for each repeated
getExtension() call to the Gnss interface. This was causing
asserts to fail which were checking for multiple
simultaneous instances of these interfaces.
Death notifications should still required to be created for all
interfaces including callback interfaces.
Bug: 35391416
Test: Verified that Maps still work after forcing a system_server
crash.
Change-Id: I8ce0888016637a3d9b62db5619a3c8c569262c0d
We need google shims on the vendor partition because they are providing
an implementation of a vendor defined interface. They were written by
google just as a courtesy/to make the transition easier. They're
basically a set for vendors to assemble their hal implementations
from.
Bug: 34135607
Test: marlin persist.hal.binderization on/off
Change-Id: I2e2af5af39264cf290259755bb9b2eb9827a21f5
Java intermediates are now created in the common rather than the device-
specific generated sources subdirectory.
Bug: 33842609
Test: make
Change-Id: I16af4cd19bf0b24baa0ad9ebfdb86b4d959c8672
Connecting GnssBatching from fused_location.h
through to the IGnss(Batching).hal
Test: Basic regular GNSS and batched GNSS
functional tests on Pixel
Change-Id: I3ff72c626acece891fd9e5ef2802b3489dac5dd2
This is required to make sure that the constants in gps.h does not
ever go out of sync with the enums created out of them in the HAL
interface definition.
Test: mm
Change-Id: I103cf46d198e9f5de9e0cfd64f2795f0094ecb39
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 adedd 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: I26784a17e82c044002395e1929f8862cc9de63cb
Enum created using constants from gps.h that was missed during
original conversion.
Bug: 31974439
Test: mm
Change-Id: I5ce7bd25c5fac9860ac352f8bc873feddfb31062
New HAL APIs for Gnss location batches,
still to be connected (default implementation)
similarly to this aspect of fused_location.h,
initially as a single client system API.
Bug: 31974439
Test: Ensured it builds and runs GPS & FLP (in Test app & maps)
Change-Id: I27d3fd2cffad25039b006e60c4496a440dc86c13
Test: On Angler, GPS location available on Maps. All interfaces other
than AGnssRil, AGnssNavigationMessage and GnssNi are functionally tested. AGnssRil and
AGnssNavigationMessage are not implemented by conventional GPS HALs in Google devices that would be
upgrading to O.
Bug:31974439
Change-Id: I66225225ed79bc1735d6dd27954b1f9f69ffc557
Bug: 31974439
Test: On enabling additional logging on the conventional GNSS HAL,
it could be seen via logs that the config parameters were getting
set correctly.
Change-Id: I2cb35fbd3484035b0b1ffba8675951be400fd40c
Enum sizes and names cleaned up, fixed errors in comments,
cleaned up some variable names.
Also removed AGnssRefLocation since it appears to be unused.
Bug:31974439
Test: mm
Change-Id: I88f43caac5ce9c7137e28b29ba55ea330786745f
To allow genrules with more than one tool, rename the tool property to
tools and make it an array, replace $tool with $(location <label>),
and use $() for other variables for consistency.
Bug: 31948427
Test: compare build.ninja
Change-Id: I3d714f70a2af0dc60faeee10e09b6ed166601f1d
A Debug interface as well as a configuration interface will be added in
another CL.
Bug: 31974439
Test: mma
Change-Id: I977d95fc815172bd2aae7c78f81e1fc7c9bce72a