Commit graph

90 commits

Author SHA1 Message Date
Tri Vo
f5d136c231 Update Android.bp HIDL makefiles
Allow HAL definition libs to be static.

Bug: 32920003
Bug: 64040096
Test: update-all-google-makefiles.sh
Change-Id: I1483d572bea6799717d1614fb7d52fe225e31104
2017-08-03 00:30:01 +00:00
Wyatt Riley
6ec696b7fe Clear notification count to handle different flows
Notifiation count now cleared in Setup (and
discarded messages warned about in Teardown.)

Fixes: 62645164
Test: passes on device
Change-Id: I7e88da7251464f281d7c98140d7a9b5a73d79542
2017-06-19 15:03:06 -07:00
Wyatt Riley
94c7a049dd Cleaner VTS for wider use
Better start/stop testing

Change-Id: I4f50fd6a9aaecac3455590f100f15117336993c9
Fixes: 35810250
Test: VTS passes on 2 different device variants
2017-06-16 13:06:50 -07:00
Wyatt Riley
2180b6bc51 GNSS HAL radio group for E911 SUPL
Bug: 37730994
Test: Builds, boots, basic GPS works
Change-Id: I2ceda822d6414348f61c183765f68068764d36cc
2017-05-30 08:23:31 -07:00
Steven Moreland
7708449766 Update makefiles. (2/2) am: 76cfb84140
am: f203167814

Change-Id: I1f13401266f3c2f78c627088b09c79cd7c8d0276
2017-05-22 17:29:04 +00:00
Steven Moreland
f203167814 Update makefiles. (2/2)
am: 76cfb84140

Change-Id: I24d21438a5b2f447d2160929c6ff6e17b2b66820
2017-05-22 17:15:20 +00:00
Steven Moreland
76cfb84140 Update makefiles. (2/2)
Adds default configuration for all hals.

Bug: 38415912
Test: pass
Change-Id: Idd1f3a2b7b16ad956d31784a513e93a066cdd02e
2017-05-19 15:54:03 -07:00
Steven Moreland
d09cf863da Update makefiles for c++-sources and c++-headers. (2/2) am: bc71124120
am: a32ebce372

Change-Id: Ic1dd8f98e72656bc12a6e5224e541f75afb79daa
2017-05-11 01:38:14 +00:00
Steven Moreland
a32ebce372 Update makefiles for c++-sources and c++-headers. (2/2)
am: bc71124120

Change-Id: Icf36068a4d863a8520854c7a83e51f960c188e12
2017-05-11 00:48:55 +00:00
Steven Moreland
bc71124120 Update makefiles for c++-sources and c++-headers. (2/2)
Test: pass
Bug: 38174080
Change-Id: Icad451f9f9ad5f46412356a4171a8a1222f8e545
2017-05-09 19:58:43 -07:00
Yu Liu
4464eeb903 Renamed enums in IGnssNavigationMessageCallback.hal from GNSS prefix to GPS prefix; also made some comment fixes in IGnssMeasurementCallback.hal am: 216b87090d
am: 858ba19de2

Change-Id: I53fcb6595c20c2d243a4259e7b394baec7635520
2017-05-05 20:31:56 +00:00
Yu Liu
858ba19de2 Renamed enums in IGnssNavigationMessageCallback.hal from GNSS prefix to GPS prefix; also made some comment fixes in IGnssMeasurementCallback.hal
am: 216b87090d

Change-Id: I4ac5a047e375280b51dc37498650017f44327689
2017-05-05 20:22:03 +00:00
Yu Liu
216b87090d Renamed enums in IGnssNavigationMessageCallback.hal from GNSS prefix to
GPS prefix; also made some comment fixes in IGnssMeasurementCallback.hal

BUG: 37946308
Test: Existing tests still pass.

Change-Id: Ia29c3f3943f0373e18634ddeede2ff5eb8998050
2017-05-05 20:08:47 +00:00
Steven Moreland
5c63b11f76 Remove redundant dependencies
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
2017-05-02 15:29:30 -07:00
Steven Moreland
2cd0d42d72 Remove redundant dependencies
android.hidl.base@1.0 and android.hidl.manager@1.0 are built into libhidltransport.

Test: links
Bug: 33276472
Change-Id: I08aaad80f7e2fc262aa3a8b66fe932e8133a928d
2017-05-02 10:17:28 -07:00
Yipeng Cao
f245522ed1 Merge "Fixed the hasCarrierFrequency flag in GnssStatus(part 2)" into oc-dev am: dc24ef0ff3
am: 0a210de9e2

