Commit graph

325 commits

Author SHA1 Message Date
Steven Moreland
91799c6cc7 Merge "Fix typo in update makefiles." am: 9960148420 am: f07e364ce3 am: 71a193a425
am: 9fdd241905

Change-Id: I1c852fa6600fecc44d5afd1a65a5f51c43ecf71c
2017-09-27 00:02:52 +00:00
Steven Moreland
9fdd241905 Merge "Fix typo in update makefiles." am: 9960148420 am: f07e364ce3
am: 71a193a425

Change-Id: I45ae43f07323254212532acbf0f891d11143b05f
2017-09-26 23:58:51 +00:00
Steven Moreland
f07e364ce3 Merge "Fix typo in update makefiles."
am: 9960148420

Change-Id: I949634e72f817f3a5411130e968acd8efd1d7725
2017-09-26 23:51:07 +00: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
7741c0e17b Update makefiles for hidl adapter. am: 26a0bb2762 am: 73949c1d7e am: b94d0c7290
am: 97228c0b7d

Change-Id: Ic4b459b7c1a4325453945b0b00db8f44e4e77c26
2017-09-26 13:36:09 +00:00
Steven Moreland
97228c0b7d Update makefiles for hidl adapter. am: 26a0bb2762 am: 73949c1d7e
am: b94d0c7290

Change-Id: If766cffbcc003cc0cc5eb98969b924aa918fff44
2017-09-26 12:53:13 +00:00
Steven Moreland
73949c1d7e Update makefiles for hidl adapter.
am: 26a0bb2762

Change-Id: I3a811f5cd49bd4e81e8fcd5c8e88922115812539
2017-09-25 23:14:44 +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
Jaekyun Seok
81104ae5ef Add 'vendor.' prefix to a vendor HAL service name
To prevent property name collisions between properties of system and
vendor, 'vendor.' prefix must be added to a vendor HAL service name.
You can see the details in http://go/treble-sysprop-compatibility.

Test: succeeded building and tested on a walleye device
Bug: 36796459
Change-Id: I4e8fbee791ec917a8f627a1366f4d44ec7e6febc
2017-09-12 08:01:42 +09:00
Wyatt Riley
7d43d56883 Merge "Adding owners for gnss hardware interfaces" into oc-mr1-dev am: f5c1dc8d31
am: be60baf628

Change-Id: I0b961f42e44fcf53c470b07c6ffb35f5b3f2e1c4
2017-08-31 00:35:03 +00:00
Wyatt Riley
be60baf628 Merge "Adding owners for gnss hardware interfaces" into oc-mr1-dev
am: f5c1dc8d31

Change-Id: Ic10186038702d7aa5368e956abc1e22f17b996cc
2017-08-31 00:23:46 +00:00
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
Wyatt Riley
c0815b53e0 Merge "Setting clock flags for GPS clock" into oc-mr1-dev am: 67be3e2f64
am: 0c254185f9

Change-Id: Idb5c9982bd718b161e01fa32bc4d44aea1dfa2d1
2017-08-30 02:59:54 +00:00
Wyatt Riley
0c254185f9 Merge "Setting clock flags for GPS clock" into oc-mr1-dev
am: 67be3e2f64

Change-Id: I102f0c5c6e122a8f98d1c422dcc18670d72b2767
2017-08-29 23:41:53 +00:00
TreeHugger Robot
67be3e2f64 Merge "Setting clock flags for GPS clock" into oc-mr1-dev 2017-08-29 23:31:22 +00:00
Tri Vo
ba6d6e5767 VTS tests now static link to HAL def libs.
am: b65c5ceff2

Change-Id: I393284c12ca5cb79926c4891ffcfd332408a5b9f
2017-08-28 19:23:36 +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
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
Wyatt Riley
be612f3e51 Demoting Overactive Error Messagess
Change-Id: I4a2b30fc3ec077f819a4284034bbb02d02c0a464
Fixes: 36404093
Test: Boot and watch Info logs
2017-08-23 11:49:51 -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
Steven Moreland
4f25c2a143 Revert "Revert "Updating all makefiles.""
Relanding without automotive/vehicle/2.0+2.1 changes.

This reverts commit 1020ebedfb.

