Commit graph

58 commits

Author SHA1 Message Date
Wyatt Riley
205881e364 IGnssDebug.hal field fixes
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
2017-03-30 18:31:20 -07:00
Andreas Huber
40d3a9bd8c Convert all comments into "doxygen-ready" comments.
Bug: 36453077
Test: mma
Change-Id: I0b1f77dfae5d2258969e33d85ecf45401ffbdfaa
2017-03-29 08:53:58 -07:00
Steven Moreland
561b602a99 Update makefiles for hwtypes.h
Test: pass
Change-Id: I8e17eb8fec5d2f04c526476a866c8dd54ccb556c
2017-03-28 15:18:20 +00:00
Wyatt Riley
c139959874 Improved GNSS HAL VTS test coverage
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
2017-03-23 15:34:45 -07:00
Wyatt Riley
917640b752 GNSS Service handling System death
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
2017-03-21 11:41:24 -07:00
Yuexi Ma
ed2bb4e8f8 Change vts target base test class name
Test: make vts succeed, vts kernel tests and nfc hal tests passed
Change-Id: I95f10ad7a66f261c9030357edd411fe6d94150e8
2017-03-10 21:14:48 +00:00
Steven Moreland
48c52249e4 hidl_default to all targets.
find hardware/interfaces -type f -not -path "*/.git/*" \
-exec sed -i -e '/cc_[A-Za-z_]\+/ { N; s/$/\n    defaults:
\["hidl_defaults"\],/ }' {} \;
./hardware/interfaces/update-makefiles.sh

Test: pass
Bug: 35840847
Change-Id: If10c464d2f000b85c8d698fe11a4f3fc4cfad282
2017-03-08 19:31:53 -08:00
Steven Moreland
4b52395aa5 GNSS: remove warnings from VTS tests.
Test: pass
Change-Id: I8f06bb4ce8d58164d2c2ebd8885daef75df45481
2017-03-08 19:20:45 -08:00
Wyatt Riley
6d0738bd70 Binderized GPS - restoring Xtra service
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
2017-03-08 14:09:41 -08:00
Yuexi Ma
50d7e27b3c change all vts hidl tests to use VtsHalHidlTargetBaseTest (internal master)
Bug: 33385836
Test: mma
Change-Id: Ide3ebc4a9db76257f33bcfb0c1f13d2ee33eeaa8
2017-02-28 22:27:28 +00:00
TreeHugger Robot
2653620a61 Merge "Remove viral dependency on libhwbinder. (2/2)" 2017-02-24 23:25:26 +00:00
Chris Phoenix
ab5c1e192e 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: I0c0cafff5079e49edbfd1f4858760f0c53d3fcfa
2017-02-24 12:52:41 -08:00
Steven Moreland
7e73d5b908 Remove viral dependency on libhwbinder. (2/2)
find hardware/interfaces -name Android.mk -exec sed -i -e '/libhwbinder
\\/d' {} \;
find hardware/interfaces -name Android.bp -exec sed -i -e
'/"libhwbinder"/d' {} \;
./hardware/interfaces/update-makefiles.sh

Note, automotive has some actual dependencies on libhwbinder, filed
b/35758626 for this.

Test: everything links
Test: (sanity) booted marlin on internal master with these changes
Bug: 35710429
Change-Id: I6d0726c8130d00684b978efbdd48e3ae396f12e5
2017-02-24 12:18:01 -08:00
Wyatt Riley
6c26ed7e9f GNSS HAL initial VTS tests
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
2017-02-22 17:08:22 -08:00
TreeHugger Robot
e006f953b1 Merge "Return existing interfaces for repeated getExtension calls" 2017-02-21 19:00:53 +00:00
Hridya Valsaraju
e020ce28f2 Return existing interfaces for repeated getExtension calls
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
2017-02-21 07:51:20 -08:00
Aurimas Liutikas
1b3db1fbb8 Fix unused parameter warning and enable -Werror.
Test: make -j60 android.hardware.gnss@1.0-impl still compiles
Change-Id: I98256f95b77944d514971be02dc4d4b445359541
2017-02-18 09:30:20 -08:00
Steven Moreland
a0da1a1c3d Move hidl shims to the vendor partition. (2/2)
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
2017-02-13 15:03:41 -08:00
Tri Vo
85120c9490 Update makefiles to use filgroup syntax.
Test: mmma hardware/interfaces
Change-Id: I9f4457c78b7820c242d46359f0debe05d1b3e482
2017-02-10 12:45:19 -08:00
Andreas Huber
d38f147c35 Updated makefiles
Java intermediates are now created in the common rather than the device-
specific generated sources subdirectory.

Bug: 33842609
Test: make
Change-Id: I16af4cd19bf0b24baa0ad9ebfdb86b4d959c8672
2017-02-09 17:20:45 -08:00
TreeHugger Robot
930086babd Merge "Migrate driver/profiler to test/vts-testcase/hal." 2017-02-08 18:33:06 +00:00
Tri Vo
ce608bec74 Migrate driver/profiler to test/vts-testcase/hal.
Bug: 34893894
Test: mmma hardware/interfaces

Change-Id: Ic72716577a5f93700c5056986aff55a76b170b5d
2017-02-07 19:13:28 -08:00
Wyatt Riley
43bce98b6e Merge "GNSS Batching - Default implementation" 2017-02-08 02:08:01 +00:00
Wyatt Riley
8791e7bf2c GNSS Batching - Default implementation
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
2017-02-06 13:39:12 -08:00
Zhuoyao Zhang
e9b43bbe2d Update Andriod.bp for hals.
* Add driver/profiler build rule for all hals.

