Commit graph

2912 commits

Author SHA1 Message Date
Eric Laurent
711437527c Merge "audio: Align bluetooth start behavior" into 24D1-dev am: ef4519eeaa
Original change: https://googleplex-android-review.googlesource.com/c/platform/hardware/interfaces/+/26946076

Change-Id: Ie3c889130f9328f4e4fc4d8f137d5e2c01d61320
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
2024-04-16 14:18:52 +00:00
Eric Laurent
ef4519eeaa Merge "audio: Align bluetooth start behavior" into 24D1-dev 2024-04-16 13:47:01 +00:00
Shunkai Yao
8c47facb19 DynamicsProcessing VTS: remove stageInUse testing am: f137ba4d92
Original change: https://googleplex-android-review.googlesource.com/c/platform/hardware/interfaces/+/26924921

Change-Id: I7372c7686b7480ad799165c234ce5dad9598dc26
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
2024-04-16 08:08:52 +00:00
Mikhail Naganov
3f4c7fca29 audio: Align bluetooth start behavior
When staring data transfer, if resuming of BT session fails,
the legacy implementation does not indicate an error, thus
causing an automatic retry on next transfer. The AIDL
implementation was switching the stream into an error state
instead. This could cause audio mute when the BT stack is
slow on resuming. This CL aligns AIDL with legacy behavior.

Bug: 333307789
Test: repro steps in the bug
Change-Id: I5080a7275671287f188930f041c5ff2215ed2e1e
2024-04-12 12:37:39 -07:00
Treehugger Robot
429baea512 Merge "PresetReverb: Add test to validate PresetReverb Effect" into main am: 0f141d1967 am: bdb87cf0ce
Original change: https://android-review.googlesource.com/c/platform/hardware/interfaces/+/3029145

Change-Id: I253596a95ee4a6dbac2a37299d73b797d82ce70f
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
2024-04-11 17:52:14 +00:00
Treehugger Robot
0f141d1967 Merge "PresetReverb: Add test to validate PresetReverb Effect" into main 2024-04-11 17:25:34 +00:00
Shunkai Yao
f137ba4d92 DynamicsProcessing VTS: remove stageInUse testing
As we relaxed the requirement of stageInUse to setParam

Bug: 328012516
Test: atest VtsHalDynamicsProcessingTargetTest
Change-Id: I62e7d07c54133a0e3aed1952f80010faba783a20
2024-04-11 17:16:34 +00:00
Mikhail Naganov
1676108800 audio: Use allow list for device connection types in TryConnectMissingDevice am: 7b9b9e03e5 am: 30cf023827
Original change: https://android-review.googlesource.com/c/platform/hardware/interfaces/+/3038652

Change-Id: I293c710b7bab3f5ab5d09022abe2f821210eba23
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
2024-04-11 07:53:03 +00:00
Sneha Patil
94abce0998 PresetReverb: Add test to validate PresetReverb Effect
Added tests to verify increase in energy with decrease in room and hall size.
Remove preset reverb default implementation from the config file.

Bug: 305866207
Test: atest VtsHalPresetReverbTargetTest
Change-Id: I565612d90b7c7e52cb23bd5dbfaf02235f65cc67
2024-04-11 09:25:56 +05:30
Mikhail Naganov
7b9b9e03e5 audio: Use allow list for device connection types in TryConnectMissingDevice
Limit the connection types to test to the following:
 - HDMI*
 - IP_V4
 - USB

Only these connection types can be easily checked by the HAL
for presence of an external device.

Bug: 326888643
Test: atest VtsHalAudioCoreTargetTest
Change-Id: I659e14a150b3043ead8d844cd89a2c4700d57efd
2024-04-10 23:50:37 +00:00
Kiyoung Kim
160c3d610c Merge "Remove VNDK definition(s)" into main 2024-04-09 02:13:37 +00:00
Kiyoung Kim
9e9d41d162 Remove VNDK definition(s)
As of VNDK deprecation, any libraries that defines vndk is no longer
valid anymore. This change removes all VNDK definition(s) from modules
which was VNDK.

