OSDN Git Service
Chien-Yu Chen [Mon, 9 Jan 2017 23:55:06 +0000 (15:55 -0800)]
camera: Add enableZsl key
Add enableZsl key so the apps can choose to allow or disallow out of
order frame contents and metadata like timestamp.
Test: Updated camera CTS tests pass
Bug:
30983997
Change-Id: I7d0fd82e551bb50420c438472e5240f3fee65d73
Eric Laurent [Thu, 19 Jan 2017 01:07:08 +0000 (17:07 -0800)]
audio: add missing definition for stub HAL module name
Bug:
34360879
Test: make and check stub HAL is loaded in HIDL passthrough mode
Change-Id: Ibb90a7f77baf910b6c68c1a5adf1c39c8a8d9906
Mark Salyzyn [Wed, 11 Jan 2017 18:59:20 +0000 (18:59 +0000)]
Merge "liblog: use log/log.h when utilizing ALOG macros" am:
1fb716a12b am:
b239ca4c99 am:
f70588249e
am:
c464b07573
Change-Id: I003bd76cf3dd0662c59b9e59b59fae3eb919d950
Mark Salyzyn [Wed, 11 Jan 2017 18:53:20 +0000 (18:53 +0000)]
Merge "liblog: use log/log.h when utilizing ALOG macros" am:
1fb716a12b am:
b239ca4c99
am:
f70588249e
Change-Id: If008012618637a2a53a64c1ee580318bee6b63d6
Mark Salyzyn [Wed, 11 Jan 2017 18:48:19 +0000 (18:48 +0000)]
Merge "liblog: use log/log.h when utilizing ALOG macros" am:
1fb716a12b
am:
b239ca4c99
Change-Id: I72bacdf8bca3f057ef26721520042907e62d8d4c
Mark Salyzyn [Wed, 11 Jan 2017 18:42:18 +0000 (18:42 +0000)]
Merge "liblog: use log/log.h when utilizing ALOG macros"
am:
1fb716a12b
Change-Id: I2281b2971bc4557f566da755884e7c7465e4c9bb
Treehugger Robot [Wed, 11 Jan 2017 18:36:13 +0000 (18:36 +0000)]
Merge "liblog: use log/log.h when utilizing ALOG macros"
Mark Salyzyn [Tue, 10 Jan 2017 22:29:16 +0000 (14:29 -0800)]
liblog: use log/log.h when utilizing ALOG macros
Test: compile
Bug:
30465923
Change-Id: I4329b7dc918b732b07662a27c52e3b431ffabd3d
Jean-Michel Trivi [Thu, 29 Dec 2016 20:22:06 +0000 (12:22 -0800)]
Define AUDIO_USAGE_ASSISTANT in list of audio attributes usage
This change was autogenerated from an update in
hardware/interfaces/audio/common/2.0/types.hal
with script
hardware/interfaces/update-base-files.sh
Test: make
Bug:
30947943
Change-Id: I703b30e9a55299a0db84613361af537aa7d7c599
Mark Salyzyn [Tue, 27 Dec 2016 22:07:26 +0000 (22:07 +0000)]
Merge "media: Replace cutils/log.h with android/log.h or log/log.h" am:
c8a6e3ea23 am:
6f84b4a716 am:
bbe900ad12
am:
5af7309214
Change-Id: I0c2a7a0727acf4a97506ee801c4f081145a88ff1
Mark Salyzyn [Tue, 27 Dec 2016 22:00:26 +0000 (22:00 +0000)]
Merge "media: Replace cutils/log.h with android/log.h or log/log.h" am:
c8a6e3ea23 am:
6f84b4a716
am:
bbe900ad12
Change-Id: I4ee9513b768984ab477c3585bd5e78032208c092
Mark Salyzyn [Tue, 27 Dec 2016 21:52:55 +0000 (21:52 +0000)]
Merge "media: Replace cutils/log.h with android/log.h or log/log.h" am:
c8a6e3ea23
am:
6f84b4a716
Change-Id: I3b93f676e39094a3ca5e2161e968185662f6dcf2
Mark Salyzyn [Tue, 27 Dec 2016 21:45:24 +0000 (21:45 +0000)]
Merge "media: Replace cutils/log.h with android/log.h or log/log.h"
am:
c8a6e3ea23
Change-Id: Iddd2d6d623ba7e6b605937def3bc9b0ac36b4d38
Treehugger Robot [Tue, 27 Dec 2016 21:34:30 +0000 (21:34 +0000)]
Merge "media: Replace cutils/log.h with android/log.h or log/log.h"
Mark Salyzyn [Wed, 28 Sep 2016 22:40:31 +0000 (15:40 -0700)]
media: Replace cutils/log.h with android/log.h or log/log.h
- replace cutils/log.h with android/log.h (main buffer logging)
- replace cutils/log.h with log.log.h (+SafetyNet logging)
Test: compile
Bug:
26552300
Bug:
32189077
Change-Id: Icda4d5ab7555c0b801fd362fdcfa65b1500612b4
Dan Willemsen [Tue, 20 Dec 2016 21:10:51 +0000 (21:10 +0000)]
Merge "Add system/media/Android.bp" am:
0eb9eac06d am:
138805b4e5 am:
f6e848ea0e
am:
5fe9d20ae7
Change-Id: Ie6830955695d2931cdd83ea2f6b18c608aab79e3
Dan Willemsen [Tue, 20 Dec 2016 21:00:50 +0000 (21:00 +0000)]
Merge "Add system/media/Android.bp" am:
0eb9eac06d am:
138805b4e5
am:
f6e848ea0e
Change-Id: I5472ee8eb49501a18e895643944ccb51b93373bc
Dan Willemsen [Tue, 20 Dec 2016 20:42:11 +0000 (20:42 +0000)]
Merge "Add system/media/Android.bp" am:
0eb9eac06d
am:
138805b4e5
Change-Id: Ib17b17aee989e69cc9c8d212704c94f84b6e4393
Dan Willemsen [Tue, 20 Dec 2016 20:33:27 +0000 (20:33 +0000)]
Merge "Add system/media/Android.bp"
am:
0eb9eac06d
Change-Id: Ic355591646ebc2815107f871dc9e26680cca7704
Treehugger Robot [Tue, 20 Dec 2016 20:25:08 +0000 (20:25 +0000)]
Merge "Add system/media/Android.bp"
Dan Willemsen [Tue, 20 Dec 2016 18:52:30 +0000 (10:52 -0800)]
Add system/media/Android.bp
Test: Soong's build.ninja identical before/after (except GlobRules)
Change-Id: Ib294d1afcfcc311748f436232d27ed381dc99fb6
Glenn Kasten [Fri, 16 Dec 2016 20:42:26 +0000 (12:42 -0800)]
fifo: non-throttling reader doesn't see data already written
Test: see the multi-reader interactive test
Change-Id: I0663a8e90efc094fef09401c6e2ecd3391100efa
Eric Laurent [Fri, 9 Dec 2016 22:32:30 +0000 (14:32 -0800)]
audio: add definitions for MMAP stream mode
Bug:
33398120
Test: make marlin-eng
Change-Id: I7892f1e0920df05aa4f8dbf459a02a8b6e27958d
Glenn Kasten [Wed, 14 Dec 2016 16:37:05 +0000 (16:37 +0000)]
Merge "fifo: add index references with caching and deferred operations"
Glenn Kasten [Wed, 30 Nov 2016 19:05:32 +0000 (11:05 -0800)]
fifo: add index references with caching and deferred operations
The new classes will eventually permit a set of related atomic operations,
wakes, and waits to be grouped together into a single sequence point,
and will allow use of event flags to wait for the "or" of multiple FIFOs.
This is one of a series of CLs to isolate the dependencies.
Test: not ready yet for testing
Change-Id: I2055b7a35721222cd914973cd210f5c0dca7d4ef
TreeHugger Robot [Tue, 13 Dec 2016 19:31:02 +0000 (19:31 +0000)]
Merge "Ran hardware/interfaces/update-base-files.sh."
Jean-Michel Trivi [Tue, 13 Dec 2016 16:19:06 +0000 (16:19 +0000)]
Merge "Unique ID for audio players"
Steven Moreland [Mon, 12 Dec 2016 17:51:39 +0000 (09:51 -0800)]
Ran hardware/interfaces/update-base-files.sh.
Test: pass
Change-Id: Iafd710d21c3bd7d843894d945fc86b055a04dda6
Mikhail Naganov [Wed, 7 Dec 2016 23:40:13 +0000 (23:40 +0000)]
Merge "Audio HAL: extensions for new SoC features"
Glenn Kasten [Wed, 7 Dec 2016 18:30:33 +0000 (10:30 -0800)]
Fix build by removing symlink
Bug:
33379123
Test: builds OK
Change-Id: I4ff1662d0ffc66653b0139302cb25e9f3d2730ef
Jean-Michel Trivi [Thu, 1 Dec 2016 16:33:56 +0000 (08:33 -0800)]
Unique ID for audio players
Add new type of unique ID to track high level players
such as MediaPlayer or SoundPool.
Test: make
Bug:
30955183
Change-Id: I509a33cbf44260dd3ef48b956f41a2145b0ca9ef
Mikhail Naganov [Tue, 22 Nov 2016 02:38:12 +0000 (18:38 -0800)]
Audio HAL: extensions for new SoC features
Extended audio offload structure: added support to add bit_width,
offload buffer size and usage info.
Add support for direct PCM audio stream. Direct PCM stream
is used when pcm data required post processing in DSP. Direct PCM
stream is used for both track offload and PCM offload.
Add audio proxy source device.
Add support for additional audio formats: EVRC, QCELP, WMA, WMA,
AAC_ADIF, AMR, MP2, FLAC, ALAC, APE, DSD, LDAC.
Extend audio channel mask.
Test: make
Change-Id: If01c351a810015b1ff97fc9f2e62c49eba8ea069
Glenn Kasten [Sat, 3 Dec 2016 01:37:47 +0000 (01:37 +0000)]
Merge "fifo: index is now a class instead of a struct"
Glenn Kasten [Sat, 3 Dec 2016 00:26:33 +0000 (16:26 -0800)]
fifo: Fix build
Test: builds OK on macOS
Change-Id: Icac55988f07fb4e4960edd218a33963b6908ce33
Glenn Kasten [Fri, 2 Dec 2016 23:04:39 +0000 (15:04 -0800)]
fifo: index is now a class instead of a struct
And confirm that it is POD with static_assert.
Test: builds OK
Change-Id: I9a703c38e9c67e406a1ac6a699a1ffd851efcb8a
Glenn Kasten [Fri, 2 Dec 2016 00:12:59 +0000 (16:12 -0800)]
fifo: isolate atomic and futex operations
Now all atomic and futex operations occur in fifo_index.cpp.
This implements these TODOs:
- Abstract out atomic operations to audio_utils_fifo_index
- Replace friend by setter and getter, and abstract the futex
This is one of a series of CLs to isolate the dependencies.
Test: builds OK on Android, host Linux, and host macOS
Change-Id: Ic4ec46d472c583dce8525f11ed8cb1db89928d30
Glenn Kasten [Thu, 1 Dec 2016 16:30:21 +0000 (08:30 -0800)]
fifo: pull out clock_nanosleep and futex to separate files
This is one of a series of CLs to isolate the dependencies.
Test: builds OK on Android, host Linux, and host macOS
Change-Id: I8ffcab7b7c4acab141b09f75f593ebde1ebec31a
Pavlin Radoslavov [Thu, 1 Dec 2016 03:28:17 +0000 (03:28 +0000)]
Merge "Fix a typo in config rate check: 441000 -> 44100" am:
84e3d1d1e2 am:
7f7c9626de am:
819eafd923
am:
2bf4141a10
Change-Id: I7fd229ef487c726ff7f96b0377a7ebec3768af49
Pavlin Radoslavov [Thu, 1 Dec 2016 03:22:46 +0000 (03:22 +0000)]
Merge "Fix a typo in config rate check: 441000 -> 44100" am:
84e3d1d1e2 am:
7f7c9626de
am:
819eafd923
Change-Id: I4a0c07f6cc8ec510747cf499a3b305bb323b1d44
Pavlin Radoslavov [Thu, 1 Dec 2016 03:17:45 +0000 (03:17 +0000)]
Merge "Fix a typo in config rate check: 441000 -> 44100" am:
84e3d1d1e2
am:
7f7c9626de
Change-Id: I76380d4d6685f359c947dff027069414402780df
Pavlin Radoslavov [Thu, 1 Dec 2016 03:12:45 +0000 (03:12 +0000)]
Merge "Fix a typo in config rate check: 441000 -> 44100"
am:
84e3d1d1e2
Change-Id: I3e570ab57af3f836f4224770301631b06e2204ba
Treehugger Robot [Thu, 1 Dec 2016 03:02:33 +0000 (03:02 +0000)]
Merge "Fix a typo in config rate check: 441000 -> 44100"
Glenn Kasten [Wed, 30 Nov 2016 23:39:19 +0000 (23:39 +0000)]
Merge "fifo: add shutdown state"
rago [Wed, 30 Nov 2016 21:02:17 +0000 (13:02 -0800)]
Fix security vulnerability: potential OOB write in audioserver
Bug:
32705438
Bug:
32703959
Test: cts security test
(audio_effect.h moved to a different branch)
Change-Id: I36a2c19373e0a2c7bf90599c0ccbb9e0ad815f65
Pavlin Radoslavov [Wed, 30 Nov 2016 18:45:32 +0000 (10:45 -0800)]
Fix a typo in config rate check: 441000 -> 44100
Test: code compilation
Change-Id: I073e1a46723417259c265720168d90938fd38dd0
Glenn Kasten [Tue, 22 Nov 2016 22:00:23 +0000 (14:00 -0800)]
fifo: add shutdown state
FIFO is marked shutdown after an "impossible" error is detected.
Usually this indicate that memory has become corrupted, or incorrect use
of the API. Once the FIFO is marked shutdown, all APIs that can return
an error code will return -EIO, and all other APIs will become no-ops.
Test: see unit tests in tests/ directory
Change-Id: I559145d613c3783a699a9ced062dd5ff3676769d
Jean-Michel Trivi [Mon, 28 Nov 2016 22:24:36 +0000 (22:24 +0000)]
Merge "Count a11y stream as public stream"
Jean-Michel Trivi [Wed, 23 Nov 2016 23:45:48 +0000 (15:45 -0800)]
Count a11y stream as public stream
Test: see bug
Bug:
30448020
Change-Id: Iad817f47d11e8766fea5d1c083823e2b132b606d
Glenn Kasten [Tue, 22 Nov 2016 22:02:11 +0000 (14:02 -0800)]
fifo: update documentation
Test: builds OK
Change-Id: I5a3d0bdadc8613e8501b020457665135677f7b63
Glenn Kasten [Tue, 22 Nov 2016 22:00:53 +0000 (14:00 -0800)]
fifo: use const more
Test: builds OK
Change-Id: I81c4e656b65e9e50f3affb11aeb3b51b0a36e3f7
Glenn Kasten [Mon, 21 Nov 2016 20:13:03 +0000 (20:13 +0000)]
Merge "Rename hysteresis-related symbols for clarity"
Glenn Kasten [Mon, 21 Nov 2016 20:12:52 +0000 (20:12 +0000)]
Merge "Fix a few clang-tidy warnings"
Glenn Kasten [Fri, 18 Nov 2016 23:28:42 +0000 (15:28 -0800)]
Fix a few clang-tidy warnings
Test: still builds
Change-Id: I6583f4f600a24a0784cdda186f77d5cacffc6cc5
Glenn Kasten [Fri, 18 Nov 2016 21:34:01 +0000 (13:34 -0800)]
Rename hysteresis-related symbols for clarity
Test: builds OK
Change-Id: I47fc5c5b7cbe6354386e143a5f9a5b5f9f3322a6
Glenn Kasten [Wed, 16 Nov 2016 20:53:33 +0000 (20:53 +0000)]
Merge "fifo: add new APIs, and other cleanup"
TreeHugger Robot [Mon, 14 Nov 2016 22:34:54 +0000 (22:34 +0000)]
Merge "audio_route: do not check for null pointer when freeing"
David Lin [Sat, 12 Nov 2016 01:43:16 +0000 (17:43 -0800)]
audio_route: do not check for null pointer when freeing
Remove null pointer checks when doing free()
Test: make
Change-Id: I29c3641aa66b2a56676fa1cc062055bbf16d67d3
Signed-off-by: David Lin <dtwlin@google.com>
Glenn Kasten [Wed, 19 Oct 2016 19:14:46 +0000 (12:14 -0700)]
fifo: add new APIs, and other cleanup
New APIs:
- buffer()
- capacity()
- flush()
- frameSize()
- totalFlushed()
- totalLost()
- totalReleased()
Pull up mFifo from writer and reader into the base provider class.
Update documentation.
Test: see unit tests in tests/ directory
Change-Id: I1957638c69e1c1dc742b6f024156def757736867
Eric Laurent [Fri, 11 Nov 2016 01:41:23 +0000 (17:41 -0800)]
radio metadata: add API to get channel and subchannel
Change-Id: I5a5a3a58b8ec667f9a1d911210efd897fc7713cb
Andy Hung [Thu, 10 Nov 2016 19:15:45 +0000 (19:15 +0000)]
Merge "Add AUDIO_UID_INVALID"
Mikhail Naganov [Tue, 8 Nov 2016 22:11:36 +0000 (14:11 -0800)]
Move AUDIO_DEVICE_*_STUB to system/audio.h.
This is needed so it can be used in TypeConverter.cpp
Bug:
32740394
Change-Id: Ie203df160b808818813c169733641b6ac68b96f5
Test: make & check on the device
Andy Hung [Tue, 8 Nov 2016 01:38:17 +0000 (17:38 -0800)]
Add AUDIO_UID_INVALID
Test: power manager should show correct uids for audio
Bug:
32713790
Change-Id: Ib6993e2f06b3fbf5a96942088fa1bae30c5f781f
Marco Nelissen [Mon, 7 Nov 2016 18:00:33 +0000 (18:00 +0000)]
Merge "IAudioPolicyService is now in libaudioclient instead of libmedia" am:
76f97250ff am:
c58a77360c am:
ff3b15b22e
am:
7a73cf02af
Change-Id: Ib5c3c7d3a527e47051d795be4a26e30f525c989a
Marco Nelissen [Mon, 7 Nov 2016 17:41:49 +0000 (17:41 +0000)]
Merge "IAudioPolicyService is now in libaudioclient instead of libmedia" am:
76f97250ff am:
c58a77360c
am:
ff3b15b22e
Change-Id: I8903ffc0ae1b6422d0e10a12817694f072c9a8c9
Marco Nelissen [Mon, 7 Nov 2016 17:32:14 +0000 (17:32 +0000)]
Merge "IAudioPolicyService is now in libaudioclient instead of libmedia" am:
76f97250ff
am:
c58a77360c
Change-Id: Id3f3ec80c10a4d698e6f38660cfc911337b91e69
Marco Nelissen [Mon, 7 Nov 2016 17:22:52 +0000 (17:22 +0000)]
Merge "IAudioPolicyService is now in libaudioclient instead of libmedia"
am:
76f97250ff
Change-Id: Ie85b93bdb1b0b004c4cfe9cd2f60cc8766781ced
Marco Nelissen [Mon, 7 Nov 2016 17:14:24 +0000 (17:14 +0000)]
Merge "IAudioPolicyService is now in libaudioclient instead of libmedia"
Marco Nelissen [Fri, 4 Nov 2016 16:40:24 +0000 (09:40 -0700)]
IAudioPolicyService is now in libaudioclient instead of libmedia
Test: build
Change-Id: I865f3c4bc3a4823802944501f2b3f86711059356
Marco Nelissen [Fri, 4 Nov 2016 21:53:53 +0000 (21:53 +0000)]
Merge "IAudioPolicyService is now in libaudioclient instead of libmedia"
Marco Nelissen [Fri, 4 Nov 2016 16:40:24 +0000 (09:40 -0700)]
IAudioPolicyService is now in libaudioclient instead of libmedia
Test: build
Change-Id: I865f3c4bc3a4823802944501f2b3f86711059356
Dan Willemsen [Thu, 3 Nov 2016 20:55:28 +0000 (13:55 -0700)]
Fix build with MacOS 10.12 SDK
Test: mmma -j system/media/audio_utils
Change-Id: Iab8270c5f2dc011a984acd78f59a3adca73b7a62
Thierry Strudel [Thu, 27 Oct 2016 17:12:43 +0000 (10:12 -0700)]
audio_route: check if subpath name is valid before trying to add it
Change-Id: I38d38c14d68e0fb25af188661cba750b7cac8f99
Eric Laurent [Thu, 27 Oct 2016 15:23:04 +0000 (15:23 +0000)]
Merge "radio: fix metadata compatibility for 64 bit processes"
Eric Laurent [Tue, 25 Oct 2016 18:01:45 +0000 (11:01 -0700)]
radio: fix metadata compatibility for 64 bit processes
Clarify use of 32 bit quantities in radio metadata APIs and internal
implementation: all "unsigned int" are replaced by uint32_t.
Also use size_t when appropriate.
Add checks and asserts on invalid metadata buffer size.
Change-Id: I1e4a995858467c679b978574c305d82814f8c15c
Glenn Kasten [Wed, 19 Oct 2016 22:46:02 +0000 (15:46 -0700)]
fifo: fix bugs in lost frames calculation
Problems fixed:
- did not set the lost frames parameter to zero when no frames were lost
- under-estimated lost frames by mFrameCount
Also improved the documentation about lost frames.
Test: currently using interactive test only, need an automated version
Change-Id: I8b056c9cf4ead0d197b4c3aeff8482a08710c4cb
Glenn Kasten [Thu, 20 Oct 2016 18:21:52 +0000 (18:21 +0000)]
Merge "fifo: address code review comments"
Glenn Kasten [Wed, 19 Oct 2016 18:13:55 +0000 (11:13 -0700)]
fifo: address code review comments
Test: builds OK
Change-Id: I20f1a2e0cf0d63e4a5c04bf7f3364d0e71d8e929
Mikhail Naganov [Tue, 18 Oct 2016 21:46:05 +0000 (14:46 -0700)]
Move audio hw module names to system/
The names of the modules are also used in audio policy configuration
files and thus need to be consistent between HAL and the framework.
Change-Id: I55510bfe425f9fd8634530ed6db25514da081007
Test: make
Mikhail Naganov [Tue, 18 Oct 2016 16:10:05 +0000 (16:10 +0000)]
Merge "Move audio_effects_conf.h to system/audio_effects"
Mikhail Naganov [Tue, 18 Oct 2016 16:01:15 +0000 (16:01 +0000)]
Merge "Move parameters that are used by framework to system/audio.h"
Glenn Kasten [Fri, 14 Oct 2016 15:51:20 +0000 (08:51 -0700)]
Handle futex status EWOULDBLOCK
Test: very difficult, as it is a race condition that rarely occurs
Change-Id: Ibefe09d1e9d940f57463391ed339325b42b84b73
Glenn Kasten [Tue, 18 Oct 2016 01:08:29 +0000 (01:08 +0000)]
Merge "Address code review comments"
Mikhail Naganov [Tue, 18 Oct 2016 01:03:56 +0000 (18:03 -0700)]
Move audio_effects_conf.h to system/audio_effects
This file is not really related to HAL.
Also removed executable bit from it, thanks to gkasten@
Change-Id: I4f4fb2bb6c3792b06dfad2ecdd5e07428acbc351
Test: make
Glenn Kasten [Tue, 18 Oct 2016 00:51:26 +0000 (00:51 +0000)]
Merge "Change struct timeout * to const struct timeout *"
Mikhail Naganov [Tue, 18 Oct 2016 00:14:23 +0000 (17:14 -0700)]
Move parameters that are used by framework to system/audio.h
Note that it is preferred to use the constants from AudioParameter
instead of directly using preprocessor defines.
Change-Id: I072448d06ec6dbbc760519004576bc4fe1438062
Test: make
Mikhail Naganov [Mon, 17 Oct 2016 22:22:49 +0000 (22:22 +0000)]
Merge "Split audio effects headers"
Glenn Kasten [Sun, 16 Oct 2016 21:51:15 +0000 (14:51 -0700)]
Change struct timeout * to const struct timeout *
Test: builds OK
Change-Id: I8372ab83a50d527dffbfbb5014ac93ee4692f577
Mikhail Naganov [Fri, 14 Oct 2016 22:33:43 +0000 (15:33 -0700)]
Split audio effects headers
There can be different use cases with the code that imports
headers from audio_effects (each subsequent case is a more
specialized version of the previous one):
1. The code only needs access to general structures
like effect_descriptor_t:
==> include system/audio_effect.h
Example: AudioSystem in libmedia
2. The code that needs access to effect UUID and parameter
enums:
==> include system/audio_effects/effect_foo.h
Examples: media framework effects interfaces, libwilhelm
3. The code that needs to call into HAL:
==> include hardware/audio_effect.h
Examples: libaudiohal (implementation part) and libeffects
4. The code that implements a particular effect:
==> include audio_effects/effect_foo.h
Examples: cts/.../CTSTestEffect.cpp
stuff in hardware/, vendor/
Change-Id: Iee601a7a4d485ea35ff5c16ed595b219911585f7
Test: make
Glenn Kasten [Fri, 14 Oct 2016 15:49:02 +0000 (08:49 -0700)]
Address code review comments
Change INT_MAX to INT32_MAX.
Document iovec == NULL.
Strengthen TODO about re-architecting futex code.
Rename getSize() to size().
Simplify error handling for mFifo.diff().
Rename masked to offset.
Test: re-run tests/*
Change-Id: I51f0c457399ac12b282eee726d13c3ca9a58ed1a
Glenn Kasten [Fri, 14 Oct 2016 17:05:26 +0000 (10:05 -0700)]
Fix build
Test: build for sdk_phone_mips-sdk (mac) works
Change-Id: Ifc839975d1fc994beeb1651280e2d94e8b7a3199
Glenn Kasten [Fri, 14 Oct 2016 16:31:25 +0000 (16:31 +0000)]
Merge "fifo: improve blocking and hysteresis"
Mikhail Naganov [Thu, 13 Oct 2016 19:56:00 +0000 (12:56 -0700)]
Remove dependencies on hardware/audio_effects.h
Also remove some outstanding dependencies on hardware/audio.h.
Most of the time, only constants and structs from audio_effects.h
are used, and no actual calls into HAL are made. Move these
definitions into system/audio_effect.h, and generate constants
from TREBLE HAL definition.
Test: make
Change-Id: Ib28eca67f40bf8285d3352407f1ee82b6c77ad54
Eric Laurent [Thu, 13 Oct 2016 00:35:00 +0000 (00:35 +0000)]
audio_route: add check on enum values am:
ff3e6f401e am:
e12085b0c5
am:
95f7f2c505
Change-Id: I09f7d7f0547b2970231fe2e12babcc1f3041cc8c
Eric Laurent [Thu, 13 Oct 2016 00:27:11 +0000 (00:27 +0000)]
audio_route: add check on enum values am:
ff3e6f401e
am:
e12085b0c5
Change-Id: Ib2155fab97702c791c58ad2f27f5f5d6e0f790ba
Eric Laurent [Thu, 13 Oct 2016 00:19:40 +0000 (00:19 +0000)]
audio_route: add check on enum values
am:
ff3e6f401e
Change-Id: I4a9cfe96aac7472dde381cbb8756353d3cd128f2
Glenn Kasten [Sun, 2 Oct 2016 20:00:19 +0000 (13:00 -0700)]
fifo: improve blocking and hysteresis
Work-in-progress to eventually allow the application to independently
specify whether or not each index (both front and rear) is also treated
as a futex, and if so whether it is single-process or multi-process.
This involves replacing mIsPrivate by an enum for futex types.
Still to do:
- provide an API to configure this
- better testing
Continued work on the setHysteresis() API.
Still to do:
- better testing and debugging, especially for the read side.
- do the right thing based on data currently in the buffer
Added available() API to return the number of frames that could
be obtain()ed. For reader, this is the fill level.
Miscellaneous:
- renamed sharedRear to writerRear
- improved Doxygen comments
- address code review comments by explaining why some paramters are
passed by reference and some as pointers
- updated tests
- fixed bug: iovec was not being clear on error
- changed default value of throttlesWriter to true
Test: tests/fifo_*
Change-Id: I0e19052abc90453ca48a0299d2c2de62468de6cb
Eric Laurent [Wed, 12 Oct 2016 17:38:12 +0000 (10:38 -0700)]
audio_route: add check on enum values
Bug:
31906753
Bug:
32064030
Change-Id: Id57ad136b2c314040e6cbf62a3319ecef2c2114b
Mikhail Naganov [Tue, 11 Oct 2016 00:20:51 +0000 (17:20 -0700)]
Remove redundant prefixes from enums in audio common types and effects HAL
Re-generated audio-base.h from updated files. No changes
in effective constant values, only in source mapping comments.
Bug:
31800672
Change-Id: I28ed14a0f5cfa53ada105258cd196fa86b401857
Test: make
Eric Laurent [Fri, 7 Oct 2016 01:36:52 +0000 (18:36 -0700)]
audio: add constants to audio-base.h
move some constants to audio-base.h to avoid double
definitions with audio::common::2_0::types.h
Change-Id: I6ef18a2fbb173ae183cf2cd07c9758b2efd57752
Colin Cross [Fri, 7 Oct 2016 00:10:37 +0000 (00:10 +0000)]
Merge "Removes gnu designated initializers." am:
594952f2a2 am:
e72d1d1206 am:
9e36579d55
am:
cb5d4c6038
Change-Id: Ibc603eaa5413040eb5bf090c21f88132836ba624