Change-Id: Ia52cd55c8268ba9aff1e6c7a82c65f43065036b3
2017-04-29 22:49:30 +00:00
Yipeng Cao
a36f51a127 Fixed the hasCarrierFrequency flag in GnssStatus(part 2)
Bug: 37757180
Fixes: 37757180
Test: cts-tradefed run cts -m CtsLocationTestCases --test
android.location.cts.GnssStatusTest

Change-Id: I817931ee2c19d0a770c4d7cedfbd69bc85db6978
Chang-Id: Icf4240ccad7ff367d96001973c0be36308ffd9b3
2017-04-28 21:35:58 +00:00
Wyatt Riley
64c92b0749 Merge "Adding missing NI_TYPE for emergency SUPL" into oc-dev am: 12d67e3c7a
am: ab3e767b88

Change-Id: I4b413866cbf6d0b888f9b67e0c5725cd0068ea91
2017-04-25 18:15:47 +00:00
Wyatt Riley
0442918818 Adding missing NI_TYPE for emergency SUPL
Bug: 37409476
Test: Builds, and on Pixel, basic GPS & XTRA tests pass.
Partners & Pixel team running the SUPL emergency tests.

Change-Id: I6b23608e82dc1682d61a516ab5045d3b78b53cae
2017-04-21 12:52:46 -07:00
Wyatt Riley
a190e88f10 Merge "Clarifying comments and todos, update current.txt." into oc-dev am: f6470873f4
am: 6cedad005d

Change-Id: Ia5d83ddaaca9893ce0ba253263d2fc757f40c4d4
2017-04-21 15:46:39 +00:00
Wyatt Riley
5c196e0e66 Clarifying comments and todos, update current.txt.
**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
2017-04-20 16:04:48 +00:00
Wyatt Riley
6a2a4f29c6 Narrowing class & user id for GNSS HAL
Bug: 36084078
Fixes: 36084078

Test: Delete-all GPS tests at desk - Xtra downloads fine
and GPS works

Change-Id: I58fa7d970178577cf177a337117b259dbd9fd266
2017-04-19 14:44:48 +00:00
TreeHugger Robot
206e38a4e8 Merge "Remove dependency of libhardware_legacy from HAL services" into oc-dev 2017-04-18 00:26:33 +00:00
Iliyan Malchev
554df3738d gnss@1.0-service: drop inet group, add gps group
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>
2017-04-15 05:12:55 +00:00
Hung-ying Tyan
4f1bbf7074 Remove dependency of libhardware_legacy from HAL services
It is not used by these services.

Bug: 33241851
Bug: 37305530
Test: built and tested on sailfish
Change-Id: I1f3f51981d7dfcf27e18d9495c747b7662867d31
2017-04-15 10:17:16 +08:00
Iliyan Malchev
2dd67a43b6 Route GNSS HAL libbinder traffic to /dev/vndbinder
Bug: 36565056
Test: no more gnss crash at boot

Change-Id: Ib3935beeb521dc655c5d7e1b6d7680a190b3fd4e
Signed-off-by: Iliyan Malchev <malchev@google.com>
2017-04-14 16:01:30 +00:00
Andreas Huber
6fa374ad30 Changed the naming scheme for Treble java libraries (both static and dynamic)
android.hardware.foo@1.0-java => android.hardware.foo-V1.0-java

Bug: 37207894
Test: make
Change-Id: I177f4cfd42f635aaf096107af8309d1a82ed41c0
Merged-In: I8b91881dfaf507c0d852d56ebbb53ff0987e5a34
2017-04-13 15:23:46 -07:00
Steven Moreland
3b36a83a2e Update makefiles (2/2).
Test: pass
Change-Id: Ic7e6289c7b69785491708ce0acbb657f4c704229
2017-04-11 09:58:03 -07:00
Wyatt Riley
5ed473340c Merge "Add command line strictness to VTS" into oc-dev 2017-04-07 15:03:50 +00:00
Steven Moreland
4e7a307730 Fix transitive includes.
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
2017-04-06 22:04:05 +00:00
Wyatt Riley
0ee0cfbd74 Add command line strictness to VTS
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
2017-04-06 10:32:51 -07:00
Wyatt Riley
e8a8b29c6e Clarifying broadcast vs. predicted orbits
Clarifying comment changes only, no functional
changes.

Bug: 30955615
Test: Builds and GPS still works on device
Change-Id: I073527c11d095651d712b41fc5b301ac478e4082
2017-03-31 15:23:00 -07:00
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