Bug: 328994089
Test: AOSP CF build succeeded
Change-Id: Ifdade9c052db29c989ce7d804ba725361c4c0f8e
Merged-In: Ic4f19113c1a25dc1fb60cdfbf9c0f2d8d7d09726
2024-04-05 02:29:07 +00:00
Shunkai Yao
2475cd33e9 Merge "Effect AIDL VTS: relax dynamics processing effect parameter validations" into 24D1-dev am: ad6464d43b
Original change: https://googleplex-android-review.googlesource.com/c/platform/hardware/interfaces/+/26767676

Change-Id: Ia9115becb833d3021f499708cf431e484d87bf0e
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
2024-04-04 18:44:54 +00:00
Shunkai Yao
ad6464d43b Merge "Effect AIDL VTS: relax dynamics processing effect parameter validations" into 24D1-dev 2024-04-04 18:18:58 +00:00
Shunkai Yao
1a84397f13 Effect VTS: skip VolumeDataTest for U am: 40d9d9ee7b
Original change: https://googleplex-android-review.googlesource.com/c/platform/hardware/interfaces/+/26767393

Change-Id: I7dc684435f10e10a50ba7a4dd17a821b551bc80f
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
2024-04-02 20:46:28 +00:00
Shunkai Yao
76ea415c28 Merge changes I561d6864,I7e909cda into 24D1-dev
* changes:
  Use customized toString for UUID in VTS
  Effect VTS: skip VolumeDataTest for U
2024-04-02 20:30:03 +00:00
Shunkai Yao
653c996424 Merge "Use customized toString for UUID in VTS" into main am: e237f5bfaa am: ffac5b2200
Original change: https://android-review.googlesource.com/c/platform/hardware/interfaces/+/3020095

Change-Id: Ibbc6d195102bfb025e50241dd44ac36b08c53ba6
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
2024-04-02 17:34:30 +00:00
Shunkai Yao
6f8224538e Use customized toString for UUID in VTS
Bug: 331955595
Test: atest --test-mapping hardware/interfaces/audio/aidl/vts:presubmit
Change-Id: I561d68645f3f159b029135b0dfa66511a2800d72
Merged-In: I561d68645f3f159b029135b0dfa66511a2800d72
2024-04-02 17:03:38 +00:00
Shunkai Yao
e237f5bfaa Merge "Use customized toString for UUID in VTS" into main 2024-04-02 16:45:52 +00:00
Shunkai Yao
40d9d9ee7b Effect VTS: skip VolumeDataTest for U
Volume effect implementation was not ready on early U releases

Bug: 327497904
Test: atest --test-mapping hardware/interfaces/audio/aidl/vts:presubmit
Change-Id: I7e909cdabbd167bd2e012dbe378ccb3402d33f53
2024-03-30 00:13:23 +00:00
Shunkai Yao
53238b1afd Effect AIDL VTS: relax dynamics processing effect parameter validations
relaxing several parameter checking to align with HIDL
 - no need to have stage in use to set bands/channels
 - band settings no need to be sorted by frequency

Bug: 328012516
Test: atest VtsHalDynamicsProcessingTargetTest
Change-Id: If12d978ee69ee7f087a7e8758513a9c6bacf817f
2024-03-29 23:17:05 +00:00
Shunkai Yao
fb8f39452a Use customized toString for UUID in VTS
Bug: 331955595
Test: atest --test-mapping hardware/interfaces/audio/aidl/vts:presubmit
Change-Id: I561d68645f3f159b029135b0dfa66511a2800d72
2024-03-29 18:49:15 +00:00
Mikhail Naganov
efb21cb338 audio: Fix some VTS issues on real devices am: 69d60aa02c am: 677c969ad7
Original change: https://android-review.googlesource.com/c/platform/hardware/interfaces/+/3018422

