| Commit message (Collapse) | Author | Age | Files | Lines |
|
|
|
|
|
|
|
|
|
| |
Add checks on parameter size field in effect command handlers
to avoid overflow leading to invalid comparison with min allowed
size for command and reply buffers.
Bug: 26347509.
Change-Id: I20e6a9b6de8e5172b957caa1ac9410b9752efa4d
(cherry picked from commit ad1bd92a49d78df6bc6e75bee68c517c1326f3cf)
|
|\ |
|
| |
| |
| |
| |
| |
| |
| |
| |
| | |
Use boolean property ro.audio.ignore_effects to load or ignore the
audio effects.
Bug 21906334
Change-Id: I778f8b10a1caf25f7679705b1c83775223df011d
|
|/
|
|
|
|
|
|
|
|
|
| |
Check consistency of effect command reply sizes before
copying to reply address.
Also add null pointer check on reply size.
Also remove unused parameter warning.
Bug: 21953516.
Change-Id: I4cf00c12eaed696af28f3b7613f7e36f47a160c4
|
|
|
|
|
|
|
|
|
| |
Fixed automatic level compensation for bundled effects. Fixed algorithm
now works even in corner cases previously reported.
Cross band coefficients and cross interaction of BassBoost fixed.
bug: 18769691
Change-Id: I51570836bdbff37e7c4b26f2b4282e4ded7fa56d
|
|\
| |
| |
| |
| |
| |
| | |
compensation" into lmp-mr1-dev
* commit 'd11d1be13a6391290fb1dbc572292973dae59909':
Fix for Bundled Effects level compensation
|
| |\
| | |
| | |
| | |
| | |
| | |
| | | |
into lmp-mr1-dev
* commit '7ebc167072be27f42b7c58bd7f9378c8311533ab':
Fix for Bundled Effects level compensation
|
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
From the bundled effects, only the EQ had some sort of level compensation
based on the settings of the effect. This patch takes into account all
the status and levels of EQ, BassBoost and Virtualizer bundled effects
and makes an informed decision on the overall level compensation.
bug: 18213779
Change-Id: I23c7b94fe7da5482f0bbef0cbe715ea464631420
|
|\ \ \
| |/ /
| | |
| | |
| | |
| | |
| | | |
TARGET_SIMULATOR checks"
* commit 'e10615e7f8749de887c8d25579398ad9e4094077':
Remove obsolete TARGET_SIMULATOR checks
|
| |\ \
| | | |
| | | |
| | | |
| | | | |
* commit '6b3e1a6d59b304a2dde7f03ba184feac0dec4ee7':
Remove obsolete TARGET_SIMULATOR checks
|
| | | |
| | | |
| | | |
| | | | |
Change-Id: I9d499190e49fed16aac686ae5da79ab870605ff4
|
|\ \ \ \
| |/ / /
| | | |
| | | |
| | | |
| | | |
| | | | |
virtualization mode" into lmp-mr1-dev
* commit '59ca7038364cd6fa3e7e1b52e0e90fa0960af038':
Fix for AOSP force virtualization mode
|
| |\ \ \
| | |/ /
| |/| /
| | |/
| | |
| | |
| | | |
into lmp-mr1-dev
* commit '4df79c2c13f6163f7d99360a7fb34bf3bd2a5115':
Fix for AOSP force virtualization mode
|
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
Forcing virtualization mode with AOSP code might erroneously return
a failed code, whilst it was correct. Fixing the condition for
Automatic mode.
bug: 18538537
Change-Id: I5837ff40cfed04c99889813cd56ebe3d90793e74
|
|\ \ \
| |/ /
| | |
| | |
| | | |
* commit '575174d62b665900203d70a4d410329b8918c2af':
Move libldnhncr to libc++.
|
| | |
| | |
| | |
| | |
| | | |
Bug: 15193147
Change-Id: If59c4a53a497f501881d26d58b80b11371906527
|
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
uuids need to be unique, and things don't work properly when they're
not.
Also fix/enhance/extend the dumpEffectDescriptor() method, and
include a list of effects in audioflinger dumpsys.
Change-Id: I3dfbc5ed0f7272c7809e337f2929212ece047ee4
|
|\ \ \
| |/ /
| | |
| | |
| | | |
* commit 'fed1594d707b4623015e538b2f69faf76bc38d5b':
Clean up some makefiles.
|
| |\ \
| | |/
| |/|
| | |
| | | |
* commit '9916dcb6ef041752fda5f13192ee6a36e51e73a5':
Clean up some makefiles.
|
| | |
| | |
| | |
| | |
| | |
| | | |
The build system takes care of these things for you.
Change-Id: Ife9605f15786e11d6a3578f56ed83dd9ae7fd799
|
| | |
| | |
| | |
| | |
| | | |
Change-Id: I88106ba25f9b3f8fd7616307ab6b7fbe7afb6e91
(cherry picked from commit bfd3b9764841971be2d2f04a3fdbd41a2144beea)
|
| | |
| | |
| | |
| | | |
Change-Id: I19252ea8d7dd8b9eee2532059c5a3a98d53c3b28
|
|/ /
| |
| |
| | |
Change-Id: I88106ba25f9b3f8fd7616307ab6b7fbe7afb6e91
|
| |
| |
| |
| |
| |
| |
| | |
No unused parameter warning in EffectBundle.
Remove unused variable declaration in Virtualizer_getParameter
Change-Id: Ibe1c051bc21ac53a6c770b4aa28e31ac5d559d9b
|
| |
| |
| |
| | |
Change-Id: I210129f5742b046f7ceef48194f039352eff596d
|
| |
| |
| |
| |
| | |
Change-Id: Ia658ab4b6320d19fdb50f123c930918724ff0ef3
Signed-off-by: Andy Hung <hunga@google.com>
|
|\ \
| |/
| |
| | |
Change-Id: Ic4e75541193f733f477cfa4a2a87cce4c9e64f45
|
| |
| |
| |
| |
| |
| | |
and use same suffixes
Change-Id: I90b74ede171ba2550db4a220cfd1ad2e3caefe2d
|
| |
| |
| |
| |
| |
| | |
and corresponding downmix effect
Change-Id: I1576aa373ca7acbb84d7742bacbd9c2da04a7a4c
|
|\ \
| |/
| |
| |
| | |
* commit 'e1ff1051ffee8fb650741ad133f0f28b73eb7a73':
libeffects: 64 bit compile issues
|
| |
| |
| |
| | |
Change-Id: I32dd13d307572fee91150f8e113df78924ca0067
|
|\ \
| |/
| |
| |
| | |
* commit '3a46b975070b503bc71b53b70f9a8f5bd03bba1c':
LP64 fixes for media/libeffects
|
| |\ |
|
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
Changes include:
[x] In get parameter series of functions, replaced size_t*
formal parameter type with uint32_t* where actual parameter
passed was uint32_t*.
[x] In set parameter series of functions, changed size_t
formal parameter to uint32_t where actual parameter was
uint32_t.
[x] Changed the definition of LVM_UINT32 from unsigned
long to uint32_t as unsigned long is 64-bit in LP64.
[x] Used other stdint.h types for other LVM_types for
consistency.
[x] Use of uintptr_t for the pNextMember of the INST_ALLOC
structure, rather than LVM_UINT32, for portablility.
[x] Use of uintptr_t where pointers are used in arithmetic.
[x] Replaced the use of 0xFFFFFFFC with ~3 in places where
it was used to clear last two bits.
[x] Removed int casts where cmdSize and *replySize, both
uint32_t, were being compared with sizeof().
Change-Id: Ibec0b4d8e9b855f44b1cd853be6df84d13cf4186
Signed-off-by: Marcus Oakland <marcus.oakland@arm.com>
Signed-off-by: Ashok Bhat <ashok.bhat@arm.com>
|
|\ \ \
| |/ /
| | |
| | |
| | |
| | |
| | | |
capture command"
* commit 'e276aef8779a7763b318ebd3e8c09f35ff37a95c':
Fix preventing from heap overwrite in capture command
|
| |\ \ |
|
| | |/
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
There was heap usage error in case of stop of audio framework while
capturePoint was negative. Pointer to reply data was moved but
final silence write was done using original buffer size.
Now silence set is done at the beginning under the condition
that framework has stopped.
Change-Id: I7dab1e922f1813e5fbfd4a64c8b0d15d9465520c
|
|/ /
| |
| |
| |
| | |
bug:13225538
Change-Id: I4fccc414923f7e62cd46d691c67cb44b9692c225
|
|\ \
| |/
|/| |
|
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
&pDwmModule->config may equal to pConfig that triggers
a building error using gcc 4.7:
In file included from bionic/libc/include/signal.h:33:0,
from bionic/libc/include/sys/select.h:34,
from bionic/libc/include/unistd.h:34,
from system/core/include/cutils/log.h:34,
from frameworks/av/media/libeffects/downmix/EffectDownmix.c:19:
In function 'memcpy',
inlined from 'Downmix_Configure' at frameworks/av/media/libeffects/downmix/EffectDownmix.c:651:11,
inlined from 'Downmix_Init' at frameworks/av/media/libeffects/downmix/EffectDownmix.c:605:9,
inlined from 'DownmixLib_Create' at frameworks/av/media/libeffects/downmix/EffectDownmix.c:233:9:
bionic/libc/include/string.h:114:31: error: call to '__memcpy_overlap_error' declared with attribute error: memcpy called with overlapping regions
Change-Id: Ia6c4aab72e38d00d8d526f3b56fc03b6530b4e44
|
| |
| |
| |
| |
| |
| |
| | |
We were reading some uninitialized memory when creating bass boost and EQ
effects, and using memcpy() with identical source and destination.
Change-Id: I15ea1b2c52ae05cbf54aef04351e89805e0ebf8e
|
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
There was a problem with previous fix in 1b6c677c where
the THD was good for max volume but bad for intermediate
high volume values.
This fix does not limit the band gains but apply a correction
to the effect bundle volume to take into account not only the
maximum band gain but also the average band gains weighted
by the proximity of each band to the max gain band.
Change-Id: Ibedd8da7fce1e163117f29b3da332da0e4fe8889
|
| |\ |
|
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
Limit maximum gain in all EQ bands according to current volume so that
total gain (current volume + band gain) does not exceed a certain limit.
The gain difference between bands is preserved.
Change-Id: Ice5a9705a0b3353e8778b4c539a29ca9cdf60390
|
| |/
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
The audio preprocessings are not included in the default
audio_effect.conf file anymore as the audio HAL has to provide
support for them.
Each product implementing one or more audio pre processing must
provide a specific audio_effects.conf file.
Change-Id: I3b2f636f7a8748829798d4b9057d269db9424d62
|
| |\ |
|
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
Fix a bug resetting the headroom parameters when
calling the bundle process function for the first time.
Change-Id: Ie05f97606f415954340ff2a4e48cd7b0a97a063b
|
| |/
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
The ThreadBase class now has a separate member for input
and output devices (mInDevice, mOutDevice).
Only query get_supported_devices() from audio HAL if the function
is exposed and if the audio policy manager did not specify the
audio module to open.
Also fixed bug in AEC preprocessing that would reset
to default output device when an input device was given.
Change-Id: I19d4d06aeb920b068e3ef31e6e6be6345ce5d67a
|
| |
| |
| |
| |
| |
| |
| | |
Fix a bug in LVM bundle wrapper preventing audio framework sample rate
from being sent to the effect bundle instance.
Change-Id: I912027f866d3beac91b977bba869ff96a92d2a62
|
| |
| |
| |
| |
| |
| |
| |
| | |
Setting the module path breaks multilib builds where there may
be two different module paths for 32-bit and 64-bit, and there
is no need to set it to the default.
Change-Id: I95d62b5573a0bb0ab6c97649041dfc4309cbc8b9
|