Commit graph

87 commits

Author SHA1 Message Date
Steven Moreland
527fd76a0e Update for hidl adapter module defaults.
Test: pass
Change-Id: Idc6a943149a279bf17cfcfd0f2571473e53bbbbf
2017-10-04 12:47:03 -07:00
Steven Moreland
a5299ee739 Fix typo in update makefiles.
Bug: 37518178
Test: pass
Change-Id: Ic401b3a473f15ca4c01e58b3072e19db7c31b653
2017-09-26 21:59:43 +00:00
Steven Moreland
26a0bb2762 Update makefiles for hidl adapter.
Bug: 37518178
Test: manual
Change-Id: I50e999907d3c64d2b039272b823971998da64d1b
2017-09-25 18:35:56 +00:00
Tri Vo
b65c5ceff2 VTS tests now static link to HAL def libs.
And use VtsHalTargetTestDefaults.
This is a partial cherry-pick of ag/2657080.

Bug: 64040096
Test: compiles, works fine in internal master
Merged-In: I8f6995e9536a9aefe283ee3effec9f5a7f03b620
Change-Id: I76d47d8546416b57055841851a9d343026cd3210
2017-08-25 11:48:10 -07:00
Steven Moreland
dfd8287506 Updating all makefiles.
Bug: 64487114
Test: none
Merged-In: I8608c8f636c35f21e4246a805a9eff6d14124e0a
Merged-In: I2fa89b6661c39859ec4fb62c4bb0a05a35e645f0
Merged-In: Ifdc3c17cb2b85c18b37dac2d03bb5c8935c23180
Change-Id: I170fa1c4fe39f8109b1670db58ef99bb11afc0be
2017-08-14 20:25:09 +00:00
Wyatt Riley
d50d573be9 Merge "Clear notification count to handle different flows" into oc-dev
am: fae2852b2b

Change-Id: I279f8f186304b44b2a1e003deee414d9db09199d
2017-06-20 11:03:47 +00:00
Wyatt Riley
5d190b774e 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
(cherry picked from commit 6ec696b7fe)
2017-06-20 11:33:09 +08:00
Wyatt Riley
22d5682994 Merge "GNSS HAL radio group for E911 SUPL" into oc-dev
am: 2888555e9f

Change-Id: I4de153702f9df1a34fedaa4267192ae35e2498d4
2017-05-31 04:03:35 +00:00
Wyatt Riley
1ed15c4375 GNSS HAL radio group for E911 SUPL
Bug: 37730994
Test: Builds, boots, basic GPS works
Change-Id: I2ceda822d6414348f61c183765f68068764d36cc
Merged-In: I2ceda822d6414348f61c183765f68068764d36cc
2017-05-30 08:29:04 -07: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
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
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
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
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
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
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
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