commit 543fb84c764406213ce0e5d832a199c755dad284
author Anand Mohan <quic_anamoh@quicinc.com> Wed Aug 16 11:36:06 2023 +0530
committer Anand Mohan <quic_anamoh@quicinc.com> Tue Sep 12 00:06:01 2023 -0700
tree 4678d7a95f768ef922a1b2d36985f93c188c73a2
parent 4fa4617881fee88dff929150452ed27c0aac4b80

pal: skip a2dp reconfiguration if a2dp is in suspended state

Reconfig process involves suspending of a2dp streams and then resuming
it on a2dp. In scenarios where there is a delay in setting SCO mode
after a2dp is suspended, APM will send routing to A2dp for some streams.
Start of stream over A2dp is handled during suspend by switching to
Speaker/Handset in stream start. If reconfigA2dp setparam is received
when stream is in open state and not started, reconfig will proceed as
after stream open stream is considered as active. Reconfiguration will
try to switch previously suspended streams to A2dp during resume part
which is not correct as A2dp is suspended.

Fixed this by checking a2dpSuspended state is false before starting
reconfiguration.

Change-Id: Iab6ec08ec09effa248cac7e8c7e4778d23bc1f28

resource_manager/src/ResourceManager.cpp

diff --git a/resource_manager/src/ResourceManager.cpp b/resource_manager/src/ResourceManager.cpp
index 374d1ff..7e54817 100644
--- a/resource_manager/src/ResourceManager.cpp
+++ b/resource_manager/src/ResourceManager.cpp
@@ -9371,7 +9371,8 @@
 
                 dev->setDeviceParameter(param_id, param_payload);
                 dev->getDeviceParameter(param_id, (void **)&current_param_bt_a2dp);
-                if (current_param_bt_a2dp->reconfig == true) {
+                if ((current_param_bt_a2dp->reconfig == true) &&
+                    (current_param_bt_a2dp->a2dp_suspended == false)) {
                     param_bt_a2dp.a2dp_suspended = true;
                     mResourceManagerMutex.unlock();
                     status = dev->setDeviceParameter(PAL_PARAM_ID_BT_A2DP_SUSPENDED,