Promotion of av-userspace.lnx.1.0-00039.
CRs Change ID Subject
--------------------------------------------------------------------------------------------------------------
966076 I3566705f745247075d6f12b625fcf929970ff507 hal: fix no audio issue in voice call
946181 I3675da4504a93a084ad582e5e6eb7c75f38e736e hal: fix resource leak issues in platform.c
958813 I4d8d4026d3abb388c4b1ea2c8de328a61d9bbd81 hal: fix to enable aanc handset dynamically during voice
Change-Id: Ie40cdcdae9993c3db932df4cb921ecfd8af98718
CRs-Fixed: 966076, 958813, 946181
diff --git a/hal/audio_extn/audio_extn.c b/hal/audio_extn/audio_extn.c
index 7c10cb7..e65ec76 100644
--- a/hal/audio_extn/audio_extn.c
+++ b/hal/audio_extn/audio_extn.c
@@ -1,5 +1,5 @@
/*
- * Copyright (c) 2013-2015, The Linux Foundation. All rights reserved.
+ * Copyright (c) 2013-2016, The Linux Foundation. All rights reserved.
* Not a Contribution.
*
* Copyright (C) 2013 The Android Open Source Project
@@ -430,11 +430,13 @@
// Refresh device selection for anc playback
list_for_each(node, &adev->usecase_list) {
usecase = node_to_item(node, struct audio_usecase, list);
- if (usecase->type == PCM_PLAYBACK) {
+ if (usecase->type != PCM_CAPTURE) {
if (usecase->stream.out->devices == \
AUDIO_DEVICE_OUT_WIRED_HEADPHONE ||
usecase->stream.out->devices == \
- AUDIO_DEVICE_OUT_WIRED_HEADSET) {
+ AUDIO_DEVICE_OUT_WIRED_HEADSET ||
+ usecase->stream.out->devices == \
+ AUDIO_DEVICE_OUT_EARPIECE) {
select_devices(adev, usecase->id);
ALOGV("%s: switching device completed", __func__);
break;
diff --git a/hal/audio_hw.c b/hal/audio_hw.c
index f610096..ae7853e 100644
--- a/hal/audio_hw.c
+++ b/hal/audio_hw.c
@@ -959,7 +959,7 @@
* usecase. This is to avoid switching devices for voice call when
* check_usecases_codec_backend() is called below.
*/
- if (voice_is_in_call(adev) && adev->mode == AUDIO_MODE_IN_CALL) {
+ if (voice_is_in_call(adev) && adev->mode != AUDIO_MODE_NORMAL) {
vc_usecase = get_usecase_from_list(adev,
get_usecase_id_from_usecase_type(adev, VOICE_CALL));
if ((vc_usecase) && ((vc_usecase->devices & AUDIO_DEVICE_OUT_ALL_CODEC_BACKEND) ||
@@ -3607,8 +3607,7 @@
if (adev->mode != mode) {
ALOGD("%s: mode %d\n", __func__, mode);
adev->mode = mode;
- if ((mode == AUDIO_MODE_NORMAL || mode == AUDIO_MODE_IN_COMMUNICATION) &&
- voice_is_in_call(adev)) {
+ if ((mode == AUDIO_MODE_NORMAL) && voice_is_in_call(adev)) {
voice_stop_call(adev);
platform_set_gsm_mode(adev->platform, false);
adev->current_call_output = NULL;
diff --git a/hal/msm8916/platform.c b/hal/msm8916/platform.c
index 4839451..7cdfc62 100644
--- a/hal/msm8916/platform.c
+++ b/hal/msm8916/platform.c
@@ -1677,6 +1677,7 @@
ALOGE("%s: Failed to init audio route controls, aborting.",
__func__);
free(my_data);
+ mixer_close(adev->mixer);
return NULL;
}
adev->snd_card = snd_card_num;
@@ -1686,6 +1687,7 @@
}
retry_num = 0;
snd_card_num++;
+ mixer_close(adev->mixer);
}
if (snd_card_num >= MAX_SND_CARD) {
diff --git a/hal/msm8960/platform.c b/hal/msm8960/platform.c
index e7d6920..be50289 100644
--- a/hal/msm8960/platform.c
+++ b/hal/msm8960/platform.c
@@ -243,6 +243,7 @@
adev->audio_route = audio_route_init(MIXER_CARD, MIXER_XML_PATH);
if (!adev->audio_route) {
ALOGE("%s: Failed to init audio route controls, aborting.", __func__);
+ mixer_close(adev->mixer);
return NULL;
}
diff --git a/hal/msm8974/platform.c b/hal/msm8974/platform.c
index e091547..65ef702 100644
--- a/hal/msm8974/platform.c
+++ b/hal/msm8974/platform.c
@@ -1335,6 +1335,7 @@
if (!snd_card_name) {
ALOGE("failed to allocate memory for snd_card_name\n");
free(my_data);
+ mixer_close(adev->mixer);
return NULL;
}
ALOGD("%s: snd_card_name: %s", __func__, snd_card_name);
@@ -1401,6 +1402,7 @@
free(my_data);
free(snd_card_name);
free(snd_card_name_t);
+ mixer_close(adev->mixer);
return NULL;
}
adev->snd_card = snd_card_num;
@@ -1409,6 +1411,7 @@
}
retry_num = 0;
snd_card_num++;
+ mixer_close(adev->mixer);
}
if (snd_card_num >= MAX_SND_CARD) {