commit b0f5dc3b65eb3d9628d345b445a2739bc5507181
author Haynes Mathew George <hgeorge@codeaurora.org> Fri Oct 06 18:35:12 2017 -0700
committer Haynes Mathew George <hgeorge@codeaurora.org> Wed Oct 11 18:01:49 2017 -0700
tree 71d3650686c299e91138e6338c96ff91f810c9c0
parent fc9149ca59deb6f6b5d9da6f7b6946dffdd5171f

audio: Add device latency in out_get_render_position

Account for device latency when returning frames returned
in out_get_render_position

CRs-Fixed: 2114873
Change-Id: If5e9e1d6b68f9668c503904e86212568e9f41595

hal/audio_hw.c

diff --git a/hal/audio_hw.c b/hal/audio_hw.c
index d0c0cfa..b2e9dec 100644
--- a/hal/audio_hw.c
+++ b/hal/audio_hw.c
@@ -524,6 +524,17 @@
     return 0;
 }
 
+static inline void adjust_frames_for_device_delay(struct stream_out *out,
+                                                  uint32_t *dsp_frames) {
+    // Adjustment accounts for A2dp encoder latency with offload usecases
+    // Note: Encoder latency is returned in ms.
+    if (AUDIO_DEVICE_OUT_ALL_A2DP & out->devices) {
+        unsigned long offset =
+                (audio_extn_a2dp_get_encoder_latency() * out->sample_rate / 1000);
+        *dsp_frames = (*dsp_frames > offset) ? (*dsp_frames - offset) : 0;
+    }
+}
+
 __attribute__ ((visibility ("default")))
 bool audio_hw_send_gain_dep_calibration(int level) {
     bool ret_val = false;
@@ -4052,6 +4063,7 @@
         if (!out->non_blocking && !(out->flags & AUDIO_OUTPUT_FLAG_COMPRESS_OFFLOAD)) {
             *dsp_frames = get_actual_pcm_frames_rendered(out);
              ALOGVV("dsp_frames %d sampleRate %d",(int)*dsp_frames,out->sample_rate);
+             adjust_frames_for_device_delay(out, dsp_frames);
              return 0;
         }
 
@@ -4080,11 +4092,13 @@
             ret = -EINVAL;
         } else {
             ret = 0;
+            adjust_frames_for_device_delay(out, dsp_frames);
         }
         pthread_mutex_unlock(&out->lock);
         return ret;
     } else if (audio_is_linear_pcm(out->format)) {
         *dsp_frames = out->written;
+        adjust_frames_for_device_delay(out, dsp_frames);
         return 0;
     } else
         return -EINVAL;