commit 11393709a7defb6353f5e2ed2b3c0b331911970f
author Naresh Tanniru <ntanniru@codeaurora.org> Wed Sep 14 11:25:37 2016 +0530
committer Gerrit - the friendly Code Review server <code-review@localhost> Thu Sep 22 05:23:40 2016 -0700
tree 7d2858d84e7778d22e3c054bbf76d19b56f88dbd
parent 63b9307d726aa6ac565a4e55539210a94c8c6eed

audiopolicy: Avoid spurious routing in APM::stopSource

- On AudioOutput::stopSource, ignore routing calls
on an output descriptor to the same device

Change-Id: I10026d3d9147e45d3ed2b93160922bc3bb4bc2d0

policy_hal/AudioPolicyManager.cpp

diff --git a/policy_hal/AudioPolicyManager.cpp b/policy_hal/AudioPolicyManager.cpp
index 1b113ed..9b950d9 100644
--- a/policy_hal/AudioPolicyManager.cpp
+++ b/policy_hal/AudioPolicyManager.cpp
@@ -1073,6 +1073,7 @@
                         outputDesc->sharesHwModuleWith(desc) &&
                         (newDevice != desc->device())) {
                         audio_devices_t dev = getNewOutputDevice(mOutputs.valueFor(curOutput), false /*fromCache*/);
+                        bool force = desc->device() != dev;
                         uint32_t delayMs;
                         if (dev == prevDevice) {
                             delayMs = 0;
@@ -1081,7 +1082,7 @@
                         }
                         setOutputDevice(desc,
                                     dev,
-                                    true,
+                                    force,
                                     delayMs);
                 }
             }