OSDN Git Service
Kevin Rocard [Tue, 16 May 2017 19:13:11 +0000 (19:13 +0000)]
Merge changes from topic 'audio-primary-hal-vts-fix' into oc-dev
am:
c558ddf3fa
Change-Id: I15868923cdfd165e20468d1c885a256fefd0eeca
TreeHugger Robot [Tue, 16 May 2017 18:52:39 +0000 (18:52 +0000)]
Merge changes from topic 'audio-primary-hal-vts-fix' into oc-dev
* changes:
Audio HAL: A speech volume outside of [0,1] is an error
Audio HAL VTS: Some methods are not optional
Jeff Tinker [Tue, 16 May 2017 18:50:29 +0000 (18:50 +0000)]
Merge "Revert "Fix resource leaks in drm hal"" into oc-dev
am:
db15c4b6ee
Change-Id: Ib2b3078f95637fe3c4f4f6e0bd73498374f945f5
Howard Chen [Tue, 16 May 2017 18:30:21 +0000 (18:30 +0000)]
Add HIDL_FETCH_IScheduleTest declaration am:
82ddb247b2 am:
8acff9a619
am:
c57ac16490
Change-Id: I3471b2ff91acc53005fc391224de121c069437de
Howard Chen [Tue, 16 May 2017 18:30:19 +0000 (18:30 +0000)]
Add IScheduleTest for VTS libhwbinder scheduler test am:
35ae446548 am:
5a42dc1812
am:
80080bf2df -s ours
Change-Id: Id5a06a05a2cdff43623ecd7975274b8ca9adc809
Howard Chen [Tue, 16 May 2017 18:26:49 +0000 (18:26 +0000)]
Add HIDL_FETCH_IScheduleTest declaration am:
82ddb247b2
am:
8acff9a619
Change-Id: I3ba552765de4332f97367efc223457c0b641c084
Howard Chen [Tue, 16 May 2017 18:26:47 +0000 (18:26 +0000)]
Add IScheduleTest for VTS libhwbinder scheduler test am:
35ae446548
am:
5a42dc1812
Change-Id: I00a873e7233d6f29dd6c71bbac9d145d23038214
Jeff Tinker [Tue, 16 May 2017 18:25:50 +0000 (18:25 +0000)]
Merge "Revert "Fix resource leaks in drm hal"" into oc-dev
Howard Chen [Tue, 16 May 2017 18:24:18 +0000 (18:24 +0000)]
Add HIDL_FETCH_IScheduleTest declaration
am:
82ddb247b2
Change-Id: Idb6e9dca36c6e7b455c62dbc66d09def5f51fd65
Howard Chen [Tue, 16 May 2017 18:24:16 +0000 (18:24 +0000)]
Add IScheduleTest for VTS libhwbinder scheduler test
am:
35ae446548
Change-Id: I6e9b9866c0e7ab9e9beccd6137226a43dd33c858
Jeff Tinker [Tue, 16 May 2017 18:21:39 +0000 (18:21 +0000)]
Revert "Fix resource leaks in drm hal"
This caused b/
38343526 which is a chase list issue, reverting for now.
This reverts commit
4e1fe736af1b88485bba762b11bc0c11a447819c.
Change-Id: Ib9ffcd11e575e3f05f54cbea17fda2f2eb3780b5
Howard Chen [Mon, 8 May 2017 06:04:05 +0000 (14:04 +0800)]
Add HIDL_FETCH_IScheduleTest declaration
Fix the bug that makes the registerAsServer fail.
sp<IScheduleTest> server =
IScheduleTest::getService(serviceName, true);
status_t status = server->registerAsService(serviceName);
Bug:
38099333
Test: oc-dev on pixel sailfish with a vts test case
"libhwbinder_latency -i 1"
Change-Id: I892835a25ef7df56a6424da41c0fc477b3a532d8
Howard Chen [Fri, 7 Apr 2017 12:15:28 +0000 (20:15 +0800)]
Add IScheduleTest for VTS libhwbinder scheduler test
Add IScheduleTest.hal:
send(cfg, callerSta)
cfg: 1 for verbose output
caller_status: (caller_rt_priority <<16 )| caller_cpu
with the return value defined as:
((1 if no priority inheritance)<<16) | (1 if no cpu sync)
The implementation is in ScheduleTest.cpp
Bug:
36705188
Test: sailfish/prebuilt kernel/oc-dev with the libhwbinder_latency
vts test case
Merged-In: Ie2a837c8d0d4fa95c6fd0ebd50e76412cb808df8
Change-Id: Ic0498bc2b036cedced906963bb8e489dcca47a1a
Zhuoyao Zhang [Tue, 16 May 2017 00:08:54 +0000 (00:08 +0000)]
Merge "Omx vts tests." into oc-dev
am:
1d619f743d
Change-Id: I80b3c97d186f56210b761e1cd642c209e5071358
Zhuoyao Zhang [Mon, 15 May 2017 23:10:36 +0000 (23:10 +0000)]
Merge "Omx vts tests." into oc-dev
Ruchi Kandoi [Mon, 15 May 2017 22:24:47 +0000 (22:24 +0000)]
NFC: vts: Add commands to reset the NFC controller before other commands
am:
8e3c66e28b
Change-Id: Icc9fad4989d4d9e1b862f98bfcf3527476738366
Kevin Rocard [Tue, 9 May 2017 00:08:11 +0000 (17:08 -0700)]
Audio HAL: A speech volume outside of [0,1] is an error
Hals are supposed to received normalized volumes, between 0 and 1.
Previously volumes outside [0,1] were clamp to this range.
This clamping has the capability to hide bugs thus return an error if
such volume is received.
Test: vts-tradefed run vts --module VtsHalAudioV2_0Target
Test: call/play music/record/video...
Bug:
36311550
Change-Id: Iab70f9c651540ea2434d10939d28c1c842db19e0
Signed-off-by: Kevin Rocard <krocard@google.com>
Ruchi Kandoi [Fri, 12 May 2017 21:19:25 +0000 (14:19 -0700)]
NFC: vts: Add commands to reset the NFC controller before other commands
Bandwidth and WriteInvalidAndThenValidCommand test assumes that the NFC
controller will be initialized before the test. This is not necessarily
true for all devices. Send CORE_RESET_CMD and CORE_INIT_CMD before
creating a loop-back connection.
CORE_RESET_RSP doesn't necessary return the configuration that was sent
in CORE_RESET_CMD. For instance, if CORE_RESET_CMD requested keeping
the configuration, and if the controller doesn't support that, the
configuration will be resetted and CORE_RESET_RSP will reflect that.
Bug:
38267593
Test: vts-tradefed run commandAndExit vts --skip-all-system-status-check
--primary-abi-only --skip-preconditions
--module VtsHalNfcV1_0Target -l INFO
Change-Id: I65ae9746054ca767b928bdd461faa02f03719aea
Signed-off-by: Ruchi Kandoi <kandoiruchi@google.com>
(cherry picked from commit
209a274c31f8654000561b080f64ee84dd221ed9)
Jeff Tinker [Mon, 15 May 2017 20:03:22 +0000 (20:03 +0000)]
Merge "Fix resource leaks in drm hal" into oc-dev
am:
14b91049a7
Change-Id: Iedefddb99f498e968a22b50fe09c589f43907ff6
TreeHugger Robot [Mon, 15 May 2017 19:54:09 +0000 (19:54 +0000)]
Merge "Fix resource leaks in drm hal" into oc-dev
Ruchi Kandoi [Mon, 15 May 2017 17:27:27 +0000 (17:27 +0000)]
NFC: vts: Add commands to reset the NFC controller before other commands am:
209a274c31 am:
cd825026ca
am:
b5beef1765
Change-Id: I70dc4ec91c3ea52dec1bfddf6fc55634250b28fe
Ruchi Kandoi [Mon, 15 May 2017 17:20:24 +0000 (17:20 +0000)]
NFC: vts: Add commands to reset the NFC controller before other commands am:
209a274c31
am:
cd825026ca
Change-Id: I743c32a2b475540ba9df2137d90415c11222ea7d
Ruchi Kandoi [Mon, 15 May 2017 17:17:55 +0000 (17:17 +0000)]
NFC: vts: Add commands to reset the NFC controller before other commands
am:
209a274c31
Change-Id: I6d8cf3abf2dca69c984b313c593a01b282b23dd1
Ruchi Kandoi [Fri, 12 May 2017 21:19:25 +0000 (14:19 -0700)]
NFC: vts: Add commands to reset the NFC controller before other commands
Bandwidth and WriteInvalidAndThenValidCommand test assumes that the NFC
controller will be initialized before the test. This is not necessarily
true for all devices. Send CORE_RESET_CMD and CORE_INIT_CMD before
creating a loop-back connection.
CORE_RESET_RSP doesn't necessary return the configuration that was sent
in CORE_RESET_CMD. For instance, if CORE_RESET_CMD requested keeping
the configuration, and if the controller doesn't support that, the
configuration will be resetted and CORE_RESET_RSP will reflect that.
Bug:
38267593
Test: vts-tradefed run commandAndExit vts --skip-all-system-status-check
--primary-abi-only --skip-preconditions
--module VtsHalNfcV1_0Target -l INFO
Change-Id: I65ae9746054ca767b928bdd461faa02f03719aea
Signed-off-by: Ruchi Kandoi <kandoiruchi@google.com>
Ram Mohan M [Fri, 28 Apr 2017 06:48:35 +0000 (12:18 +0530)]
Omx vts tests.
Bug:
32023356
Test: make vts.
cover basic encoder & decoder test to all components
Till the previous commit, few components went untested.
This commit covers all softomx components.
Test: tested on softomx components
Change-Id: I4c996a3743a6190d9c6d4c835aa89daefdc14658
enumerate profile and level for encoder components
Change-Id: I4b9d3b3106d33a0241a7e36bd7189c1b3ef40ef0
bug fix: omx android compile as 32bit on 64bit platforms
thanks lajos
Change-Id: If11b20eeb6a75319f9e0cd6a62524dadab2ba8af
Add more VTS tests for IOmxStore and IOmxNode
Change-Id: I3401025b4f0b821f469222707c75b49fe4f0b0c5
avoid false flagging of errors
If testing of the given component is not supported application do not flag
an error, instead exit gracefully.
Change-Id: Ic91a92614d18b204cc50b7d8f6dc1a0ca81ff782
Add support for g711, gsm and raw audio decoder components
Change-Id: Ied70abdabb397a7a63e13293a8b35365a6b23e4f
split eos test
the eos test in decoders handles three different cases. These are now
separated in to different test fixtures.
Change-Id: I30c102109ce60a345f878d03d481337462fc817f
Change-Id: I3401025b4f0b821f469222707c75b49fe4f0b0c5
Kevin Rocard [Wed, 3 May 2017 18:16:05 +0000 (11:16 -0700)]
Audio HAL VTS: Some methods are not optional
Some mandatory methods were allowed to return NOT_SUPPORTED
although their implementations is mandatory.
Test: vts-tradefed run vts --module VtsHalAudioV2_0Target
Test: call/play music/record/video...
Bug:
36311550
Change-Id: Ibe4b3cf73257309975ed11269a38315051fa9064
Signed-off-by: Kevin Rocard <krocard@google.com>
Steven Moreland [Fri, 12 May 2017 21:55:18 +0000 (21:55 +0000)]
Merge "Update makefiles from anywhere." am:
a24592b396 am:
1ad1c673db
am:
40699c4d25
Change-Id: I8ac3efdf6efa470fc21d301ecb2410ef22f85337
Steven Moreland [Fri, 12 May 2017 21:49:39 +0000 (21:49 +0000)]
Merge "Update makefiles from anywhere." am:
a24592b396
am:
1ad1c673db
Change-Id: I40076615df8cd0b520f19c496e5d8ee8de5f6094
Steven Moreland [Fri, 12 May 2017 21:43:29 +0000 (21:43 +0000)]
Merge "Update makefiles from anywhere."
am:
a24592b396
Change-Id: I48f496f85a7da83e4506634ee9b90792471fac6c
Steven Moreland [Fri, 12 May 2017 21:33:36 +0000 (21:33 +0000)]
Merge "Update makefiles from anywhere."
Chia-I Wu [Fri, 12 May 2017 19:59:18 +0000 (19:59 +0000)]
Merge "graphics: set maxThreads to 4 for the composer" into oc-dev
am:
15e33c6d17
Change-Id: I4e5c759f467d6085e9dc1d57ca65db0273343eef
Chia-I Wu [Fri, 12 May 2017 19:50:55 +0000 (19:50 +0000)]
Merge "graphics: set maxThreads to 4 for the composer" into oc-dev
Myles Watson [Fri, 12 May 2017 18:15:30 +0000 (18:15 +0000)]
Merge "Bluetooth: Add a second thread for the scheduler" am:
2709c56ad5 am:
31f6040908
am:
f56e2909f9
Change-Id: Icd6191fb8f079126c1a4991683d9ee6b1015f18c
Myles Watson [Fri, 12 May 2017 18:10:27 +0000 (18:10 +0000)]
Merge "Bluetooth: Add a second thread for the scheduler" am:
2709c56ad5
am:
31f6040908
Change-Id: I4fb1e8f7a8d011b4c1cb59245dce5c9d20a05b2d
Myles Watson [Fri, 12 May 2017 18:08:28 +0000 (18:08 +0000)]
Merge "Bluetooth: Add a second thread for the scheduler"
am:
2709c56ad5
Change-Id: I0375940ba054d6b12d00957e96e0de860625c273
Treehugger Robot [Fri, 12 May 2017 18:03:54 +0000 (18:03 +0000)]
Merge "Bluetooth: Add a second thread for the scheduler"
Myles Watson [Fri, 12 May 2017 18:01:57 +0000 (18:01 +0000)]
Bluetooth: Add a second thread for the scheduler
am:
33f36706d0
Change-Id: I4730416f799b0947690d4a4544be41bfed191afb
Chia-I Wu [Thu, 11 May 2017 22:20:10 +0000 (15:20 -0700)]
graphics: set maxThreads to 4 for the composer
There are implmentations that can dead lock when calling certain
functions in certain states. It works fine before HIDL because
SurfaceFlinger calls those functions in another thread. We allow
for 4 hwbinder threads in this commit to simulate how SurfaceFlinger
calls hwcomposer.
Bug:
38183197
Test: camera, videos, multi windows, screencap, screenrecord
Change-Id: Ie05b2ca349b8c1ed9a3ac962981434f2efee92d9
Kevin Rocard [Fri, 12 May 2017 06:57:47 +0000 (06:57 +0000)]
Merge changes from topic 'audio-primary-hal-vts-fix' into oc-dev
am:
257c2f1a43
Change-Id: Ic8850d915b5672e7529f99e7d57845b2de44857a
Jaesoo Lee [Fri, 12 May 2017 05:16:36 +0000 (05:16 +0000)]
configstore: include types.h instead of ISurfaceFlingerConfigs.h am:
92c77bd6e7 am:
4b1848247f
am:
d21a105380
Change-Id: I1a674bd0827b0dccdb01fded168cf3122de81afa
Jaesoo Lee [Fri, 12 May 2017 05:12:35 +0000 (05:12 +0000)]
configstore: include types.h instead of ISurfaceFlingerConfigs.h am:
92c77bd6e7
am:
4b1848247f
Change-Id: I7563d66174d3fd5c6c7a35389bb5d5ed78c7ac3c
Jaesoo Lee [Fri, 12 May 2017 05:10:34 +0000 (05:10 +0000)]
configstore: include types.h instead of ISurfaceFlingerConfigs.h
am:
92c77bd6e7
Change-Id: I2782872f927b8fa9a0cc1da877d0f1cac3680150
Myles Watson [Thu, 11 May 2017 23:39:59 +0000 (16:39 -0700)]
Bluetooth: Add a second thread for the scheduler
Test: service call bluetooth_manager 6 # (Stop Bluetooth)
/data/nativetest64/VtsHalBluetoothV1_0TargetTest/\
VtsHalBluetoothV1_0TargetTest
Bug:
38178265,
34461621
Change-Id: Ia8f6683ca47c9deac43d1047846caee5688f1eca
(cherry picked from commit
74c601361c5f2d68561b545b7b7a52a3916ba8e9)
Myles Watson [Thu, 11 May 2017 23:39:59 +0000 (16:39 -0700)]
Bluetooth: Add a second thread for the scheduler
Test: service call bluetooth_manager 6 # (Stop Bluetooth)
/data/nativetest64/VtsHalBluetoothV1_0TargetTest/\
VtsHalBluetoothV1_0TargetTest
Bug:
38178265,
34461621
Change-Id: Ia8f6683ca47c9deac43d1047846caee5688f1eca
Steven Moreland [Thu, 11 May 2017 22:42:32 +0000 (15:42 -0700)]
Update makefiles from anywhere.
Test: with update-all-google-makefiles.sh
Change-Id: I373e270b481d9074f440c3c0826199a207e654f7
Kevin Rocard [Thu, 11 May 2017 22:34:39 +0000 (22:34 +0000)]
Merge changes from topic 'audio-primary-hal-vts-fix' into oc-dev
* changes:
Audio HAL: A volume/gain outside of [0,1] is an error
Audio HAL VTS: Log test unexpected behaviour
Audio HAL VTS: Improve ASSERT of Result and Return
Audio HAL VTS: stopped stream state getters may return INVALID_STATE
Audio HAL VTS: setGain allow -0.0
Audio HAL VTS: Some methods are optional
Audio HAL VTS: GetPresentationPosition may return 0 on stop stream
Audio HAL VTS: getDevice() == NONE => not supported
Audio HAL VTS: Getter test assert logic was incorrect
Audio HAL VTS: debugDump can only test for crash
Audio HAL VTS: Sanitize prepareFor{Writing,Reading} input size
Audio HAL VTS: refactor prepareFor{Reading,Writing}
Audio HAL VTS: Sanitize setMode input
Audio HAL VTS: Allow OK when setting a non existing parameter
Audio HAL VTS: differentiate getParam success/failure/not_implemented
Audio HAL VTS: Fix style on modified files
Steven Moreland [Thu, 11 May 2017 00:55:46 +0000 (00:55 +0000)]
Merge changes from topic 'fix-hidl-generate' into oc-dev
am:
4be51a8512 -s ours
Change-Id: I91054a4a0ffcf6ff117bd4b6c3dcc3a94e7231c2
Steven Moreland [Thu, 11 May 2017 00:49:04 +0000 (00:49 +0000)]
Update makefiles for c++-sources and c++-headers.
am:
8716f3063e -s ours
Change-Id: If3655769859781369e8cb8456f133e0c35636254
Steven Moreland [Thu, 11 May 2017 00:48:55 +0000 (00:48 +0000)]
Update makefiles for c++-sources and c++-headers. (2/2)
am:
bc71124120
Change-Id: Icf36068a4d863a8520854c7a83e51f960c188e12
Martijn Coenen [Thu, 11 May 2017 00:47:30 +0000 (00:47 +0000)]
Merge "Start processes that need real-time with CAP_SYS_NICE." into oc-dev
am:
6c062f77c1
Change-Id: I921160390dc9f90e7872729194ed1d1240ce68b7
Jaesoo Lee [Tue, 2 May 2017 06:15:14 +0000 (15:15 +0900)]
configstore: include types.h instead of ISurfaceFlingerConfigs.h
Utils.h does not need to include ISurfaceFlingerConfigs.h.
Bug:
37727469
Test: successfully built sailfish-userdebug
Change-Id: I6650020bfc3b265d871ca3244b8900c336f98419
Steven Moreland [Wed, 10 May 2017 22:51:15 +0000 (22:51 +0000)]
Actually fix build breakage with cleanspec. am:
319b8292be am:
b16a4d26d3
am:
6ad2f6c984
Change-Id: I09a208dcd24cf17d3171ed5d4f64282a2642dd56
Steven Moreland [Wed, 10 May 2017 22:48:44 +0000 (22:48 +0000)]
Actually fix build breakage with cleanspec. am:
319b8292be
am:
b16a4d26d3
Change-Id: I235f68723215a806317828b65d9b8905598b10eb
Steven Moreland [Wed, 10 May 2017 22:47:14 +0000 (22:47 +0000)]
Actually fix build breakage with cleanspec.
am:
319b8292be
Change-Id: Ic9dcbf28c238b30eba296fedcdef08af049aef31
Steven Moreland [Wed, 10 May 2017 22:32:32 +0000 (15:32 -0700)]
Actually fix build breakage with cleanspec.
OUT_DIR instead of OUT
Test: manual
Change-Id: I2e48c6f2f39ff2318517ff55cbfa6ca912c05021
Steven Moreland [Wed, 10 May 2017 21:03:08 +0000 (21:03 +0000)]
Add CleanSpec to hardware/interfaces. am:
e468cf8004 am:
7780055fd1
am:
aae66d171a
Change-Id: Id1a1c32443e342915dd40147560bb1987d782f9d
Steven Moreland [Wed, 10 May 2017 20:58:37 +0000 (20:58 +0000)]
Add CleanSpec to hardware/interfaces. am:
e468cf8004
am:
7780055fd1
Change-Id: I5d5f2618fc6d18034439d3cb1c31290d825f1c2e
Steven Moreland [Wed, 10 May 2017 20:55:36 +0000 (20:55 +0000)]
Add CleanSpec to hardware/interfaces.
am:
e468cf8004
Change-Id: If1978c73a668dacaf8f3b2842a704c5b28d5135e
Steven Moreland [Wed, 10 May 2017 20:37:21 +0000 (13:37 -0700)]
Add CleanSpec to hardware/interfaces.
Fixes build breakage from interface changing after -Lc++-sources
and -Lc++-headers was added to hidl-gen.
Test: wip
Change-Id: I9ee0a0e7b72f8f2a3253d205943a696c07129a64
Steven Moreland [Wed, 10 May 2017 20:02:38 +0000 (20:02 +0000)]
Merge changes from topic 'fix-hidl-generate' into oc-dev
* changes:
Update makefiles for c++-sources and c++-headers.
Update makefiles for c++-sources and c++-headers. (2/2)
Martijn Coenen [Wed, 10 May 2017 17:55:46 +0000 (17:55 +0000)]
Merge "Start processes that need real-time with CAP_SYS_NICE." into oc-dev
Steven Moreland [Tue, 9 May 2017 21:34:19 +0000 (14:34 -0700)]
Update makefiles for c++-sources and c++-headers.
Test: pass
Bug:
38174080
Merged-In: I683455736eed35e6ced24e2763c2bdb5fb6c9601
Change-Id: I683455736eed35e6ced24e2763c2bdb5fb6c9601
Steven Moreland [Wed, 10 May 2017 02:51:19 +0000 (19:51 -0700)]
Update makefiles for c++-sources and c++-headers. (2/2)
Test: pass
Bug:
38174080
Change-Id: Icad451f9f9ad5f46412356a4171a8a1222f8e545
Steven Moreland [Wed, 10 May 2017 02:58:00 +0000 (02:58 +0000)]
Update makefiles for c++-sources and c++-headers. am:
cb360c60d8 am:
4bd4e7fb3b
am:
127b116fe5
Change-Id: I3a2aa76acef0f0d1455bfad775e87e8089d357e8
Steven Moreland [Wed, 10 May 2017 02:52:52 +0000 (02:52 +0000)]
Update makefiles for c++-sources and c++-headers. am:
cb360c60d8
am:
4bd4e7fb3b
Change-Id: Ief274574dda12a38f62f49953a88f612d5b3586c
Steven Moreland [Wed, 10 May 2017 02:47:25 +0000 (02:47 +0000)]
Update makefiles for c++-sources and c++-headers.
am:
cb360c60d8
Change-Id: If0f0c69fa29a8c1cbd0d5b4e53e386ef8b68757e
PO HUNG CHEN [Wed, 10 May 2017 02:42:02 +0000 (02:42 +0000)]
Merge "Bugfix: Add HIDL_FETCH_IScheduleTest declaration" into oc-dev
am:
c365952316
Change-Id: If6399162d868c7498ca6e4a5d9453b4425ab77cc
Sanket Padawe [Wed, 10 May 2017 02:41:46 +0000 (02:41 +0000)]
Merge "Update IRadioResponse.hal documentation to fix failing VTS" into oc-dev
am:
41ea816c9d
Change-Id: I3f9efd6422652efc967a9c8a5bda284f87a4a458
PO HUNG CHEN [Wed, 10 May 2017 02:28:10 +0000 (02:28 +0000)]
Merge "Bugfix: Add HIDL_FETCH_IScheduleTest declaration" into oc-dev
Sanket Padawe [Wed, 10 May 2017 02:27:04 +0000 (02:27 +0000)]
Merge "Update IRadioResponse.hal documentation to fix failing VTS" into oc-dev
Naina Nalluri [Fri, 5 May 2017 18:00:05 +0000 (11:00 -0700)]
Update IRadioResponse.hal documentation to fix failing VTS
update IRadioResponse.hal documentation with
nas/data error codes
Bug:
35671595
Test: Only doc update.
Change-Id: I5a106e05b487f7deecb208af6465e5ea5238b26e
Steven Moreland [Tue, 9 May 2017 21:34:19 +0000 (14:34 -0700)]
Update makefiles for c++-sources and c++-headers.
Test: pass
Bug:
38174080
Change-Id: I683455736eed35e6ced24e2763c2bdb5fb6c9601
Eino-Ville Talvala [Tue, 9 May 2017 21:32:28 +0000 (21:32 +0000)]
Camera: Remove TODOs in interfaces
am:
71619beef7
Change-Id: I43e85210ab1972f8245376831f578816be4dea78
Eino-Ville Talvala [Tue, 9 May 2017 21:32:10 +0000 (21:32 +0000)]
Merge "Camera: Clean up TODOs that aren't going to be addressed." into oc-dev
am:
0b3d86e05a
Change-Id: Ic0ca4e1eff6716466ef86bc6a660bc371da875d6
Ram Periathiruvadi [Tue, 9 May 2017 21:31:44 +0000 (21:31 +0000)]
Merge "Add DOOR_LOCK to default config." into oc-dev
am:
c63bdfbd74
Change-Id: I2386d09d237957da9f7ea146412b5a36cbf208a6
Eino-Ville Talvala [Tue, 9 May 2017 18:13:20 +0000 (11:13 -0700)]
Camera: Remove TODOs in interfaces
These are tracked elsewhere and won't be addresed for the initial
interface versions.
Update current.txt hashes; comment-only update
Test: Compiles
Bug:
34108979
Bug:
33269977
Bug:
33262893
Change-Id: I0c0521cf69649d871f64a67773ce822db2487044
Jeff Tinker [Tue, 9 May 2017 02:27:10 +0000 (19:27 -0700)]
Fix resource leaks in drm hal
bug:
36408047
test: manual tests to verify leaks are fixed
Change-Id: Idd79e99e211d81f8b3e58cf52c9b01ce9d370dc4
TreeHugger Robot [Tue, 9 May 2017 17:30:35 +0000 (17:30 +0000)]
Merge "Camera: Clean up TODOs that aren't going to be addressed." into oc-dev
TreeHugger Robot [Tue, 9 May 2017 17:21:13 +0000 (17:21 +0000)]
Merge "Add DOOR_LOCK to default config." into oc-dev
Martijn Coenen [Tue, 9 May 2017 16:57:00 +0000 (09:57 -0700)]
Start processes that need real-time with CAP_SYS_NICE.
Bug:
37293077
Test: verified CAP_SYS_NICE granted in /proc/pid/status
Change-Id: If404450cbeb077d331b231bda1af2d6eae774b75
Kevin Rocard [Tue, 9 May 2017 00:08:11 +0000 (17:08 -0700)]
Audio HAL: A volume/gain outside of [0,1] is an error
Hals are supposed to received normalized volumes, between 0 and 1.
Previously volumes outside [0,1] were clamp to this range.
This clamping has the capability to hide bugs thus return an error if
such volume is received.
Test: vts-tradefed run vts --module VtsHalAudioV2_0Target
Test: call/play music/record/video...
Bug:
36311550
Change-Id: Ia4880bdff6111cbcdae6a4ebee921eddae141ee4
Signed-off-by: Kevin Rocard <krocard@google.com>
Eino-Ville Talvala [Mon, 8 May 2017 22:29:28 +0000 (15:29 -0700)]
Camera: Clean up TODOs that aren't going to be addressed.
These TODOs aren't in scope for the wrapper libraries.
Leave the few TODOs that have bugs tracking them, and the ones that
are from autogenerated metadata.
Test: Builds
Bug:
34108979
Change-Id: Ib28467a4fbcfaab4a0809334bc62eabb9290797a
Yifan Hong [Mon, 8 May 2017 22:12:06 +0000 (22:12 +0000)]
Merge "Revert "Set omx and configstore as optional."" into oc-dev
am:
b8e84d240f
Change-Id: Iaf5f7aacacd2eb72b1ba98e19f5fe7589580bdbd
Kevin Rocard [Mon, 8 May 2017 18:53:07 +0000 (11:53 -0700)]
Audio HAL VTS: Log test unexpected behaviour
Some test output infos that are useful to understand how they run.
Unfortunately the xml report does not seem to be saved by VTS tradefed.
Thus output them in logcat.
Test: vts-tradefed run vts --module VtsHalAudioV2_0Target
Test: call/play music/record/video...
Bug:
36311550
Change-Id: I9a2cc10160c3b1c8f81db0464efbc6b26600cadc
Signed-off-by: Kevin Rocard <krocard@google.com>
Kevin Rocard [Wed, 3 May 2017 02:21:58 +0000 (19:21 -0700)]
Audio HAL VTS: Improve ASSERT of Result and Return
Previously tests on Result and Return were using ASSERT_NO_FATAL_FAILURE
and helper methods.
This leaded to complex error messages were the error did not pointed to
the helper methods instead of the ASSERT_RESULT call.
Additionally SCOPE_TRACE messages are repeated
for each ASSERT_NO_FATAL_FAILURE level.
Use ::testing::AssertionResult to improve drastically the error messages.
Test: vts-tradefed run vts --module VtsHalAudioV2_0Target
Test: call/play music/record/video...
Bug:
36311550
Change-Id: If705502546606d678df3f1966b0cd9f3ef8c2529
Signed-off-by: Kevin Rocard <krocard@google.com>
Kevin Rocard [Wed, 3 May 2017 17:57:06 +0000 (10:57 -0700)]
Audio HAL VTS: stopped stream state getters may return INVALID_STATE
A never started stream should have its render position and next write
timestamp at 0 or indicate that the state is invalid.
Test: vts-tradefed run vts --module VtsHalAudioV2_0Target
Test: call/play music/record/video...
Bug:
36311550
Change-Id: I62e16066bb22101ee8f75154fc6c85a66be2f402
Signed-off-by: Kevin Rocard <krocard@google.com>
Kevin Rocard [Wed, 3 May 2017 18:19:25 +0000 (11:19 -0700)]
Audio HAL VTS: setGain allow -0.0
Previously -0.0 was consider an invalid value as not in the range [0,1].
But it is quite difficult in C++ to differentiate -0.0 and 0.0 as -0.0 == 0.0
and such difference has no impact in practice.
Thus leave the implementation support or not -0.0.
Test: vts-tradefed run vts --module VtsHalAudioV2_0Target
Test: call/play music/record/video...
Bug:
36311550
Change-Id: Ia0ebcb325f77adcf8471620f418da1cbe8995e36
Signed-off-by: Kevin Rocard <krocard@google.com>
Kevin Rocard [Wed, 3 May 2017 18:16:05 +0000 (11:16 -0700)]
Audio HAL VTS: Some methods are optional
Although the method documentation does not say it,
some HIDL interface methods are optional.
Update the tests to allow NOT_SUPPORTED to be returned.
Test: vts-tradefed run vts --module VtsHalAudioV2_0Target
Test: call/play music/record/video...
Bug:
36311550
Change-Id: If31acc2dbdb6d1d563910e85c99401c48f4f3f86
Signed-off-by: Kevin Rocard <krocard@google.com>
Kevin Rocard [Wed, 3 May 2017 17:52:43 +0000 (10:52 -0700)]
Audio HAL VTS: GetPresentationPosition may return 0 on stop stream
GetPresentationPosition returns the last time a sample was written to
the hardware.
Calling it on a stop stream should return 0 sample written, but the
timestamps of that measure is of little importance.
Thus allow this timestamp to be 0 indicating that the measure was never
actually made.
Test: vts-tradefed run vts --module VtsHalAudioV2_0Target
Test: call/play music/record/video...
Bug:
36311550
Change-Id: I3c33b60f98e8cbea269a7739cc1889af932dcff5
Signed-off-by: Kevin Rocard <krocard@google.com>
Kevin Rocard [Wed, 3 May 2017 02:34:29 +0000 (19:34 -0700)]
Audio HAL VTS: getDevice() == NONE => not supported
getDevice does not return a Result, thus it can not return NOT_SUPPORTED.
Consider NONE as not supported.
Test: vts-tradefed run vts --module VtsHalAudioV2_0Target
Test: call/play music/record/video...
Bug:
36311550
Change-Id: I3b6f7a1fbc1d1535faf549f5b031461cb39d1722
Signed-off-by: Kevin Rocard <krocard@google.com>
Kevin Rocard [Wed, 3 May 2017 01:58:58 +0000 (18:58 -0700)]
Audio HAL VTS: Getter test assert logic was incorrect
Test: vts-tradefed run vts --module VtsHalAudioV2_0Target
Test: call/play music/record/video...
Bug:
36311550
Change-Id: Iaf2d71829a15b12dcf56e825773c8a697896a264
Signed-off-by: Kevin Rocard <krocard@google.com>
Kevin Rocard [Wed, 3 May 2017 01:41:46 +0000 (18:41 -0700)]
Audio HAL VTS: debugDump can only test for crash
DebugDump does not return an error code, thus the implementation can not
return not implemented.
As a result, the test can not expect any output from the function.
Only test that the call does not crash and add a log if the function is
probably not implemented in the test.
Test: vts-tradefed run vts --module VtsHalAudioV2_0Target
Test: call/play music/record/video...
Bug:
36311550
Change-Id: I2c18958bceb1eb638491f9afce9d8e8025ccd3ec
Signed-off-by: Kevin Rocard <krocard@google.com>
Kevin Rocard [Wed, 3 May 2017 01:34:59 +0000 (18:34 -0700)]
Audio HAL VTS: Sanitize prepareFor{Writing,Reading} input size
Return an error if framesCount or frameSize are null to avoid a division
by zero when calculating the buffer size.
The message queues are allocated with a buffer size but if two big they will
assert not return an error.
Thus take some margin on the buffer size check.
Note that both function should be refactored as 99% identical.
Test: vts-tradefed run vts --module VtsHalAudioV2_0Target
Test: call/play music/record/video...
Bug:
36311550
Change-Id: I0576e9016ef2e567c8d4e171c6237883d9865db9
Signed-off-by: Kevin Rocard <krocard@google.com>
Kevin Rocard [Wed, 3 May 2017 01:31:24 +0000 (18:31 -0700)]
Audio HAL VTS: refactor prepareFor{Reading,Writing}
Those functions had lots of copy paste on errors and
the following patch will even add more error detections.
Refactor the hidl_cb call to avoid all duplication.
Note that both function should be refactored as 99% identical.
Test: vts-tradefed run vts --module VtsHalAudioV2_0Target
Test: call/play music/record/video...
Bug:
36311550
Change-Id: I40d6926b4f9f5e3aba51e878f55fb013f4ca09c1
Signed-off-by: Kevin Rocard <krocard@google.com>
Kevin Rocard [Wed, 3 May 2017 01:16:00 +0000 (18:16 -0700)]
Audio HAL VTS: Sanitize setMode input
Some values of AudioMode are exposed although implementation detail.
Make sure the client can not use them.
Test: vts-tradefed run vts --module VtsHalAudioV2_0Target
Test: call/play music/record/video...
Bug:
36311550
Change-Id: If513c2a06efa8a92459f0af80c63232fc63302b0
Signed-off-by: Kevin Rocard <krocard@google.com>
Kevin Rocard [Wed, 3 May 2017 17:43:21 +0000 (10:43 -0700)]
Audio HAL VTS: Allow OK when setting a non existing parameter
setHwAvSync and setParameters were implemented in the pre-hidl interface
as set_parameters.
Unfortunately set_parameters did not return an error if a key was not
implemented.
As most HIDL implementation will be a wrapper around the pre-hidl
interface, allow those functions to return OK on not implemented key.
Test: vts-tradefed run vts --module VtsHalAudioV2_0Target
Test: call/play music/record/video...
Bug:
36311550
Change-Id: Icfcaa02b7d63e03375fddc90dc5a803754c1874f
Signed-off-by: Kevin Rocard <krocard@google.com>
Yifan Hong [Mon, 8 May 2017 21:10:56 +0000 (21:10 +0000)]
Merge "Revert "Set omx and configstore as optional."" into oc-dev
Kevin Rocard [Wed, 3 May 2017 00:38:34 +0000 (17:38 -0700)]
Audio HAL VTS: differentiate getParam success/failure/not_implemented
When sending parameters to the HAL (and some getters are implemented
with getParameters), the client expect a status consistent
with the other HIDL methods. Ie: not implemented or success and failure.
Unfortunately, the legacy get_parameter interface, which currently most
Audio HIDL implementation are a wrapper around, do not return such error
code.
Get parameters return a list of key values.
- If a requested key does not return a key value pair, consider it not
implemented
- If a requested key returns a key not followed by a correct value,
consider it a failure
- otherwise it is a success
Test: vts-tradefed run vts --module VtsHalAudioV2_0Target
Test: call/play music/record/video...
Bug:
36311550
Change-Id: Id6711e9c1974fe5a336b6de83a9b6d14f74437c9
Signed-off-by: Kevin Rocard <krocard@google.com>
Kevin Rocard [Fri, 5 May 2017 21:02:55 +0000 (14:02 -0700)]
Audio HAL VTS: Fix style on modified files
In order to avoid style inconsistency as well as non functional
modification in following patches,
fix the style of all files modified for the fix of the VTS tests.
Patch generated with:
$ clang-format -i --style file -- <file list>
Bug:
36311550
Test: compile
Change-Id: I53dbcdabf959a6100e34a2ee4d0f951d525049cb
Ram Periathiruvadi [Thu, 4 May 2017 18:06:17 +0000 (11:06 -0700)]
Add DOOR_LOCK to default config.
One of the triggers for Bluetooth Auto connect is a door unlock signal.
Adding it to the list of default supported properties.
Bug: b/
37674555
Test: Auto connect triggers on adb shell dumpsys activity service
com.android.car inject-event zoned-boolean 0x16200b02 1 false
Change-Id: Icfed9ea57239cd2397f3e71dcfaccd767a2e9739
(cherry picked from commit
e17a009d5de6601e0482cc5495aef16c562c6e30)