Commit graph

96 commits

Author SHA1 Message Date
Wyatt Riley
f5c1dc8d31 Merge "Adding owners for gnss hardware interfaces" into oc-mr1-dev 2017-08-30 23:57:23 +00:00
Wyatt Riley
ad3ef076cf Adding owners for gnss hardware interfaces
Bug # is example where owners file would have
appropriate expedited the fix.

Bug: 64482271
Test: Treehugger (if relevant) - OWNERS file only
Change-Id: If82a2794ae40d48542006674c3c57a1e21c5124b
2017-08-30 16:04:01 +00:00
TreeHugger Robot
67be3e2f64 Merge "Setting clock flags for GPS clock" into oc-mr1-dev 2017-08-29 23:31:22 +00:00
Wyatt Riley
2525d9e579 Setting clock flags for GPS clock
Full bias nanosecond flag is now set correctly for
legacy GpsData (vs. modern GnssData) implementations
of gps.h measurements, going through default GNSS
HAL implementation.

Change-Id: Iec8f03994b6d9e509221e1323b54dc997319fccc
Fixes: 64482271
Test: Before/after on device with GnssLogger
2017-08-25 07:02:58 -07:00
Tri Vo
0c2ce357b1 VTS tests now static link to HAL def libs. #2
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
2017-08-10 17:44:57 +00:00
Justin Yun
608d773ef8 Update make file for vndk enabled.
Update the Android.bp generated with hidl-gen.

Test: build with and without BOARD_VNDK_VERSION=current
Bug: 63866913
Change-Id: I1a9db1df49e0f13c5790da2b118ae9ec63ba34a7
2017-08-04 14:12:23 +09:00
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