Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit 01e54e61 authored by Alexy Joseph's avatar Alexy Joseph
Browse files

hal: Reduce offload latency and log spam

Offload latency is set to 96. Change
it to 50 to reflect the rendering latency more correctly
Modify pcm offload latency to return offload
latency rather than the latency based on buffering
since volume is applied immediately for offload use
cases. Also reduce log spam in offload use cases

Change-Id: If28d49d66e8f989074c4c7eaec523855e9f4290a
CRs-Fixed: 787448
parent e0b22e0e
Loading
Loading
Loading
Loading
+3 −9
Original line number Diff line number Diff line
@@ -75,7 +75,7 @@

#define COMPRESS_OFFLOAD_NUM_FRAGMENTS 4
/* ToDo: Check and update a proper value in msec */
#define COMPRESS_OFFLOAD_PLAYBACK_LATENCY 96
#define COMPRESS_OFFLOAD_PLAYBACK_LATENCY 50
#define COMPRESS_PLAYBACK_VOLUME_MAX 0x2000

#define PROXY_OPEN_RETRY_COUNT           100
@@ -1995,12 +1995,6 @@ static uint32_t out_get_latency(const struct audio_stream_out *stream)
    uint32_t latency = 0;

    if (is_offload_usecase(out->usecase)) {
        if (out->use_small_bufs == true)
            latency = ((out->compr_config.fragments *
                   out->compr_config.fragment_size * 1000) /
                   (out->sample_rate * out->compr_config.codec->ch_in *
                   audio_bytes_per_sample(out->format)));
        else
        latency = COMPRESS_OFFLOAD_PLAYBACK_LATENCY;
    } else {
        latency = (out->config.period_count * out->config.period_size * 1000) /
@@ -2093,7 +2087,7 @@ static ssize_t out_write(struct audio_stream_out *stream, const void *buffer,
    }

    if (is_offload_usecase(out->usecase)) {
        ALOGD("copl(%p): writing buffer (%zu bytes) to compress device", out, bytes);
        ALOGVV("copl(%p): writing buffer (%zu bytes) to compress device", out, bytes);
        if (out->send_new_metadata) {
            ALOGD("copl(%p):send new gapless metadata", out);
            compress_set_gapless_metadata(out->compr, &out->gapless_mdata);