Change-Id: Ia84d34b5233648d0b3540f083c9b5b506cd57241
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
2024-03-29 02:51:52 +00:00
Mikhail Naganov
69d60aa02c audio: Fix some VTS issues on real devices
1. Skip testing of stream I/O on certain types of mix ports.

2. Skip testing of connection of BT SCO device.

Bug: 300735639
Bug: 326888356
Bug: 328010709
Bug: 331516432
Test: atest VtsHalAudioCoreTargetTest
Change-Id: I9b8bbf2014e223375c8f8400ff2af32268803706
2024-03-28 17:02:27 -07:00
Mikhail Naganov
2677d603bc Merge "audio: Allow to inherit Binder RT priority in AIDL" into 24D1-dev am: 9a9a5d2a88
Original change: https://googleplex-android-review.googlesource.com/c/platform/hardware/interfaces/+/26723206

Change-Id: Icdff933310168a01f1e2d6b56df3c7bec1fa1394
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
2024-03-27 16:31:16 +00:00
Mikhail Naganov
efb45bc792 audio: Allow to inherit Binder RT priority in AIDL
This option is on by default for HIDL interfaces, however
in AIDL it needs to be enabled explicitly.

Bug: 331469541
Test: m
(cherry picked from https://googleplex-android-review.googlesource.com/q/commit:d975a9e6ec0b6f0eeac99673b9b06e11146874ba)
Merged-In: I971185a46da4222160fb04b8b77f805ef0396562
Change-Id: I971185a46da4222160fb04b8b77f805ef0396562
2024-03-27 16:20:33 +00:00
Mikhail Naganov
d975a9e6ec audio: Allow to inherit Binder RT priority in AIDL
This option is on by default for HIDL interfaces, however
in AIDL it needs to be enabled explicitly.

Bug: 331469541
Test: m
Change-Id: I971185a46da4222160fb04b8b77f805ef0396562
2024-03-26 17:22:47 -07:00
Mikhail Naganov
41e1e9e50c audio: Implement blocking in remote submix when there is no sink am: 6507376fb9
Original change: https://googleplex-android-review.googlesource.com/c/platform/hardware/interfaces/+/26684914

Change-Id: I6a4a1b6c0eab6808db990e3bc341e9945812a1b9
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
2024-03-25 16:14:25 +00:00
Mikhail Naganov
a41ff5134d audio: Implement blocking in remote submix when there is no sink
The HIDL implementation blocked for the audio buffer duration
when the output side of the remote submix pipe does not have
the sink connected. This behavior was accidentally removed
when fixing b/327220024.

Also, limit the amount of the debug messages displayed when
the sink is shutdown to avoid spamming the syslog.

Bug: 328347445
Test: repro steps from the bug
Test: atest VtsHalAudioCoreTargetTest
Test: atest --test-filter=".*AudioPlaybackCaptureTest.*" CtsMediaAudioTestCases
(cherry picked from https://googleplex-android-review.googlesource.com/q/commit:6507376fb940198e8c39ad6d5345d132b1dd7219)
Merged-In: Ia66cb7b4567d42a41bf4715b7d725d36510ac50c
Change-Id: Ia66cb7b4567d42a41bf4715b7d725d36510ac50c
2024-03-25 16:04:27 +00:00
Mikhail Naganov
6507376fb9 audio: Implement blocking in remote submix when there is no sink
The HIDL implementation blocked for the audio buffer duration
when the output side of the remote submix pipe does not have
the sink connected. This behavior was accidentally removed
when fixing b/327220024.

Also, limit the amount of the debug messages displayed when
the sink is shutdown to avoid spamming the syslog.

Bug: 328347445
Test: repro steps from the bug
Test: atest VtsHalAudioCoreTargetTest
Test: atest --test-filter=".*AudioPlaybackCaptureTest.*" CtsMediaAudioTestCases
Change-Id: Ia66cb7b4567d42a41bf4715b7d725d36510ac50c
2024-03-22 13:05:48 -07:00
Mikhail Naganov
7f50205333 Merge "audio: Adjust stream buffer size calculation" into main am: 222d034461 am: a5855a1592
Original change: https://android-review.googlesource.com/c/platform/hardware/interfaces/+/3008508

Change-Id: I8484832349e3af69cec9ca2a56f315f225b83427
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
2024-03-22 04:08:11 +00:00
Mikhail Naganov
222d034461 Merge "audio: Adjust stream buffer size calculation" into main 2024-03-22 03:36:44 +00:00
Mikhail Naganov
cbb2976223 audio: Adjust stream buffer size calculation
When calculating buffer size, use powers of two
for "large" buffers only on SRs >= 44100 Hz.
With lower SRs, sizes rounded up to a multiple
of 2 give too high latency.

Bug: 328432688
Test: repro steps from b/328432688#comment3
Test: atest VtsHalAudioCoreTargetTest
Change-Id: I605ef8507fef030d349e85c33ed0229c9566ddb5
2024-03-21 16:40:30 -07:00
Mikhail Naganov
df65834372 audio: Do not use A2DP/LE status when populating device port
The events about the status of A2DP and LE are sent by
the framework asynchronously with attempts to connect
an external device. As a result, the connection attempt
may be rejected by the bluetooth HAL module.

Bug: 327711955
Test: atest VtsHalAudioCoreTargetTest
(cherry picked from https://googleplex-android-review.googlesource.com/q/commit:a46d52771a3e4a7816ea722088153ee6b65f59c3)
Merged-In: Iecef0ee6a66cd290237fdcccd6a7016e9622c4b0
Change-Id: Iecef0ee6a66cd290237fdcccd6a7016e9622c4b0
2024-03-20 16:48:08 +00:00
Mikhail Naganov
419b007f42 audio: Do not use A2DP/LE status when populating device port am: a46d52771a
Original change: https://googleplex-android-review.googlesource.com/c/platform/hardware/interfaces/+/26636280

Change-Id: If0f4c8a811cc17760fbecf3e9f72cbec0e5ee4b4
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
2024-03-20 09:17:07 +00:00
Mikhail Naganov
a46d52771a audio: Do not use A2DP/LE status when populating device port
The events about the status of A2DP and LE are sent by
the framework asynchronously with attempts to connect
an external device. As a result, the connection attempt
may be rejected by the bluetooth HAL module.

Bug: 327711955
Test: atest VtsHalAudioCoreTargetTest
Change-Id: Iecef0ee6a66cd290237fdcccd6a7016e9622c4b0
2024-03-19 17:42:48 -07:00
Shunkai Yao
bb35effe0d Audio effect aidl log reduce and level adjust
Bug: 329334917
Test: atest AudioEffectTest
Test: atest CtsMediaAudioTestCases
Test: atest CtsEffectTestCases
Test: audio use cases on pixel AIDL audio hal
Change-Id: I49876924b230f2ac834d3b0e494a03b814c4bb6e
Merged-In: I49876924b230f2ac834d3b0e494a03b814c4bb6e
2024-03-16 00:00:12 +00:00
Shunkai Yao
5226f35235 Merge "Audio effect aidl log reduce and level adjust" into 24D1-dev am: 3035c98d27
Original change: https://googleplex-android-review.googlesource.com/c/platform/hardware/interfaces/+/26588428

Change-Id: Ia577cf04652149b4a69f636744945e7ed769ef8e
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
2024-03-15 19:16:45 +00:00
Shunkai Yao
3035c98d27 Merge "Audio effect aidl log reduce and level adjust" into 24D1-dev 2024-03-15 18:57:05 +00:00
Kiyoung Kim
b3b01a6cc2 Merge "Remove VNDK definition(s)" into main 2024-03-15 01:37:03 +00:00
Shunkai Yao
dd43b56f56 Audio effect aidl log reduce and level adjust
Bug: 329334917
Test: atest AudioEffectTest
Test: atest CtsMediaAudioTestCases
Test: atest CtsEffectTestCases
Test: audio use cases on pixel AIDL audio hal
Change-Id: I49876924b230f2ac834d3b0e494a03b814c4bb6e
2024-03-14 22:14:11 +00:00
Shunkai Yao
691b1c0ee2 Merge changes I95f19d20,I57d924d1 into 24D1-dev am: 2ff1f0a2eb
Original change: https://googleplex-android-review.googlesource.com/c/platform/hardware/interfaces/+/26571715

Change-Id: Ia256b0a9e0a58bb60d8fb9f9937a6107bdfce423
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
2024-03-14 17:33:13 +00:00
Shunkai Yao
0915ffcb62 Merge "Effect AIDL VTS: skip data path testing for offloading effects" into 24D1-dev am: b2869538a7
Original change: https://googleplex-android-review.googlesource.com/c/platform/hardware/interfaces/+/26576426

Change-Id: I9bb7d90adf2c56ab754bc983b3e8e3bdf5135310
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
2024-03-14 17:32:12 +00:00
Shunkai Yao
e0c1f31ff6 minor improvements for VtsHalAudioEffectFactoryTargetTest
Bug: 327678656
Test: atest VtsHalAudioEffectFactoryTargetTest
Change-Id: I95f19d20883142cd500c552966bd8f02256a1be5
Merged-In: I95f19d20883142cd500c552966bd8f02256a1be5
2024-03-14 17:32:07 +00:00
Shunkai Yao
2ff1f0a2eb Merge changes I95f19d20,I57d924d1 into 24D1-dev
* changes:
  minor improvements for VtsHalAudioEffectFactoryTargetTest
  Effect Aidl: make EffectFactoryHelper methods static only
2024-03-14 17:31:43 +00:00
Shunkai Yao
50e478ba73 Effect AIDL VTS: skip data path testing for offloading effects
Bug: 328330990
Test: atest VtsHalBassBoostTargetTest
Test: atest VtsHalDownmixTargetTest
Test: atest VtsHalLoudnessEnhancerTargetTest
Test: atest VtsHalVolumeTargetTest
Change-Id: Ic720e74cf25f5282dfa52dca354a5eccf9071f61
Merged-In: Ic720e74cf25f5282dfa52dca354a5eccf9071f61
2024-03-14 17:10:37 +00:00
Shunkai Yao
b2869538a7 Merge "Effect AIDL VTS: skip data path testing for offloading effects" into 24D1-dev 2024-03-14 16:58:44 +00:00
Shunkai Yao
f0cb5ec61f Effect AIDL VTS: skip data path testing for offloading effects
Bug: 328330990
Test: atest VtsHalBassBoostTargetTest
Test: atest VtsHalDownmixTargetTest
Test: atest VtsHalLoudnessEnhancerTargetTest
Test: atest VtsHalVolumeTargetTest
Change-Id: Ic720e74cf25f5282dfa52dca354a5eccf9071f61
2024-03-14 02:04:11 +00:00
Shunkai Yao
c43c4558a0 minor improvements for VtsHalAudioEffectFactoryTargetTest
Bug: 327678656
Test: atest VtsHalAudioEffectFactoryTargetTest
Change-Id: I95f19d20883142cd500c552966bd8f02256a1be5
2024-03-13 22:21:57 +00:00
Shunkai Yao
9f79875faa Effect Aidl: make EffectFactoryHelper methods static only
Bug: 327678656
Test: atest --test-mapping hardware/interfaces/audio/aidl/vts:presubmit
Change-Id: I57d924d19f38257e32dba95d4de0ca00532c5ab4
Merged-In: I57d924d19f38257e32dba95d4de0ca00532c5ab4
2024-03-13 22:11:33 +00:00