From 439df6e295363c6a325fb56a7f9fc9ef01ca9936 Mon Sep 17 00:00:00 2001
From: Mikhail Naganov <mnaganov@google.com>
Date: Fri, 11 Mar 2022 02:07:45 +0000
Subject: [PATCH] Mark 'session' field in audio HAL AIDL as unused.

The 'session' field is only used by the framework (if at all),
so it being moved to the HAL AIDL to the system-specific part.

Also update the comment on AudioPort.

Bug: 205884982
Test: m
Merged-In: Ia214c343bbf08c7944ea5ebf6140a92821941b46
Change-Id: Ia214c343bbf08c7944ea5ebf6140a92821941b46
(cherry picked from commit 8abc0314a7e6c6b6569d5e2a21e17c3040101d32)
---
 media/aidl/android/media/audio/common/AudioPort.aidl    | 5 +++--
 media/aidl/android/media/audio/common/AudioPortExt.aidl | 5 ++++-
 2 files changed, 7 insertions(+), 3 deletions(-)

diff --git a/media/aidl/android/media/audio/common/AudioPort.aidl b/media/aidl/android/media/audio/common/AudioPort.aidl
index 84675e3a00a64..d32b840ebfe8a 100644
--- a/media/aidl/android/media/audio/common/AudioPort.aidl
+++ b/media/aidl/android/media/audio/common/AudioPort.aidl
@@ -23,8 +23,9 @@ import android.media.audio.common.AudioProfile;
 import android.media.audio.common.ExtraAudioDescriptor;
 
 /**
- * Audio port structure describes the capabilities of an audio port
- * as well as its current configuration.
+ * Audio port structure describes the capabilities of an audio port.
+ * This is a "blueprint" which contains all the possible configurations
+ * that are supported by the port.
  *
  * {@hide}
  */
diff --git a/media/aidl/android/media/audio/common/AudioPortExt.aidl b/media/aidl/android/media/audio/common/AudioPortExt.aidl
index c4681cbb182e6..eadc0ab7c7c53 100644
--- a/media/aidl/android/media/audio/common/AudioPortExt.aidl
+++ b/media/aidl/android/media/audio/common/AudioPortExt.aidl
@@ -34,6 +34,9 @@ union AudioPortExt {
     AudioPortDeviceExt device;
     /** Information specific to mix ports. */
     AudioPortMixExt mix;
-    /** Audio session identifier. */
+    /**
+     * NOT USED. Framework audio session identifier.
+     * Use android.media.AudioPortExtSys.session on the system side.
+     */
     int session;
 }