summaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeFilesLines
* Fix potential NULL dereference in Visualizer effectstable/cm-13.0-caf-8994-ZNH2Krago2016-12-091-3/+7
| | | | | | | | | | Bug: 30229821 CYNGNOS-3312 Test: fixing CL. Existing unit tests still pass. Change-Id: I73d3fe413dd7590470c5b3b19569d6a7969c4dd8 (cherry picked from commit 0c39abc483ba980f7c417e57a0aa39139136c97a)
* Fix potential overflow in Visualizer effectrago2016-11-101-0/+8
| | | | | | | | CYNGNOS-3303 Bug: 30229821 Change-Id: Iea1c4a21735e893aeded95b980044ec0861a7ea8 (cherry picked from commit 2fa52194ef64843c2908c69527384c6c2fcdbafa) (cherry picked from commit 57ac66340ad488a17fc285b6fc2635cb7375d72b)
* DO NOT MERGE Fix AudioEffect reply overflowAndy Hung2016-07-072-3/+6
| | | | | | | Bug: 28173666 Ticket: CYNGNOS-3020 Change-Id: I055af37a721b20c5da0f1ec4b02f630dcd5aee02 (cherry picked from commit 57fd9637536d40ec8c40a6bed76a71471dab0f64)
* Merge tag 'LA.BF64.1.2.2-03340-8x94.0' into HEADstable/cm-13.0-caf-8994-ZNH2KBMatt Wagantall2016-02-262-3/+29
|\ | | | | | | | | | | | | "LA.BF64.1.2.2-03340-8x94.0" Issue-Id: SAMBAR-1290 Change-Id: I3a8cb0e2785a2a98a86871cfc5fe7d575ea86305
| * hal: Fix potential memory leaks in HALAlexy Joseph2016-02-102-3/+29
| | | | | | | | | | | | Fix potential memory leaks in HAL Change-Id: Ifd491cc185a7d1afa647f03639e57d1685d18803
* | Merge tag 'LA.BF64.1.2.2-03240-8x94.0' into HEADMatt Wagantall2016-02-162-9/+25
|\| | | | | | | | | | | | | "LA.BF64.1.2.2-03240-8x94.0" Issue-Id: SAMBAR-1280 Change-Id: Ia1481c2e8726c24c721e78df680ad8851f74f02b
| * Merge "policy_hal: fix for FM playback without active music stream"Linux Build Service Account2016-01-141-2/+3
| |\
| | * policy_hal: fix for FM playback without active music streamMingming Yin2016-01-131-2/+3
| | | | | | | | | | | | | | | | | | | | | | | | - Increase primary output ref count before getting new device - This prevents to use NULL device for FM playback. Change-Id: Idd2b36dab1990cdc1c1f603c5bdfd53b53c5c311 CRs-Fixed: 957807
| * | Merge "policy: Delay FM volume update to start of FM playback"Linux Build Service Account2016-01-142-3/+10
| |\|
| | * policy: Delay FM volume update to start of FM playbackHaynes Mathew George2016-01-132-3/+10
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Delay update of FM volume to start of FM playback. This avoids unnecessary set_parameter calls on primary output whenever volume changes on primary output. Unnecessary calls on primary output can cause breaks in audio since FastMixer will be put to HOT_IDLE more often CRs-Fixed: 948361 Change-Id: Id614d32df7f35ac93dd029a08b10e19f64ac631e
| * | policy_hal: fix for volume burst during FM device switchMingming Yin2016-01-131-1/+2
| |/ | | | | | | | | | | | | | | | | | | - Issue: Volume burst is heard during device switch (headphone and speaker) for FM playback - Root cause: delayMs passed to CheckAndSetVolume is not applied for FM playback - Fix: Double delayMs and apply FM volume with this delay Change-Id: Ifa0b214d16527973277a5c84b8a4f3fbd0760d07
| * policy: Avoid unnecessary set_parameter callsHaynes Mathew George2016-01-122-4/+11
| | | | | | | | | | | | | | | | | | | | | | | | | | APM calls a set_parameter on primary output with key fm_volume whenever a checkAndSetVolume is called. This is done irrespective of whether the volume values are same of different. A side effect of doing so is that the FastMixer associated with the MixerThread will be put into HOT_IDLE unnecessarily which in turn can cause glitches/breaks in audio. CRs-Fixed: 948361 Change-Id: I7fd8b7340e6c78ee6d00b41ccf679582338c011a
| * policy-hal: Fix for alarm tone heard even after dismissing.Ramjee Singh2015-12-231-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | -For stopping the alarm tone played during in call, handleIncallSonification must be called with second parameter i.e. starting as false from setPhoneState when the call is disconnected -isInCall method retuns false,hence handleIncallSonification is not getting called on disconnecting the call Change-Id: Ia6398eb1b0a3cb117a16f9fd4cb3637f5ce17be8
* | Merge tag 'LA.BF64.1.2.2-02240-8x94.0' into cm-13.0-caf-8994Scott Mertz2016-01-112-10/+64
|\| | | | | | | "LA.BF64.1.2.2-02240-8x94.0"
| * HAL: Identify the track offload sessionSharad Sangle2015-12-101-2/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The current logic for identifying the track offload session is using strlcat in wrong way, third parameter is given the string length to be appended, but in fact it should be the size of the destination buffer. Due to this the string returned is missing the last character, in case of "true", it contains "tru" due this the session is not considered as a track offload by the calling module. To fix this use sizeof of the destination buffer instead of string length of string to be appended. Change-Id: I3d456aad2ca49095a0dfe5883f4374e22a437256 CRs-Fixed: 946670
| * Merge "libaudiopolicymanager: DirectPCM: fix 44.1 clip playing noisy issue"Linux Build Service Account2015-12-091-6/+18
| |\
| | * libaudiopolicymanager: DirectPCM: fix 44.1 clip playing noisy issuevivek mehta2015-12-041-6/+18
| | | | | | | | | | | | | | | | | | | | | | | | | | | - adding both deepbuffer and direct pcm is causing issue where wrong config is set to ADM - add deepbuffer flag when fail to get direct output if deepbuffer was forced Change-Id: I3bd65002c7a8895d461df257111faf22101cbe30
| * | audio: update flags for selection of VoIP direct outputKarthik Reddy Katta2015-12-041-2/+2
| |/ | | | | | | | | | | | | | | | | | | | | | | Ringtone mute is observed in LINE app MT call over BT headset. VoIP direct output isn't returned due to incorrect flags update. Primary output is returned even though the FLAG_DIRECT is set, this in turn causing issues when renderer queries for getPosition(), which is resulting in mute. Fix this by updating flags, so that track gets VoIP direct output in this scenario. Change-Id: I6d570ae6ed9e0a7506cb29be5b41687417b7d221 CRs-Fixed: 893558
| * Merge DAX2 changesGao Jie2015-11-251-0/+42
| | | | | | | | | | | | | | Change-Id: If6fcfdef0b051f9015c127bdd7d28e3668e47435 (cherry picked from commit 4879ebb6d14558695044be0a22a89fc8b7ac24ec) Change-Id: I5ecf6c792bf1d5c481879a38231acfdb2a3ccc3e
* | audio: Fix flac offload not workingKeith Mok2015-12-171-0/+5
| | | | | | | | | | | | | | | | If AUDIO_FEATURE_ENABLED_FLAC_OFFLOAD rather than AUDIO_FEATURE_ENABLED_EXTN_FLAC_OFFLOAD is true, we should enable COMPRESS_METADATA_NEEDED also. Change-Id: I8f3b0cea2f59a9967d845133716dd7db5f3facb0
* | Merge remote-tracking branch 'caf/LA.BF64.1.2.2_rb4.8' into cm-13.0Scott Mertz2015-12-033-32/+27
|\ \
| * \ Merge AU_LINUX_ANDROID_LA.BF64.1.2.2_RB4.06.00.00.180.009 on remote branchLinux Build Service Account2015-11-230-0/+0
| |\ \ | | |/ | |/| | | | Change-Id: I2abc7812ea07ca2302c7c29cbf4e9cfd81997363
| * | Merge "hal: Fix condition for offload small buffer use"Linux Build Service Account2015-11-211-1/+1
| |\ \
| | * | hal: Fix condition for offload small buffer useAmit Shekhar2015-11-201-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Fix condition for offload small buffer use by masking format with AUDIO_FORMAT_MAIN_MASK before comparing with AUDIO_FORMAT_PCM_OFFLOAD Change-Id: Ia81fdc6061fbbf8c203bdfb3074730c6436fdf73 CRs-Fixed: 938198
| * | | HAL: Identify Track offload sessionAlexy Joseph2015-11-201-0/+15
| |/ / | | | | | | | | | | | | | | | | | | | | | | | | Support getparameter to retrieve if a particular output is associated with a track offloaded session Change-Id: Ibcc986514f802e59c514fdaa8b2d7f53841047a2 CRs-Fixed: 933740
| * | hal: policy: fallback to deep buffer if stream is musicvivek mehta2015-11-101-31/+11
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | - in case of direct PCM, Direct flag is added explicitly which is causing deep buffer flag not to be added for Music case. - absence of deep buffer flag is causing low latency path to be selected by default if direct pcm fails. Fastmixer path has higher power footprint. - to avoid low latency and allow stream music to use deep buffer path check if direct flag is added explicitly, if so allow adding deep buffer flag. This will allow selecting deep buffer output. Change-Id: Icc1de9f4ac3ff7981d2325fbda4891d9b8476df7
* | | policy_hal: Line up default features with rest of HALEthan Chen2015-11-271-4/+4
| | | | | | | | | | | | | | | | | | | | | | | | * PCM_OFFLOAD, EXTN_FORMATS, PROXY_DEVICE are all enabled by default in the HAL/post_proc, make them enabled by default for the policy HAL as well. Change-Id: I88a90ec138854cc5153fc9ed72dd629a92895ad3
* | | Merge remote-tracking branch 'caf/LA.BF64.1.2.2_rb4.7' into cm-13.0-caf-8994Scott Mertz2015-11-169-34/+53
|\ \ \ | | |/ | |/| | | | Change-Id: Idee04b3a7c4685a852b853b101ecfc14b4bd74f7
| * | Merge AU_LINUX_ANDROID_LA.BF64.1.2.2_RB4.06.00.00.180.007 on remote branchLinux Build Service Account2015-11-120-0/+0
| |\ \ | | |/ | |/| | | | Change-Id: I109171f75ce0d76cbe657651c4eecec3ce29c8d4
| * | Merge "hal: Fix device selection at start of the voice call"Linux Build Service Account2015-11-091-3/+3
| |\ \
| | * | hal: Fix device selection at start of the voice callShiv Maliyappanahalli2015-11-041-3/+3
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Voice call setup latency value is exceeding 500 msec. It is due to primary output selecting handset instead of voice handset causing the extra delay. Fix this by starting voice usecase first before changing device for primary output. Change-Id: I39ac8186b62d86b2047bf73860e7027d15df7fd2
| * | | Merge "hal: fix compilation issues with audio FM extention"Linux Build Service Account2015-11-056-26/+13
| |\ \ \
| | * | | hal: fix compilation issues with audio FM extentionMingming Yin2015-11-026-26/+13
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | - Update FM device references to match latest definitions to fix compilation of FM extention. - Update FM feature flag for power optimization. Change-Id: I17a9e240e552c4089d3e9c7a25e437fb5b5433f9
| * | | | policy_hal: Fix pop noise when MO call is started, while playing musicVenkata Narendra Kumar Gutta2015-11-021-7/+0
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Pops heard when MO call is initiated while music playback is in progress. This is due to a new device is set up for the output even before the leftover buffers were drained out on the previous device. This change is to make sure to add proper delay while device routing, which is equal to twice of Output descriptor latency. CRs-Fixed: 906398 Change-Id: I3b8059b8966251e0e0cc979c968205dffebc14f0
| * | | | policy_hal: add support for fm device loopbackDhananjay Kumar2015-11-012-0/+28
| |/ / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | Use device connection event against AUDIO_DEVICE_OUT_FM to configure FM device loopback. Update primary output to get routing and volume events. Change-Id: I5ddce36581568d377a08ea5afe470bdff4349ec7
| * | | Merge "policy_hal: Reject offload path for AV streaming usecase"Linux Build Service Account2015-10-311-0/+8
| |\ \ \
| | * | | policy_hal: Reject offload path for AV streaming usecaseManish Dewangan2015-10-291-0/+8
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | -Currently offload path is being used for AV streaming usecase. -This sometime results in framedrop/glitch as video rendering is getting delayed due to delay in filling audio compress buffer. -Disable compress offload for AV streaming usecase. With AV streaming usecase will be via PCM offload path. Change-Id: I799ee43f17975b33e37b94e222c4ce41f36bbc96
| * | | | Merge "audio_policy: modify few methods to appropriately override base"Linux Build Service Account2015-10-302-9/+23
| |\ \ \ \ | | |/ / / | |/| | |
| | * | | audio_policy: modify few methods to appropriately override baseDhananjay Kumar2015-10-272-9/+23
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Few methods in AudioPolicyManagerCustom is not overriding its base implemenation due to signature mismatch. Since they act as separate method they are not getting called to override base implemenation. Correct signatures to implement overriding instead of overloading. Change-Id: I6fe8fd6890ee8e896b6430f3ad170850d77b17d5
* | | | | Merge branch 'LA.BF64.1.2.2_rb4.6' of ↵Steve Kondik2015-11-0410-122/+307
|\ \ \ \ \ | | |_|_|/ | |/| | | | | | | | | | | | | | | | | | git://codeaurora.org/platform/hardware/qcom/audio into cm-13.0 Change-Id: Id805aac771873aae865d263f2f99d6941d8e1bf8
| * | | | audio_policy: modify few methods to appropriately override baseDhananjay Kumar2015-10-292-9/+23
| |/ / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Few methods in AudioPolicyManagerCustom is not overriding its base implemenation due to signature mismatch. Since they act as separate method they are not getting called to override base implemenation. Correct signatures to implement overriding instead of overloading. Change-Id: I6fe8fd6890ee8e896b6430f3ad170850d77b17d5
| * | | Merge "hal: Increase PCM offload buffering"Linux Build Service Account2015-10-272-38/+14
| |\ \ \ | | |/ / | |/| |
| | * | hal: Increase PCM offload bufferingAlexy Joseph2015-10-242-38/+14
| | | | | | | | | | | | | | | | | | | | | | | | | | | | Increase PCM offload buffering to 40ms CRs-Fixed: 924443 Change-Id: I67be148f85acc7c155ed222d652d5fedfb6a5e9b
| * | | Merge "audio_policy: Add extra check before restoring device in stopOutput"Linux Build Service Account2015-10-241-1/+8
| |\ \ \
| | * | | audio_policy: Add extra check before restoring device in stopOutputZhou Song2015-10-211-1/+8
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | When certain game apks run with sound enabled, UI lag will be observed. This is due to device to be restored with double HAL latency even when it's the same with the previous device, since setOutputDevice is synchronous call now, then it will block the subsequent calling to APM until the latency time elapsed. When the device to be restored is the same as previous device, set the latency to 0. Ported from Change-Id: I2c92675cedd8930430f81147c7f733d8dd70798c Conflicts: policy_hal/AudioPolicyManager.cpp Change-Id: I4582cdc98f4cf8189d89b648e5c3f6e0ce8089ab
| * | | | Merge "hal: check offload_callback not NULL before calling"Linux Build Service Account2015-10-201-1/+1
| |\ \ \ \ | | |/ / / | |/| | |
| | * | | hal: check offload_callback not NULL before callingvivek mehta2015-10-161-2/+2
| | | |/ | | |/| | | | | | | | | | | | | | | | | | | | | | | | | - in case when offload o/p stream is created in blocking mode flinger do not sets the callback pointer. So check for offload callback pointer before calling. Change-Id: I4192c8cc793c48d7647d7652366c8c92ceb469cc
| * | | Merge "policy-hal: squash multiple change/fixes in to policy"Linux Build Service Account2015-10-203-54/+80
| |\ \ \ | | |_|/ | |/| |
| | * | policy-hal: squash multiple change/fixes in to policySharad Sangle2015-10-153-54/+80
| | |/ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This change is squash of below changes. Only relevant changes have been ported and the other required changes are in base policy manager. audiopolicy: support extended feature in audiopolicymanager. Change-Id: I1925339b591cd29f11a71c287a2e53c0627e9e62 audiopolicy: additional change for extended feature Change-Id: I9bad6a294ddd7aee72f6f6a314666b892b730c8e audiopolicy: Do not route audio to HDMI if HDMI not connected Change-Id: I1c76ba3c3a1438463461ea7c41799633bd977283 audiopolicy: Fix for device selection during concurrent audio playback Change-Id: Ibf3c1bfd05b68d70c6cb9e8b9b95e989f27c2e75 CRs-Fixed: 894496 Change-Id: I00470a057cdeefff7bba3bffd333fb556f3d97e6
| * | Merge "hal: update error handling for pcm_prepare failures"Linux Build Service Account2015-10-171-14/+26
| |\ \