commit 4b5c425a08b5d23a46e4d1ef29e10817b71934d6
author Jaideep Sharma <quic_jaidshar@quicinc.com> Thu Jul 27 14:47:32 2023 +0530
committer Eric Laurent <elaurent@google.com> Fri Aug 04 15:05:14 2023 +0200
tree 104817cdd79ee0322ebd9bba77d705ffaf64aa1e
parent 751cfafffc67675ffce45b2f267507bdae364952

audiopolicy: Modify logging in setOutputDevices

Modify logging in setOutputDevices to log ioHandles, mixport
names for better debugging.

Also, modify the method signature to include caller, as
setOutputDevices can be modified from multiple places,
having a caller context is helpful during multiple
device connection scenarios.

Test: Build and check logs while running usecases
Bug: 293535290

Change-Id: I461bf184bfab1c45ebe0170008fc59aa5a6a29a4

services/audiopolicy/managerdefault/AudioPolicyManager.h

diff --git a/services/audiopolicy/managerdefault/AudioPolicyManager.h b/services/audiopolicy/managerdefault/AudioPolicyManager.h
index 863c785..509cc79 100644
--- a/services/audiopolicy/managerdefault/AudioPolicyManager.h
+++ b/services/audiopolicy/managerdefault/AudioPolicyManager.h
@@ -526,6 +526,7 @@
 
         /**
          * @brief setOutputDevices change the route of the specified output.
+         * @param caller of the method
          * @param outputDesc to be considered
          * @param device to be considered to route the output
          * @param force if true, force the routing even if no change.
@@ -539,7 +540,8 @@
          * @return the number of ms we have slept to allow new routing to take effect in certain
          *        cases.
          */
-        uint32_t setOutputDevices(const sp<SwAudioOutputDescriptor>& outputDesc,
+        uint32_t setOutputDevices(const char *caller,
+                                  const sp<SwAudioOutputDescriptor>& outputDesc,
                                   const DeviceVector &device,
                                   bool force = false,
                                   int delayMs = 0,