summaryrefslogtreecommitdiffstats
path: root/soundtrigger
Commit message (Collapse)AuthorAgeFilesLines
* Convert soundtrigger hal test to use VtsHalHidlTargetTestEnvBaseZhuoyao Zhang2018-02-122-18/+51
| | | | | | | | | Bug: 64203181 Test: make vts vts-tradefed run vts -m VtsHalSoundtriggerV2_0Target vts-tradefed run vts -m VtsHalSoundtriggerV2_1Target Change-Id: I747d674e4c0016cd8a1c373c8d7f744a022a98d8
* Merge "Revert "Audio V4: Split system and vendor Audio.h""Kevin Rocard2018-01-247-377/+5
|\
| * Revert "Audio V4: Split system and vendor Audio.h"Kevin Rocard2018-01-247-377/+5
| | | | | | | | | | | | | | | | This reverts commit 49c56de516b4e9556ae1b0af0ccfa7d695f5807b. Reason for revert: Breaks the build of multiple devices Change-Id: Iebd868467948b8afa5907462ccc0720cd9c4871e
* | Merge "Audio V4: Split system and vendor Audio.h"Kevin Rocard2018-01-247-5/+377
|\|
| * Audio V4: Split system and vendor Audio.hKevin Rocard2018-01-227-5/+377
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | audio.h and its dependencies (audio-effect.h, sound_trigger.h...) used to be shared between system and vendor code. This led to multiple problems: 1) Such sharing contradicts the Treble policy of strict independence of framework and vendor code. 2) When audio.h was changed, every vendor needed to update its code in the next release. This meant that audio*.h headers were mostly changed in backward compatible manner. Nevertheless, for P the HIDL interface and thus the audio.h interface are changed in backward incompatible way. (Some enum are becoming 64 bit long). 3) As the headers were common, some API used only by the framework needed to be visible to the vendors (mostly enum values). 4) Treble policy is to support at least one previous HAL version As a result the audio*.h headers are now duplicated, one set for the framework, and one for the vendor. Each set will evolve independently. After this split, the framework-only APIs will be removed from the vendor headers and vice versa. The split is implements as such: + for system code - NOT moving the libaudio_system_headers Eg: system/audio.h and system/audio_effects/effect_equalizer.h are still in system/media/audio - the legacy audio HAL API that were in libhardware headers are now in libaudiohal_legacy_headers Eg: hardware/audio.h and hardware/audio_effect.h are now in frameworks/av/media/libaudiohal/legacy/ + for vendor code - moving libaudio_system_headers and the legacy audio HAL API that were in libhardware_headers in android.hardware.audio.common.legacy@2.0 Note that those headers are now versioned, so migrating to a @4.0 HIDL HAL will mean changing the legacy dependency too. Eg: system/audio.h, system/audio-effect.h, hardware/audio.h are now in hardware/interfaces/audio/common/2.0/legacy - the legacy audio effect HAL API that was in libaudioeffects is now moved in android.hardware.audio.effect.legacy@2.0 Eg: audio_effects/effect*.h are now in hardware/interfaces/audio/effect/2.0/legacy - the legacy sound trigger HAL API that were in libhardware_headers is now moved in android.hardware.soundtrigger.legacy@2.0 Eg: hardware/sound_trigger.h is now in hardware/interfaces/audio/effect/2.0/legacy libaudioutil being used by both system and vendor, had to be renamed for system to libaudioutil_system. Vendor libs that now depend on the audio.h of a specific version and are not extensively referenced in non google code, append @2.0 to their name. Note that headers that are not expected to change in the 4.0 HAL are left in all-versions folder to avoid duplication. This is an implementation detail as the versioned libraries export the all-versions headers. Note that strict vendor-system separation is enforced by the build-system. The system headers are not available for vendor libs and vice-versa. Note that this patch is split between numerous git repository (>10), all the commits having the same Change-id for searchability. Note that audio_policy.h is no longer exposed to vendors as the legacy audio policy HAL API was never officially supported. As a result the audiopolicy stub implementation has been removed. Test: compile taimen-userdebug walleye-userdebug sailfish-userdebug marlin-userdebug gce_x86_phone-userdebug gce_x86_phone full-eng aosp_arm aosp_x86-eng Test: check that the emulator booted and played audio Test: full QA on sailfish-userdebug and taimen-userdebug Bug: 38184704 Change-Id: I950f4e0a55613d72e32eba31bd563cb5bafe2d1a Signed-off-by: Kevin Rocard <krocard@google.com>
* | soundtrigger: Remove get/setParameters interface methodsMikhail Naganov2018-01-233-168/+2
|/ | | | | | | | | These methods are not aligned with "structured interface" approach. Bug: 68823037 Test: make Change-Id: Id73d676e0386125bca678e7c7c2ea006c73a1365
* soundtrigger: Add VTS tests for v2.1Mikhail Naganov2018-01-172-0/+586
| | | | | | | | | | Re-tests the implementation of the inherited methods from v2.0. Adds tests for the methods introduced in v2.1. Bug: 68823037 Test: VtsHalSoundtriggerV2_1TargetTest Change-Id: Iab4d397d2581da1ff9e0d87c7f8d4fa8b483cdd5
* soundtrigger: Default implementation for version 2.1Mikhail Naganov2018-01-173-0/+395
| | | | | | | | | Wrapped around the 2.0 default implementation. Added functionality for retrieving and sending the data via shared memory. Bug: 68823037 Test: compiles Change-Id: Ie38dd261dc8c635462a7e2ee26672a83af915e84
* soundtrigger: Refactor the default implementation to be extensibleMikhail Naganov2018-01-175-134/+233
| | | | | | | | | | | | | | | | | | Introduce an inner "trampoline" class that implements ISoundTriggerHw. This allows minor uprev implementation to inherit from SoundTriggerHalImpl and reuse its functionality. Split SoundModelClient into an abstract common part and version-specific part. This allows the client to be redefined for the types used in callback interface extensions. Split the impl library into "core" part and the part implementing HIDL_FETCH_ISoundTriggerHw function to avoid clash with the same function introduced in minor uprev implementation. Bug: 68823037 Change-Id: Ibec647f1aa7bc6a2a0bdfd1c9f9a066e4779a1bf Test: make
* soundtrigger: Apply clang-format to 2.0 default implementationMikhail Naganov2018-01-172-236/+180
| | | | | | | | | | | | | | The implementation needs to be updated to become extensible. The current coding style is not consistent with the style required for hardware/interfaces. Reformatted with: clang-format -style=file -i <...>/SoundTriggerHalImpl.* No manual changes. Bug: 68823037 Change-Id: I5f5cbb599190a515dec8afe9026ca835b8fd20fa Test: make
* soundtrigger: Define v2.1 HALMikhail Naganov2018-01-173-0/+345
| | | | | | | | | | | | | | | SoundTrigger 2.1 HAL is an extension to 2.0 fixing the issue with of data blobs of arbitrary size that can't outgrow HwBinder's IPC transaction size limit. Instead of vectors, shared memory handles are used. Added methods for getting and setting parameters. The parameters are opaque to the framework, it does not interpret them. Bug: 68823037 Change-Id: I01c0c867e1d13023a22ccf408e84da78947d3bc2 Test: run hardware/interfaces/update-makefiles.sh
* Remove subdirsSteven Moreland2017-11-281-5/+0
| | | | | | | | Removing whenever I see these in code reviews. Test: none Merged-In: I4322f533a837d55618ec2ed2125e8966ace9d61d Change-Id: I4322f533a837d55618ec2ed2125e8966ace9d61d
* Merge "Use -Werror in hardware/interfaces" am: 02804915fb am: 8e736c21deChih-Hung Hsieh2017-11-281-0/+2
|\ | | | | | | | | | | am: f53fe9317b Change-Id: Ifc0a5be285be1a57d998db3c032e392b307cfa80
| * Merge "Use -Werror in hardware/interfaces"Chih-Hung Hsieh2017-11-281-0/+2
| |\ | | | | | | | | | | | | | | | am: 02804915fb Change-Id: Id73af9532487a85ad1d1170dbce809761d4a93a8
| | * Use -Werror in hardware/interfacesChih-Hung Hsieh2017-11-201-0/+2
| | | | | | | | | | | | | | | | | | | | | | | | * Use hidl_defaults, which has -Wall -Werror. Bug: 66996870 Test: build with WITH_TIDY=1 Change-Id: Ifbca0e176a8e031acc27256eb46d23b89cdf1a0e
| * | Merge commit '1a06284b24f5eb7bb9c1fea0817da8898b3b1bff' fromXin Li2017-11-141-0/+3
| |\ \ | | |/ | |/| | | | | | | | | | | | | oc-mr1-dev-plus-aosp into stage-aosp-master Change-Id: I2a044eb8c9981d0a8198ffe2df55559afbd76341 Merged-In: I4fb9f18884f7ef21162015a0032c4431444f7025
| * | Update makefiles for hidl_interface.Steven Moreland2017-11-101-147/+14
| | | | | | | | | | | | | | | | | | Bug: 35570956 Test: manual Change-Id: I7a220b78ee081240e1dc30ef5672ba39e3e98375
* | | Updating makefiles for hidl_interface.Steven Moreland2017-11-131-147/+14
| | | | | | | | | | | | | | | | | | | | | Bug: 64487114 Test: manual Merged-In: Ie13d9e014cf2b81c18c67f551b4644fb9f0ba812 Change-Id: Ie13d9e014cf2b81c18c67f551b4644fb9f0ba812
* | | Update for Soong java makefiles. am: c3e80fa01e am: 0fff75dee1 am: 6c811964a1Steven Moreland2017-10-111-1/+5
|\ \ \ | | |/ | |/| | | | | | | | | | am: 116161d94e Change-Id: I33643636e2511de77fdf7de57777eb67edee2e52
| * | Update for Soong java makefiles. am: c3e80fa01e am: 0fff75dee1Steven Moreland2017-10-111-1/+5
| |\| | | | | | | | | | | | | | | | am: 6c811964a1 Change-Id: I85ccbb4a15cd18938607f5bca4e065b9d7e0182b
| | * Update for Soong java makefiles.Steven Moreland2017-10-101-1/+5
| | | | | | | | | | | | | | | | | | Test: pass Bug: 33420795 Change-Id: Id9b1919a19b8ff682738cfb0869a479b4dbb4293
* | | Merge "Remove useless Android.mk files." am: 75d5cf515a am: f27cfa15d7 am: ↵Steven Moreland2017-10-052-38/+0
|\| | | | | | | | | | | | | | | | | | | | | | | 33dc30ce49 am: 0653f3b212 Change-Id: I7c2da3cf30da7b4a1eda7e5369e9caa5ebab2462
| * | Merge "Remove useless Android.mk files." am: 75d5cf515a am: f27cfa15d7Steven Moreland2017-10-052-38/+0
| |\| | | | | | | | | | | | | | | | am: 33dc30ce49 Change-Id: Ia35d20e0fc1df6f7c46b5581537d7ed387965899
| | * Merge "Remove useless Android.mk files."Treehugger Robot2017-10-052-38/+0
| | |\
| | | * Remove useless Android.mk files.Steven Moreland2017-10-042-38/+0
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | These have been c/p'd all over the place. Test: m -j nothing Bug: 33420795 Change-Id: I77979866dbb2345a41a873c84ec3fccd7b127510
* | | | Merge "Update for hidl adapter module defaults." am: 988c977079 am: ↵Steven Moreland2017-10-041-0/+1
|\| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 861651985f am: b53e6ad535 am: 9a17f41f6f Change-Id: I78d9292068b372238554fd9d35128e69f5b5f9d2
| * | | Merge "Update for hidl adapter module defaults." am: 988c977079 am: 861651985fSteven Moreland2017-10-041-0/+1
| |\| | | | | | | | | | | | | | | | | | | | | | am: b53e6ad535 Change-Id: I23269fb7a9bdd352e670a80f390527d9eef31412
| | * | Update for hidl adapter module defaults.Steven Moreland2017-10-041-0/+1
| | |/ | | | | | | | | | | | | Test: pass Change-Id: Idc6a943149a279bf17cfcfd0f2571473e53bbbbf
* | | Merge "Fix typo in update makefiles." am: 9960148420 am: f07e364ce3 am: ↵Steven Moreland2017-09-271-1/+2
|\| | | | | | | | | | | | | | | | | | | | | | | 71a193a425 am: 9fdd241905 Change-Id: I1c852fa6600fecc44d5afd1a65a5f51c43ecf71c
| * | Merge "Fix typo in update makefiles." am: 9960148420 am: f07e364ce3Steven Moreland2017-09-261-1/+2
| |\| | | | | | | | | | | | | | | | am: 71a193a425 Change-Id: I45ae43f07323254212532acbf0f891d11143b05f
| | * Fix typo in update makefiles.Steven Moreland2017-09-261-1/+2
| | | | | | | | | | | | | | | | | | Bug: 37518178 Test: pass Change-Id: Ic401b3a473f15ca4c01e58b3072e19db7c31b653
* | | Update makefiles for hidl adapter. am: 26a0bb2762 am: 73949c1d7e am: b94d0c7290Steven Moreland2017-09-261-0/+78
|\| | | | | | | | | | | | | | | | | am: 97228c0b7d Change-Id: Ic4b459b7c1a4325453945b0b00db8f44e4e77c26
| * | Update makefiles for hidl adapter. am: 26a0bb2762 am: 73949c1d7eSteven Moreland2017-09-261-0/+78
| |\| | | | | | | | | | | | | | | | am: b94d0c7290 Change-Id: If766cffbcc003cc0cc5eb98969b924aa918fff44
| | * Update makefiles for hidl adapter.Steven Moreland2017-09-251-0/+78
| | | | | | | | | | | | | | | | | | Bug: 37518178 Test: manual Change-Id: I50e999907d3c64d2b039272b823971998da64d1b
| | * VTS tests now static link to HAL def libs.Tri Vo2017-08-251-16/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 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
| | * Updating all makefiles.Steven Moreland2017-08-141-1/+4
| | | | | | | | | | | | | | | | | | | | | | | | | | | Bug: 64487114 Test: none Merged-In: I8608c8f636c35f21e4246a805a9eff6d14124e0a Merged-In: I2fa89b6661c39859ec4fb62c4bb0a05a35e645f0 Merged-In: Ifdc3c17cb2b85c18b37dac2d03bb5c8935c23180 Change-Id: I170fa1c4fe39f8109b1670db58ef99bb11afc0be
* | | sound trigger: fix array overflow in HAL wrapperEric Laurent2017-09-071-2/+4
|/ / | | | | | | | | | | Bug: 65383689 Test: make Change-Id: I9d62eb9be096222bcc3e1734a21f6ade0005b180
* | VTS tests now static link to HAL def libs. #2Tri Vo2017-08-101-16/+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
* | Update make file for vndk enabled.Justin Yun2017-08-041-0/+3
| | | | | | | | | | | | | | | | Update the Android.bp generated with hidl-gen. Test: build with and without BOARD_VNDK_VERSION=current Bug: 63866913 Change-Id: I1a9db1df49e0f13c5790da2b118ae9ec63ba34a7
* | Update Android.bp HIDL makefilesTri Vo2017-08-031-1/+1
| | | | | | | | | | | | | | | | | | Allow HAL definition libs to be static. Bug: 32920003 Bug: 64040096 Test: update-all-google-makefiles.sh Change-Id: I1483d572bea6799717d1614fb7d52fe225e31104
* | Add OWNERS for audio-related HAL default implementationsMikhail Naganov2017-07-141-0/+3
|/ | | | | Test: not needed Change-Id: Idefe7be10bde22866be82f87118b9b54e74c3f40
* Update makefiles. (2/2)Steven Moreland2017-05-221-0/+1
|\ | | | | | | | | | | am: 76cfb84140 Change-Id: I24d21438a5b2f447d2160929c6ff6e17b2b66820
| * Update makefiles. (2/2)Steven Moreland2017-05-191-0/+1
| | | | | | | | | | | | | | | | Adds default configuration for all hals. Bug: 38415912 Test: pass Change-Id: Idd1f3a2b7b16ad956d31784a513e93a066cdd02e
* | sound trigger VTS: used default HAL service nameEric Laurent2017-05-171-4/+5
|\| | | | | | | | | | | am: f0b981a42a Change-Id: I354258222b45e4f62cad91a95becd6eca142cf1d
| * sound trigger VTS: used default HAL service nameEric Laurent2017-05-171-4/+5
| | | | | | | | | | | | Bug: 37864311 Test: run VTS for module VtsHalSoundtriggerV2_0Target on marlin Change-Id: If727dcc5921742ece6252dff839ce8100f24ccd1
* | Update makefiles for c++-sources and c++-headers. (2/2)Steven Moreland2017-05-111-2/+2
|\| | | | | | | | | | | am: bc71124120 Change-Id: Icf36068a4d863a8520854c7a83e51f960c188e12
| * Update makefiles for c++-sources and c++-headers. (2/2)Steven Moreland2017-05-091-2/+2
| | | | | | | | | | | | Test: pass Bug: 38174080 Change-Id: Icad451f9f9ad5f46412356a4171a8a1222f8e545
* | Remove redundant dependenciesSteven Moreland2017-05-021-2/+0
|/ | | | | | | | | 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
* soundtrigger: Use "default" instance name for HAL serviceMikhail Naganov2017-04-142-19/+5
| | | | | | | | | | | | | | | | "default" implies using "primary" legacy device module. This doesn't change anything for binderized mode, because the server was only registering itself as "sound_trigger.primary", so it was the only possibility anyways. Usage of "stub" (debugging) module is still possible if audioflinger and corresponding services are built with USE_LEGACY_LOCAL_AUDIO_HAL=true (added makefile check to ensure that). Test: "Ok Google" from screen off state on sailfish Bug: 36570720 Change-Id: I88f1ebd4c1efd0f91e37458746c336a4beef33ae
* Merge changes from topic 'hidl-vendor-available' into oc-devSteven Moreland2017-04-121-0/+1
|\ | | | | | | | | | | * changes: Update makefiles (2/2). Update makefiles.
generated by cgit v1.2.3 (git 2.25.1) at 2024-10-05 10:10:56 +0000