commit 06b6c0f393cde2388bc85ffdbb9d7f7163cd7eb1
author Dhananjay Kumar <dhakumar@codeaurora.org> Wed Jul 24 14:28:25 2019 +0530
committer Gerrit - the friendly Code Review server <code-review@localhost> Sun Jul 28 23:21:58 2019 -0700
tree 0000e07bc0a5e4e5cc00d8c5bf5e5f15109652a5
parent b150d82cd7f4d0969f5dc72a7582f79de76f0e5e

hal: fix fragment size calculation for compressed offload usecase

Compress offload fragment size is using duration_us provided in
offload info even if its set to -1, leading to incorrect and smaller
value which in turns results in higher power consumption.
Update check for duration us to exclude invalid values and use it for
calculating fragment size only if its within specified range.

Change-Id: Ib134a67d95fe663f5f77f688469bee1bf0ce1708

hal/msm8974/platform.c

diff --git a/hal/msm8974/platform.c b/hal/msm8974/platform.c
index 5cb1189..2b9e41d 100644
--- a/hal/msm8974/platform.c
+++ b/hal/msm8974/platform.c
@@ -8057,7 +8057,9 @@
     }
 
     /* Use client specified buffer size if mentioned */
-    if ((info != NULL) && (info->duration_us > 0)) {
+    if ((info != NULL) &&
+        (info->duration_us >= MIN_OFFLOAD_BUFFER_DURATION_MS) &&
+        (info->duration_us <= MAX_OFFLOAD_BUFFER_DURATION_MS)) {
         duration_ms = info->duration_us / 1000;
         channel_count = audio_channel_count_from_in_mask(info->channel_mask);