summaryrefslogtreecommitdiffstats
path: root/services
Commit message (Collapse)AuthorAgeFilesLines
* Rename (IF_)LOGI(_IF) to (IF_)ALOGI(_IF) DO NOT MERGESteve Block2012-01-044-9/+9
| | | | | | | See https://android-git.corp.google.com/g/156801 Bug: 5449033 Change-Id: Ib08fe86d23db91ee153e9f91a99a35c42b9208ea
* Rename (IF_)LOGD(_IF) to (IF_)ALOGD(_IF) DO NOT MERGESteve Block2012-01-036-17/+17
| | | | | | | See https://android-git.corp.google.com/g/156016 Bug: 5449033 Change-Id: I4c4e33bb9df3e39e11cd985e193e6fbab4635298
* Merge "Remove dead code"Glenn Kasten2011-12-202-5/+4
|\
| * Remove dead codeGlenn Kasten2011-12-192-6/+4
| | | | | | | | Change-Id: Icf23f7f90fdeb660f4015f22cf239e6d05f5d03c
* | Merge "audio effects: rename configure command"Eric Laurent2011-12-191-1/+1
|\ \
| * | audio effects: rename configure commandEric Laurent2011-12-191-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Renamed audio effect library interface command for audio format configuration from EFFECT_CMD_CONFIGURE to EFFECT_CMD_SET_CONFIG. This makes the naming more consistent with other exixsting commands and allow adding a new command to get the configuration (EFFECT_CMD_GET_CONFIG). Same change for reverse channel configuration renamed from EFFECT_CMD_CONFIGURE_REVERSE to EFFECT_CMD_SET_CONFIG_REVERSE. Implemented EFFECT_CMD_GET_CONFIG in exisitng effect libraries. Change-Id: Ia7b1c620f13797fe5aceb3b0b4acbacce09fb067
* | | Merge "Use constants for 2 and 32"Glenn Kasten2011-12-162-8/+8
|\ \ \
| * | | Use constants for 2 and 32Glenn Kasten2011-12-162-8/+8
| | | | | | | | | | | | | | | | Change-Id: If820dfd58b6df258570750610a07af99598d9e53
* | | | Merge "Extract out audio DSP code to utility library"Glenn Kasten2011-12-164-117/+10
|\ \ \ \ | |/ / / |/| | |
| * | | Extract out audio DSP code to utility libraryGlenn Kasten2011-12-164-117/+10
| | |/ | |/| | | | | | | Change-Id: Ib8ce72028a7ea30e82baa518e381370e820ebbd0
* | | Merge "Use switch in AudioMixer::setParameter"Glenn Kasten2011-12-162-36/+49
|\ \ \
| * | | Use switch in AudioMixer::setParameterGlenn Kasten2011-12-162-36/+49
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Replace series of if/then/elses by easier-to-read switch. Also return void instead of status_t, since callers weren't checking it. Assert on bad input parameters. Change-Id: Ie1f0a297977b28501d20e1af819afed9b4750616
* | | | deleteTrackName now asserts on bad input parameterGlenn Kasten2011-12-161-17/+16
|/ / / | | | | | | | | | | | | | | | | | | This is safe, as the input parameter is always track->name(), which must be valid. Change-Id: Iea8ea3a5706c27026335526ba8851030d00681f8
* | | Merge "setActiveTrack and setBufferProvider can't fail"Glenn Kasten2011-12-162-10/+9
|\ \ \
| * | | setActiveTrack and setBufferProvider can't failGlenn Kasten2011-12-152-10/+9
| |/ / | | | | | | | | | | | | | | | | | | | | | | | | | | | Return void, not status_t, from setActiveTrack and setBufferProvider. These methods returned status_t, but the callers never checked the return value. Since these aren't externally visible APIs, they now return void, and assert on bad input parameters. Change-Id: I530ed29484596ae41e8659826ca425149c51c2a1
* | | Merge "Simplify enable/disable mixing"Glenn Kasten2011-12-163-30/+15
|\ \ \
| * | | Simplify enable/disable mixingGlenn Kasten2011-12-153-30/+15
| |/ / | | | | | | | | | | | | | | | The MIXING enum isn't needed, and now returns void instead of status_t. Change-Id: Ibe4ec24081d75ad4ab78b9c7191fc9077959c4e9
* | | Merge "Improve resistance to leaks for ConfigEvent"Glenn Kasten2011-12-152-8/+7
|\ \ \
| * | | Improve resistance to leaks for ConfigEventGlenn Kasten2011-12-152-8/+7
| |/ / | | | | | | | | | | | | | | | | | | | | | A Vector of pointers is risky, as there is no ownership (and the ThreadBase destructor was not deleting them, so if there were any left over at end it would leak). Replaced by a Vector of values. Change-Id: Iddde72dc30134adfcf724dec26cbe0a742509b8c
* | | Merge "Use NULL not 0 for pointers"Glenn Kasten2011-12-153-29/+29
|\ \ \ | |_|/ |/| |
| * | Use NULL not 0 for pointersGlenn Kasten2011-12-153-29/+29
| |/ | | | | | | Change-Id: Iab3f9abbdab617dc5a599e657ec46a0b0a002eef
* | Merge "Fix indentation and whitespace"Glenn Kasten2011-12-142-54/+53
|\ \
| * | Fix indentation and whitespaceGlenn Kasten2011-12-142-54/+53
| |/ | | | | | | | | | | Use git diff -w to verify. Change-Id: Ib65be0a1ecf65d6cad516110604e3855bf68a638
* | Merge "Improve AudioFlinger error logging"Glenn Kasten2011-12-141-3/+8
|\ \
| * | Improve AudioFlinger error loggingGlenn Kasten2011-12-141-3/+8
| |/ | | | | | | Change-Id: I8ce9aff4038cd7fa0067600faa8080b137db1939
* | Merge "AudioMixer uninitialized simple fields"Glenn Kasten2011-12-141-0/+6
|\ \
| * | AudioMixer uninitialized simple fieldsGlenn Kasten2011-12-141-0/+6
| |/ | | | | | | Change-Id: I47e3849f048d4d990c9634f9c8a2ca2de0339e93
* | Merge "Audio C++ comments"Glenn Kasten2011-12-142-4/+4
|\ \
| * | Audio C++ commentsGlenn Kasten2011-12-142-4/+4
| |/ | | | | | | Change-Id: I84906ebb9dfcfa5b96b287d18364b407f02a30c1
* | Merge "Use const char correctly in AudioFlinger"Glenn Kasten2011-12-141-3/+3
|\ \
| * | Use const char correctly in AudioFlingerGlenn Kasten2011-12-141-3/+3
| |/ | | | | | | | | | | | | Use const char [] instead of const char * to eliminate unnecessary pointer. Make the array audio_interfaces also const, in addition to the strings. Change-Id: I31f33d1dcb9a657ee136f4280fd2d46492496831
* | Merge "Use units after all times"Glenn Kasten2011-12-141-10/+12
|\ \
| * | Use units after all timesGlenn Kasten2011-12-141-10/+12
| |/ | | | | | | Change-Id: I48d3f29c37228b5d03189e4c9600824c9360cac9
* | Merge "Remove redundant clear()"Glenn Kasten2011-12-141-1/+0
|\ \
| * | Remove redundant clear()Glenn Kasten2011-12-141-1/+0
| |/ | | | | | | Change-Id: Ie5e4e63cbc8fa85ef50451dddf8f149fa864b132
* | Fix includesGlenn Kasten2011-12-142-3/+1
|/ | | | | | | | | | | Remove unused #include dlfcn.h Add #include String8.h Forward class declaration is only for references and pointers. In this case, we need the full class declaration, and were incorrectly depending on another header file to do it. Change-Id: Iff65cceb3982698cc1ebaeec405695fc190c03ff
* am 5433e25f: am 7b6aff23: Merge "audioflinger: fix audio skipping over A2DP" ↵Eric Laurent2011-12-052-8/+14
|\ | | | | | | | | | | | | into ics-mr1 * commit '5433e25f6ce013860ff2a074ad8d1158cc39ab91': audioflinger: fix audio skipping over A2DP
| * audioflinger: fix audio skipping over A2DPEric Laurent2011-12-052-8/+14
| | | | | | | | | | | | | | | | | | | | | | | | | | | | The maximum sleep time allowed in the mixer thread when audio tracks are enabled but not ready for mixing is derived from the latency reported by the output stream. This does not work for A2DP where the latency also reflects encoding, decoding and transfer time. Modified activeSleepTimeUs() to take A2DP case into account. Issue 5682206. Change-Id: I3784ac01fb6f836b5a6ce6f764fb15347586de35
* | Merge "Add camera focus move callback."Wu-cheng Li2011-11-291-1/+1
|\ \
| * | Add camera focus move callback.Wu-cheng Li2011-11-291-1/+1
| | | | | | | | | | | | | | | bug:5534973 Change-Id: Iaadcb2738ad040f94849be30ee531326f6199431
* | | am f6422f5f: am 20398fac: Merge "audioflinger: reduce sleep time to avoid ↵Eric Laurent2011-11-231-1/+23
|\ \ \ | |/ / |/| / | |/ | | | | | | underrun" into ics-mr1 * commit 'f6422f5f5d04aab47f8f36a0ea92e2140bed0105': audioflinger: reduce sleep time to avoid underrun
| * audioflinger: reduce sleep time to avoid underrunEric Laurent2011-11-221-1/+23
| | | | | | | | | | | | | | | | | | | | | | | | Progressively reduce the sleep time applied in MixerThread::threadLoop() in case of consecutive application underruns to avoid starving the audio HAL. As the default sleep time is longer than the duration of an audio buffer we ended up writing less data than needed by the audio HAL if the condition persisted. Issue 5553055. Change-Id: I2b23ee79c032efa945025db228beaecd1e07a2e5
* | Bug 5522189 re-enable ARM opts but disable inlineGlenn Kasten2011-11-171-3/+4
| | | | | | | | | | | | | | PIE did not work together with inlining. This change combines (almost all of) the performance benefits of ARM optimizations together with PIE. Change-Id: I4594d33ae5a0a7bac327ae08e30fb35343a06256
* | am 25924f8f: am 030bb998: Merge "audioflinger: fix noise when skipping to ↵Eric Laurent2011-11-142-13/+35
|\ \ | |/ | | | | | | | | | | next song" into ics-mr1 * commit '25924f8f6c0a4ca4a2eb257b72d9625f69d2525e': audioflinger: fix noise when skipping to next song
| * audioflinger: fix noise when skipping to next songEric Laurent2011-11-112-13/+35
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | When audio effects are enabled, a noise can be heard at the beginning of the new song when skipping to next song in music app. This is because some effects (especially virtualizer) have a tail. This tail was not played when previous song was stopped because effects were not processed when no tracks were present on a given session. This is to reduce CPU load when effects are enabled but no audio is playing. The tail was then rendered when the new song was started. Added a delay before stopping effect process after all tracks have been removed from a session. Issue 5584880. Change-Id: I815e0f7441f9302e8dfe413dc269a94e4cc6fd95
* | am db7d79e6: am 2b7f91b9: Merge "Fix regression for SoundPool playback" into ↵Eric Laurent2011-11-101-2/+7
|\ \ | |/ | | | | | | | | | | ics-mr1 * commit 'db7d79e6f1e1860a9bfe4756a03c753435fd0ddf': Fix regression for SoundPool playback
| * Fix regression for SoundPool playbackEric Laurent2011-11-081-2/+7
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Commit 19ddf0eb introduced a problem with applications (like SoundPool) relying on an underrun condition to detect end of playback instead of stopping the track when all data is written. AudioFlinger would keep waiting for new data in case of partial buffer filling and never reach the underrun condition. Added a mechanism to wait no more than once if not enough frames are present in the track buffer. Issue 5585490. Change-Id: I131e605ff6070831a01ddf734e68459e3bf2354b
* | am f3a892ab: Merge "AudioFlinger: mix track only when really ready" into ics-mr1Eric Laurent2011-11-081-1/+10
|\ \ | |/ | | | | | | * commit 'f3a892ab9347ce733b81ccb4913a91c586f8f367': AudioFlinger: mix track only when really ready
| * AudioFlinger: mix track only when really readyEric Laurent2011-11-031-1/+10
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The addition of low power audio playback mode made that audio buffer consumption by audio HAL can now happen in bursts. This makes that requesting audio data from an AudioTrack for mixing can happen at much shorter intervals than before. This revealed an existing problem where AudioFlinger would consider a track ready for mixing although not enough frames were ready to completely fill one output buffer, thus creating short periods of silence. The fix consists in waiting for enough frames to be ready in AudioTrack buffer before declaring a track ready for mixing. This minimum is not applied when the track is stopped to allow the buffer to be emptied completely. Change-Id: I6d04f9b65db5af85b0b53f0a5674be7ec02f9e9f
* | Bug 5522189 temporary workaroundGlenn Kasten2011-10-281-1/+1
| | | | | | | | Change-Id: I21d91d7a24df7bb6e7fc3d0fbc4786d55391fc0e