Merged-In: I8608c8f636c35f21e4246a805a9eff6d14124e0a
Change-Id: Icc49df42c2bedf443d878d5d8fbaa53441ea65db
2017-08-11 23:02:01 +00:00
Steven Moreland
ff308ea6ba Updating all makefiles.
Bug: 64487114
Test: none
Merged-In: I8608c8f636c35f21e4246a805a9eff6d14124e0a
Merged-In: I2fa89b6661c39859ec4fb62c4bb0a05a35e645f0
Change-Id: Ifdc3c17cb2b85c18b37dac2d03bb5c8935c23180
2017-08-11 22:58:47 +00:00
Steven Moreland
1020ebedfb Revert "Updating all makefiles."
This reverts commit 8ac1971678.

Reason for revert: Didn't remove automotive changes from this CL.

Merged-In: I8608c8f636c35f21e4246a805a9eff6d14124e0a
Change-Id: I1c660cffc8817ad0b33da9f6eceb3d88e7c48416
2017-08-11 22:56:56 +00:00
Steven Moreland
6da35570df Updating all makefiles.
Bug: 64487114
Test: none
Change-Id: I8608c8f636c35f21e4246a805a9eff6d14124e0a
2017-08-10 23:43:46 +00:00
Steven Moreland
8ac1971678 Updating all makefiles.
Bug: 64487114
Test: none
Merged-In: I8608c8f636c35f21e4246a805a9eff6d14124e0a
Change-Id: I2fa89b6661c39859ec4fb62c4bb0a05a35e645f0
2017-08-10 22:40:04 +00:00
Tri Vo
2b2ba2fa83 Merge "VTS tests now static link to HAL def libs. #2" into oc-mr1-dev am: 2883486706
am: a6969cdd68

Change-Id: Iecd7cfd4e02aea837647e2f12446e7375744f004
2017-08-10 18:38:53 +00: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
723c2dbd43 Update make file for vndk enabled. am: 608d773ef8
am: d66371512c

Change-Id: I35b778c39c2349346278deb6dea56b8cd3701f7d
2017-08-05 02:17:38 +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
Steven Moreland
6ae7aad171 Update makefiles (frameworks.jar).
Test: links at runtime and buildtime
Bug: 35771640
Change-Id: Ic7a71653d659115205e5bdbb782f73b774af0b29
2017-08-03 23:02:46 +00: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
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
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
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
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
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
Steven Moreland
d26dc50abb Always specify underlying enum type.
Bug: 33197891
Test: compiles
Change-Id: I3d4822ea9a665708302d3bd1460425fddf1dcda9
2016-11-29 14:22:10 -08:00
Steven Moreland
f8d68ec855 Update makefiles for logging.
Bug: 32943424
Test: pass
Change-Id: Ic88ce6adf04e0e36db53e4cc06ac2ec5a90c011c
2016-11-21 16:28:54 -08:00
Yifan Hong
6b920e43d3 Split libhidl into base and transport.
Bug: 32756130

Test: mma
Change-Id: Id21cf23abe2910f26bb7895511862443b93dc112
2016-11-18 00:00:54 -08:00
Colin Cross
b785f5b82e Support genrules with multiple tools
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
2016-11-04 22:49:15 -07:00
Steven Moreland
fcd74ebcdc Running update makefiles.
https://android-review.googlesource.com/#/c/298160/

Test: pass
Change-Id: I14fcde2d3aacc8f8cd9221f2d69e2feae9ae1226
2016-11-02 10:24:33 -07:00
Steven Moreland
69727778a7 Stop all these merge conflicts.
Test: mma -j64
Change-Id: I319198af32adf1fef180c5c2c22ba4b360fa7609
2016-10-26 12:04:37 +02:00
Hridya Valsaraju
49526a75de IGpsDebug Interface for GPS Hal
Bug: 31974439
Test: mma
Change-Id: I09097b158833c943cb5e7578673c454e63668278
2016-10-17 10:00:19 -07:00
Hridya Valsaraju
e596a71046 GPS HAL Binderization
A Debug interface as well as a configuration interface will be added in
another CL.

Bug: 31974439
Test: mma

Change-Id: I977d95fc815172bd2aae7c78f81e1fc7c9bce72a
2016-10-14 14:33:33 -07:00