qemu-arm
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [PATCH] display: xlnx_dp: Provide sufficient bytes for silent audio


From: Philippe Mathieu-Daudé
Subject: Re: [PATCH] display: xlnx_dp: Provide sufficient bytes for silent audio channel
Date: Fri, 15 Nov 2019 21:12:02 +0100
User-agent: Mozilla/5.0 (X11; Linux x86_64; rv:68.0) Gecko/20100101 Thunderbird/68.1.1

Cc'ing Gerd (Audio)

On 11/15/19 4:56 PM, Sai Pavan Boddu wrote:
Fill the audio channel with required number of bytes to cover the
elapsed time. This prevents rate control reset, and avoids debug prints
like below

log:
        Resetting rate control (65692 samples)
        ...
        Resetting rate control (65721 samples)
        ...

Signed-off-by: Sai Pavan Boddu <address@hidden>
---
  hw/display/xlnx_dp.c | 9 +++++++--
  1 file changed, 7 insertions(+), 2 deletions(-)

diff --git a/hw/display/xlnx_dp.c b/hw/display/xlnx_dp.c
index fde3b21..7058443 100644
--- a/hw/display/xlnx_dp.c
+++ b/hw/display/xlnx_dp.c
@@ -394,13 +394,18 @@ static void xlnx_dp_audio_callback(void *opaque, int 
avail)
              written = AUD_write(s->amixer_output_stream,
                                  &s->out_buffer[s->data_ptr], s->byte_left);
          } else {
+             int len_to_copy;
              /*
               * There is nothing to play.. We don't have any data! Fill the
               * buffer with zero's and send it.
               */
              written = 0;
-            memset(s->out_buffer, 0, 1024);
-            AUD_write(s->amixer_output_stream, s->out_buffer, 1024);
+            while (avail) {
+                len_to_copy = MIN(AUD_CHBUF_MAX_DEPTH, avail);
+                memset(s->out_buffer, 0, len_to_copy);
+                avail -= AUD_write(s->amixer_output_stream, s->out_buffer,
+                                   len_to_copy);
+            }
          }
      } else {
          written = AUD_write(s->amixer_output_stream,





reply via email to

[Prev in Thread] Current Thread [Next in Thread]