Test: mma
Change-Id: I98325f7af14fec7dd1bb64b1668de8c7c20ace92
2017-02-03 10:03:01 -08:00
TreeHugger Robot
f24b37b655 Merge "Add annotations to create gnss-base.h" 2017-01-24 15:51:57 +00:00
TreeHugger Robot
a7755a713e Merge "Make some of the fields comments more clear and specific." 2017-01-23 18:58:32 +00:00
Hridya Valsaraju
529331c4cb Add annotations to create gnss-base.h
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
2017-01-23 08:41:15 -08:00
gomo
d567a9955c Make some of the fields comments more clear and specific.
Test: Existing unit tests still pass.

Change-Id: I65f015255958b29a44a7f844560694518e01212b
2017-01-22 22:35:18 -08:00
Steven Moreland
f37a0a439d Ran update-makefiles.sh.
GNSS/Fingerprint/Graphics have all gotten out of sync.

Bug: 34397031
Test: compiles
Change-Id: Ie0a081a4574fd617d5d97ca30ba879f5284efd64
2017-01-19 02:10:44 +00:00
gomo
cf6b4d32e4 Unset Vertical Accuracy, Speed Accruacy and Bearing Accuracy fields as some chipsets
set them in pre-Android-O devices.
Test: Existing unit tests still pass.

Change-Id: I542a2d82d16c1017859ad08060f30a0187664ffd
2017-01-15 21:05:38 -08:00
gomo
c3d9278327 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 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
2017-01-15 17:37:56 -08:00
Mark Salyzyn
3ff5260601 liblog: use log/log.h when utilizing ALOG macros
Test: compile
Bug: 30465923
Change-Id: I67046a75be057e7905570748a1ed6bfd03c0e1c8
2017-01-11 15:47:40 -08:00
TreeHugger Robot
a8d358fb63 Merge changes from topic 'bphwfoo'
* changes:
  Bp/nFoo -> Bp/nHwFoo.
  Bp/nFoo -> Bp/nHwFoo
2017-01-11 23:43:16 +00:00
Yifan Hong
a18049ad13 Bp/nFoo -> Bp/nHwFoo.
Test: mma
Bug: 33554989
Change-Id: I4b3f852d20fdfc49f2da671dd3c12d618ffb4140
2017-01-11 13:14:59 -08:00
Hridya Valsaraju
1bf9910b48 Add GnssLocationFlag enum
Enum created using constants from gps.h that was missed during
original conversion.

Bug: 31974439
Test: mm
Change-Id: I5ce7bd25c5fac9860ac352f8bc873feddfb31062
2017-01-11 10:35:34 -08:00
Yifan Hong
7037fdbf94 Use bitfield type in GNSS HAL.
Test: mma

Change-Id: I92acf537a489fb06d942d3aa19ec9f40ffe26c23
2017-01-11 08:52:00 -08:00
Wyatt Riley
ad03ab22c1 GnssBatching API, port from Fused Location
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
2017-01-09 14:39:53 -08:00
Martijn Coenen
527924aeea Update GNSS/EVS HALs to use elided interface return values.
Test: mma
Bug: 31830743
Change-Id: Id57944c69de37b20409215ef600b553ce5e2526e
2017-01-05 11:08:52 +01:00
TreeHugger Robot
79bd5cb87a Merge "Add GNSS HAL service" 2017-01-03 21:05:30 +00:00
Yifan Hong
f35ee3b192 Revert "Revert "Update makefiles for Java package path.""
This reverts commit a8789f27cb.

Change-Id: I69980b580a556188ba868bb096ae1590b01759f5
2016-12-13 22:20:16 +00:00
Yifan Hong
a8789f27cb Revert "Update makefiles for Java package path."
This reverts commit ab0792ccdc.

Change-Id: I47fc272d9fa8875e3d7ccb10cb748c738fbf8135
2016-12-13 22:18:34 +00:00
Yifan Hong
ab0792ccdc Update makefiles for Java package path.
use V1_0 instead of 1.0.

Bug: 33554192

Test: hidl_test_java
Change-Id: Ic2d95265df10914a60a5bb743d75e839948838c3
2016-12-13 11:32:56 -08:00
Hridya Valsaraju
b5fbbfe28f Add GNSS HAL service
Bug: 31974439
Test: GPS Location available on Maps(on Angler).

Change-Id: I17ba8fab47bc25a1158089ec2b8e8cd54ec650e1
2016-12-07 13:24:15 -08:00
Yifan Hong
35e2aac51e Use bitfield in GPS HAL.
Test: compiles
Change-Id: I200a1cecddffb57596b9958564d2bf6be982cbdc
2016-12-06 00:36:36 +00:00
Yifan Hong
79f07e9c5b Update to use the correct logging library.
Test: compiles
Change-Id: I00467b964fd56e6379cddde26152e413f2f00379
2016-12-03 10:00:39 -08:00
Yifan Hong
501839495f Update makefiles for hidlizing IBase.
A new dependency android.hidl.base@1.0 is added for
all projects.

Test: mma

Change-Id: Ia70d0eb0d74de06475a339698386d383d491a43a
2016-12-03 01:01:14 -08:00
Hridya Valsaraju
29dc1e0c72 Default implementation Gnss HAL
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
2016-12-01 13:53:44 -08:00
Hridya Valsaraju
273c6d2097 Add IGnssConfiguration Interface
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
2016-12-01 13:23:18 -08:00
Hridya Valsaraju
97ecaa0a72 Gnss Hal Minor Cleanup
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
2016-12-01 13:20:49 -08:00