| Commit message (Collapse) | Author | Age | Files | Lines |
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Added SPDX-license-identifier-Apache-2.0 to:
Android.bp
audio/Android.bp
Bug: 68860345
Bug: 151177513
Bug: 151953481
Test: m all
Exempt-From-Owner-Approval: janitorial work
Change-Id: Id879ff71929846da71ae2df96ab748a4ad464d82
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
This applies to the following types:
- audio_gain_mode_t;
- audio_flags_mask_t;
- audio_channel_representation_t;
- audio_channel_mask_t;
- audio_devices_t.
Enum types are distinct thus proper overloading on the type
is possible in C++. Also, assignments to enum types are
less prone to errors.
Bug: 169889714
Test: basic audio functionality
Change-Id: I6366511b421ccab2782310ecc15a13e08d9c17af
Merged-In: I6366511b421ccab2782310ecc15a13e08d9c17af
|
|
|
|
|
|
| |
Test: build
Change-Id: I0ad1a777a0358a0f79d495747a7ca4bddf984d45
Merged-In: I0ad1a777a0358a0f79d495747a7ca4bddf984d45
|
|
|
|
|
|
|
|
| |
This reverts commit c9eb1ead66550263fb0d4769d8faf561082dfcec.
Reason for revert: Breaks the build of multiple devices
Change-Id: I2a4fe487d60b484c19d8b07d1439af934b852913
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
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>
|
|
|
|
|
|
|
|
| |
* Remove unused local variable or suppress the warning.
Bug: 66996870
Test: build with WITH_TIDY=1
Change-Id: I4d7b4f2e9e7e6c60d8bfc209f47e6f1d33720165
|
|
|
|
|
|
|
|
| |
Test: builds with BOARD_VNDK_VERSION := current
Test: (sanity) Boots and works on internal marlin.
Bug: 33241851
Bug: 29915755
Change-Id: Ic355174a67860afa13377bc9d8f0a140f59ec34e
|
|\
| |
| |
| |
| |
| |
| |
| | |
am: fa192e08eb
am: 20107d1fc9
Change-Id: I4344edee0d02c8df684147bde83ab67bf95c9a99
|
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
The commneted code is the only place in AOSP that still reference
BOARD_HAVE_BLUETOOTH and BOARD_USES_GENERIC_AUDIO. Delete it
to make things cleaner.
Test: Build
Change-Id: I37e3135bf2db2c6d46eb59d5a0de49d02c1a036d
Signed-off-by: Bin Chen <bin.chen@linaro.org>
|
|/
|
|
|
|
| |
Test: make.
Change-Id: I8b04ba9e1ab3b3fc22cf1b1baba010664c47ecc6
|
|\
| |
| |
| |
| |
| | |
am: a7355c3ab4
Change-Id: I0f4c805e27c1e51d15f63d7e6b9c0521fd9bd520
|
| |
| |
| |
| |
| |
| | |
Bug: 28341362
Change-Id: I9b370ece9506591abfff46d5edb726a7fc993298
Test: build with clang-tidy
|
|\|
| |
| |
| |
| |
| |
| |
| |
| | |
am: 1e8f444
* commit '1e8f444a0fdf1134bab3f1cf8f06cbbea96da11f':
Use libmedia header paths in legacy audio libs
Change-Id: Idfbf33afeb8f2689ee6374bc8b675c039fc31a83
|
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
The headers for these libraries include headers from libmedia,
which means they need access to the full libmedia include path.
Bug: 27804373
Test: These libraries compile when libmedia's compilation path changes
slightly.
Change-Id: I201c82c85a1513b4165c7565dfac4f4371d6fe5c
|
| |
| |
| |
| |
| | |
Bug: 27814144
Change-Id: I54b7a73ec318779fd3addb060998f4d94141152d
|
|/
|
|
|
| |
Bug: 27562099
Change-Id: Id5af47af3e8e3404c575fbb0ec02cf25378b7089
|
|
|
|
|
|
|
|
|
| |
Fix some (note the fun infinite recursion bug from the dawn of time,
apparently no one has called that function), silence those that are
style choices.
Bug: http://b/26936282
Change-Id: I99fa342abf9d134fb729ed8d906e60c011120e89
|
|
|
|
|
| |
Bug: 18903857
Change-Id: I2a5fdb7f42ba46729e66ff08893183184edc3c97
|
|
|
|
|
|
|
|
|
|
|
| |
Remove a systemTime() function from power/power.c that
doesn't appear to be called anywhere, and make several
audio/ files explicitly include utils/Timers.h for the
libutils version of systemTime(). (They appear to have been
transitively getting it via utils/threads.h's include of
utils/Condition.h.)
Change-Id: I432bb1cf1100f5829df307a146076acf035a0bdc
|
|\
| |
| |
| |
| | |
* commit 'e714af927c1e87a488178315b78b0c12c0ec9db9':
audio policy: use legacy stream type definitions
|
| |
| |
| |
| |
| |
| |
| |
| | |
New stream types have been added, so make sure that
legacy implementation uses legacy stream definitions
Bug: 18067208
Change-Id: I8042ebe9ac3e7eda380b8c91650f2ac16b44bc34
|
|\|
| |
| |
| |
| |
| |
| | |
into lmp-mr1-dev
* commit 'b4b0608b0560a7cad27aeec0af8b1728b971aa3a':
legacy audio policy: check string parameters before use
|
| |
| |
| |
| |
| | |
Bug: 18262893
Change-Id: Iee6e59db7ffae5612256ba1f74aba62bc8cf12db
|
|\|
| |
| |
| |
| | |
* commit 'a311a983eb4035b272033105119cb44d814fb2db':
audio HAL: add parameters to open stream functions
|
| |
| |
| |
| |
| |
| |
| |
| | |
Pass device address (and audio source for inputs) to
open_output_stream() and open_input_stream() audio HAL functions.
Bug: 14815883.
Change-Id: Ib1c3e98518733fa5eab54cbca4220e388d3f6211
|
|\ \
| |/
|/|
| |
| | |
* commit '39b3abc0ca227d72f66bb47a75cec2acfb4d6de0':
audio: add mechanism to re-route nullified audio
|
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
Fixes a race condition whereby two requests attempt to change the audio stream
(and interrupt each other). For example, a user space app makes a call through
the audio API and before completion, a kernel function such as jack removal is
detected and is serviced. In this case, you have a nullified audio route that
needs to be re-routed.
In order to know a routing update is needed, a boolean member mForceRouting
has been introduced to enforce routing for the next attempt to set audio output
with a non-null device.
Change-Id: Ia0aa2f27fe63c39f0980dec2e1e79dbc26b81e56
Author: Vincent Becker <vincentx.becker@intel.com>
Signed-off-by: Vincent Becker <vincentx.becker@intel.com>
Signed-off-by: Zhi Jin <zhi.jin@intel.com>
Signed-off-by: Shuo Gao <shuo.gao@intel.com>
Signed-off-by: Bruce Beare <bruce.j.beare@intel.com>
Signed-off-by: Jack Ren <jack.ren@intel.com>
Signed-off-by: Robert Beatty <robert.m.beatty@intel.com>
|
| |
| |
| |
| | |
Change-Id: I17db01e92bfa939c7a15e7f9f95197d0a0568033
|
| |
| |
| |
| |
| |
| |
| | |
Do not use popcount() to derive channel count from channel mask.
Bug: 15000850.
Change-Id: I5050ce7fd7884369eb0de37967e40052630006a9
|
| |
| |
| |
| |
| |
| |
| | |
for USB Audio Output
Bug: 13747844
Change-Id: I53b7db8102fd2ed24014c338ea5e7290cda8b2f3
|
| |
| |
| |
| |
| | |
Bug: 9428304.
Change-Id: Ie3c9827ed78703a1b2d9699e38ce628f3055138b
|
|\ \ |
|
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
AudioService now indicates in and out device connection
separately in case of combined devices like wired headset
or BT SCO headset.
Change-Id: Id20e02bb351cc732ceddccf597742ced049671a8
|
|/ /
| |
| |
| | |
Change-Id: I500d1f2f8f2199c674b289cf671b7f863a7d02ee
|
|\|
| |
| |
| |
| |
| |
| | |
openOutputStreamWithFlags API"
* commit '62038fc4c5c596f04a87388bb42bdef13d359c39':
Audio: add an openOutputStreamWithFlags API
|
| |\ |
|
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
The existing openOutputStream API did not allow passing stream flags even
though its audio hardware interface's open_output_stream counterpart does take
a "flags" argument. This means that, when the audio hw_module is implemented
using libhardware_legacy, the output flag was lost.
This commit adds an API in libhardware_legacy's audio module, allowing to pass
the flag, and updates the default wrapper in order to call this new API instead
of the old version that missed the flag.
Change-Id: I20c102d32b489f87fabbd7d02aa0b28fae13948b
Signed-off-by: David Wagner <david.wagner@intel.com>
Signed-off-by: Bruce Beare <bruce.j.beare@intel.com>
Signed-off-by: Jack Ren <jack.ren@intel.com>
|
| | |
| | |
| | |
| | | |
Change-Id: Ib2e531f115f8bd1d5f290094032f3f4a4753e726
|
|\| |
| | |
| | |
| | | |
Change-Id: Ie85a3c7d498d7685738190267916e9a205cc469e
|
| |/
| |
| |
| |
| |
| |
| |
| | |
Fix a few 64-bit warnings and set -Wno-unused-parameter, which makes
the directory warnings-clean on 32-bit and 64-bit.
Remove LOCAL_32_BIT_ONLY.
Change-Id: I688b31683f1a6598419866d9b7b6bc27a7056f4a
|
| |
| |
| |
| | |
Change-Id: I4e2517233487ca7487ed7948a974fc2cc6c41190
|
|\|
| |
| |
| |
| | |
* commit 'fa8ad9fb544e01e663709e8dccd9b358e4d3ede9':
Audio policy is 32-bit only
|
| |
| |
| |
| | |
Change-Id: I2eca0286d98bb5eebe9c68286d4ee7ed366f173d
|
|\|
| |
| |
| |
| |
| |
| | |
AUDIO_DEVICE_IN_USB_DEVICE to sDeviceNameToEnumTable"
* commit 'e6c7127ef2491ddeb995ffb48843fda4e30ff9ff':
audio policy : add AUDIO_DEVICE_IN_USB_DEVICE to sDeviceNameToEnumTable
|
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
This commit adds AUDIO_DEVICE_IN_USB_DEVICE to sDeviceNameToEnumTable[],
It had to be added here, when it was defined in system/core/include/system/audio.h
AudioPolicyManagerBase::loadHwModule() fails to parse device type of
AUDIO_DEVICE_IN_USB_DEVICE in audio_policy.conf.
By applying this commit, AUDIO_DEVICE_IN_USB_DEVICE can be used for
audio policy customization.
Change-Id: I0165956334d7105129dca4c746372dbca0d9e324
|
| |
| |
| |
| |
| |
| | |
In particular, we need AUDIO_FORMAT_PCM_24_BIT_PACKED.
Change-Id: I28abe8466024bdd4d1edbb4281c6472d45a2d94f
|
| |
| |
| |
| |
| |
| | |
And simplify by removing the unused I/O handle parameter 'output'.
Change-Id: Ie9c4df17a7378066312d4ed8790fda7a9125c95e
|
| |
| |
| |
| |
| |
| | |
Remove NUL characters from audio_policy's dump output
Change-Id: If591473dbe3e5adbc34dbbdb0d552b329f19eb6a
|
|\|
| |
| |
| |
| |
| |
| | |
LOCAL_MODULE_PATH to LOCAL_MODULE_RELATIVE_PATH"
* commit '3582b9b0e1caa4a43b8706e699903142e34ad5c0':
hardware/libhardware_legacy: convert LOCAL_MODULE_PATH to LOCAL_MODULE_RELATIVE_PATH
|
| |
| |
| |
| |
| |
| |
| |
| |
| | |
LOCAL_MODULE_RELATIVE_PATH
LOCAL_MODULE_PATH doesn't work for multiarch builds, replace it
with LOCAL_MODULE_RELATIVE_PATH.
Change-Id: I4e4ceec61d026bbe74ba604554c06104bde42e5e
|