diff mbox series

[v2,08/17] audio: remove unused noop_conv() function

Message ID 20230206185237.8358-8-vr_qemu@t-online.de (mailing list archive)
State New, archived
Headers show
Series audio: improve callback interface for audio frontends | expand

Commit Message

Volker Rümelin Feb. 6, 2023, 6:52 p.m. UTC
The function audio_capture_mix_and_clear() no longer uses
audio_pcm_sw_write() to resample audio frames from one internal
buffer to another. For this reason, the noop_conv() function is
now unused. Remove it.

Acked-by: Mark Cave-Ayland <mark.cave-ayland@ilande.co.uk>
Signed-off-by: Volker Rümelin <vr_qemu@t-online.de>
---
 audio/audio.c | 8 --------
 1 file changed, 8 deletions(-)

Comments

Marc-André Lureau Feb. 22, 2023, 10:49 a.m. UTC | #1
On Mon, Feb 6, 2023 at 10:52 PM Volker Rümelin <vr_qemu@t-online.de> wrote:
>
> The function audio_capture_mix_and_clear() no longer uses
> audio_pcm_sw_write() to resample audio frames from one internal
> buffer to another. For this reason, the noop_conv() function is
> now unused. Remove it.
>
> Acked-by: Mark Cave-Ayland <mark.cave-ayland@ilande.co.uk>
> Signed-off-by: Volker Rümelin <vr_qemu@t-online.de>

Reviewed-by: Marc-André Lureau <marcandre.lureau@redhat.com>



> ---
>  audio/audio.c | 8 --------
>  1 file changed, 8 deletions(-)
>
> diff --git a/audio/audio.c b/audio/audio.c
> index 44eb7b63b4..556696b095 100644
> --- a/audio/audio.c
> +++ b/audio/audio.c
> @@ -381,13 +381,6 @@ void audio_pcm_info_clear_buf (struct audio_pcm_info *info, void *buf, int len)
>  /*
>   * Capture
>   */
> -static void noop_conv (struct st_sample *dst, const void *src, int samples)
> -{
> -    (void) src;
> -    (void) dst;
> -    (void) samples;
> -}
> -
>  static CaptureVoiceOut *audio_pcm_capture_find_specific(AudioState *s,
>                                                          struct audsettings *as)
>  {
> @@ -485,7 +478,6 @@ static int audio_attach_capture (HWVoiceOut *hw)
>          sw->info = hw->info;
>          sw->empty = 1;
>          sw->active = hw->enabled;
> -        sw->conv = noop_conv;
>          sw->ratio = ((int64_t) hw_cap->info.freq << 32) / sw->info.freq;
>          sw->vol = nominal_volume;
>          sw->rate = st_rate_start (sw->info.freq, hw_cap->info.freq);
> --
> 2.35.3
>


--
Marc-André Lureau
diff mbox series

Patch

diff --git a/audio/audio.c b/audio/audio.c
index 44eb7b63b4..556696b095 100644
--- a/audio/audio.c
+++ b/audio/audio.c
@@ -381,13 +381,6 @@  void audio_pcm_info_clear_buf (struct audio_pcm_info *info, void *buf, int len)
 /*
  * Capture
  */
-static void noop_conv (struct st_sample *dst, const void *src, int samples)
-{
-    (void) src;
-    (void) dst;
-    (void) samples;
-}
-
 static CaptureVoiceOut *audio_pcm_capture_find_specific(AudioState *s,
                                                         struct audsettings *as)
 {
@@ -485,7 +478,6 @@  static int audio_attach_capture (HWVoiceOut *hw)
         sw->info = hw->info;
         sw->empty = 1;
         sw->active = hw->enabled;
-        sw->conv = noop_conv;
         sw->ratio = ((int64_t) hw_cap->info.freq << 32) / sw->info.freq;
         sw->vol = nominal_volume;
         sw->rate = st_rate_start (sw->info.freq, hw_cap->info.freq);